fix(android): add try catch for vfs processor chain get

Tencent · Sep 3, 2024 · 90fdf95 · 90fdf95
1 parent 05b0ba1
commit 90fdf95
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/modules/vfs/android/build.gradle b/modules/vfs/android/build.gradle
@@ -39,5 +39,6 @@ android {
 
 dependencies {
     compileOnly project(path: ':pool')
+    compileOnly project(path: ':hippy-support')
     implementation deps.annotation
 }
diff --git a/modules/vfs/android/src/main/java/com/tencent/vfs/VfsManager.java b/modules/vfs/android/src/main/java/com/tencent/vfs/VfsManager.java
@@ -19,13 +19,16 @@
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 
+import com.tencent.mtt.hippy.utils.LogUtils;
 import com.tencent.vfs.ResourceDataHolder.RequestFrom;
 import java.util.HashMap;
 import java.util.List;
 import java.util.concurrent.CopyOnWriteArrayList;
 
 public class VfsManager {
 
+    private static final String TAG = "VfsManager";
+
     @NonNull
     private final CopyOnWriteArrayList<Processor> mProcessorChain;
     private int mId;
@@ -118,7 +121,15 @@ private void traverseForward(@NonNull final ResourceDataHolder holder, final boo
         int index = holder.index + 1;
         if (index < mProcessorChain.size()) {
             holder.index = index;
-            Processor processor = mProcessorChain.get(index);
+            Processor processor = null;
+            try {
+                processor = mProcessorChain.get(index);
+            } catch (IndexOutOfBoundsException e) {
+                LogUtils.e(TAG, "traverseForward get index " + index + " processor exception: " + e.getMessage());
+            }
+            if (processor == null) {
+                return;
+            }
             if (isSync) {
                 boolean goBack = processor.handleRequestSync(holder);
                 if (goBack) {
@@ -158,7 +169,15 @@ private void traverseGoBack(@NonNull final ResourceDataHolder holder, final bool
         int index = holder.index - 1;
         if (index >= 0 && index < mProcessorChain.size()) {
             holder.index = index;
-            Processor processor = mProcessorChain.get(index);
+            Processor processor = null;
+            try {
+                processor = mProcessorChain.get(index);
+            } catch (IndexOutOfBoundsException e) {
+                LogUtils.e(TAG, "traverseGoBack get index " + index + " processor exception: " + e.getMessage());
+            }
+            if (processor == null) {
+                return;
+            }
             if (isSync) {
                 processor.handleResponseSync(holder);
                 traverseGoBack(holder, true);