Skip to content
This repository has been archived by the owner on Jun 3, 2021. It is now read-only.

[Android] fix callGcanvas导致的野指针问题 #2009

Closed
wants to merge 1 commit into from
Closed

[Android] fix callGcanvas导致的野指针问题 #2009

wants to merge 1 commit into from

Conversation

lvfen
Copy link
Contributor

@lvfen lvfen commented Jan 7, 2019

fix the issue #2008

A native crash with the stack:
SIGSEGV(SEGV_MAPERR):
#00 pc 0001a0a8 /system/lib/libc.so (strlen+71) [armeabi-v7a::bd65f13f1451c6c1a34a12381914242d]
#1 pc 002e693d /system/lib/libart.so (_ZN3art6mirror6String21AllocFromModifiedUtf8EPNS_6ThreadEPKc+8) [armeabi-v7a::383e31579b74445b901dcaeb5b37276b]
#2 pc 002b18ff /system/lib/libart.so (_ZN3art3JNI12NewStringUTFEP7_JNIEnvPKc+386) [armeabi-v7a::383e31579b74445b901dcaeb5b37276b]
#3 pc 00068b53 /lib/arm/libweexcore.so [armeabi-v7a::aa03e3a40249fd5e6a4e92bffa1aeb1a]
#4 pc 00067d41 /lib/arm/libweexcore.so (ZNSt17_Function_handlerIFSt10unique_ptrI9IPCResultSt14default_deleteIS1_EEP12IPCArgumentsEPS7_E9_M_invokeERKSt9_Any_dataS6+8) [armeabi-v7a::aa03e3a40249fd5e6a4e92bffa1aeb1a]
#5 pc 0006dc21 /lib/arm/libweexcore.so [armeabi-v7a::aa03e3a40249fd5e6a4e92bffa1aeb1a]
#6 pc 0006d1df /lib/arm/libweexcore.so [armeabi-v7a::aa03e3a40249fd5e6a4e92bffa1aeb1a]
#7 pc 00065fff /lib/arm/libweexcore.so (_ZN8WeexCore9WeexProxy6execJSEP7_JNIEnvP8_jobjectP8_jstringS6_S6_P13_jobjectArray+438) [armeabi-v7a::aa03e3a40249fd5e6a4e92bffa1aeb1a]
#8 pc 000610a1 /lib/arm/libweexcore.so [armeabi-v7a::aa03e3a40249fd5e6a4e92bffa1aeb1a]
#9 pc 00005605 /oat/arm/base.odex (oatexec+22021) [armeabi::74a1ccfee55579129f21b2286efa74e9]

the crash address is not the only and not zero.

the restore stack :
./aarch64-linux-android-addr2line -e /libweexcore.so -f 00068b53 00067d41 0006dc21 0006d1df 00065fff 000610a1

_ZN7_JNIEnv12NewStringUTFEPKc
/Android/sdk/ndk-bundle/sysroot/usr/include/jni.h:842
ZNSt17_Function_handlerIFSt10unique_ptrI9IPCResultSt14default_deleteIS1_EEP12IPCArgumentsEPS7_E9_M_invokeERKSt9_Any_dataS6
/Android/sdk/ndk-bundle/sources/cxx-stl/gnu-libstdc++/4.9/include/functional:2025 (discriminator 1)
ZNKSt8functionIFSt10unique_ptrI9IPCResultSt14default_deleteIS1_EEP12IPCArgumentsEEclES6
/Android/sdk/ndk-bundle/sources/cxx-stl/gnu-libstdc++/4.9/include/functional:2439 (discriminator 1)
_ZN12_GLOBAL__N_113IPCSenderImpl4sendEP9IPCBuffer
/weex_core/Source/IPC/IPCSender.cpp:80 (discriminator 1)
_ZN8WeexCore9WeexProxy6execJSEP7_JNIEnvP8_jobjectP8_jstringS6_S6_P13_jobjectArray
/weex_core/Source/android/jsengine/multiprocess/WeexProxy.cpp:432 (discriminator 1)

the weex core version: 0.19.0-release

@lvfen lvfen requested a review from YorkShen as a code owner January 7, 2019 08:56
@weex-bot
Copy link

weex-bot commented Jan 7, 2019

Messages
📖 No android file has been changed.
📖 android build verification finished.

Generated by 🚫 dangerJS

@lvfen lvfen closed this Jan 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants