[WEEX-562][Android] task may be null ,should be check ,not try/catch (#…

…1425)
apache · Aug 13, 2018 · 9eb436a · 9eb436a
1 parent 8320979
commit 9eb436a
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java b/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java
@@ -1744,14 +1744,15 @@ private void invokeCallJSBatch(Message message) {
           break;
         }
       }
-      task = ((ArrayList) task).toArray();
+      if (null != task){
+        task = ((ArrayList) task).toArray();
 
-      WXJSObject[] args = {
-              new WXJSObject(WXJSObject.String, instanceId),
-              WXWsonJSONSwitch.toWsonOrJsonWXJSObject(task)};
-
-      invokeExecJS(String.valueOf(instanceId), null, METHOD_CALL_JS, args);
+        WXJSObject[] args = {
+            new WXJSObject(WXJSObject.String, instanceId),
+            WXWsonJSONSwitch.toWsonOrJsonWXJSObject(task)};
 
+        invokeExecJS(String.valueOf(instanceId), null, METHOD_CALL_JS, args);
+      }
     } catch (Throwable e) {
       WXLogUtils.e("WXBridgeManager", e);
       String err = "invokeCallJSBatch#" + WXLogUtils.getStackTrace(e);