You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@weex.apache.org by "xingZhang (JIRA)" <ji...@apache.org> on 2017/11/07 13:37:00 UTC
[jira] [Created] (WEEX-96) Timer exposed on JSContext can not clear
and it has potential crash switching globalQueue
xingZhang created WEEX-96:
-----------------------------
Summary: Timer exposed on JSContext can not clear and it has potential crash switching globalQueue
Key: WEEX-96
URL: https://issues.apache.org/jira/browse/WEEX-96
Project: Weex
Issue Type: Bug
Reporter: xingZhang
Assignee: Adam Feng
Priority: Critical
1. timer can not be clear
As the WXJSCoreBridge file in [line 544](https://github.com/apache/incubator-weex/blob/master/ios/sdk/WeexSDK/Sources/Bridge/WXJSCoreBridge.m#L544) shows the code, the appId is a string , so it will never be cleared.
2. timer execute js function in global queue may have potential crash, it had better execution in current thread.
WXJSCoreBridge https://github.com/apache/incubator-weex/blob/master/ios/sdk/WeexSDK/Sources/Bridge/WXJSCoreBridge.m#L535
3. timerId will always be 1, not auto-increment as the count of timer increasing
https://github.com/apache/incubator-weex/blob/master/ios/sdk/WeexSDK/Sources/Bridge/WXJSCoreBridge.m#L513
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)