-
-
Notifications
You must be signed in to change notification settings - Fork 259
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug] Fatal signal 7 (SIGBUS), code 1 (BUS_ADRALN), fault addr 0x74ff2c183d in tid 3613 (mqt_js), pid 3455 #88
Comments
Happening to me as well, I'm using reanimated too |
Did you guys follow the installation guide with the steps mentioned in "With react-native-reanimated"? |
Also please paste the full logs from Logcat. |
Yeah @mrousavy, I can confirm I follow it, here's the log:
|
Also, I added what you suggested here #37 which is key and missing in the docs |
@sauldom102 Did you successful now? I had try but not success =( |
Can you guys confirm that react-native-mmkv v1.2.1 fixes the issue for you? |
@mrousavy Unfortunately it's not fixed in v1.2.1 |
I'm using reanimated 1.x, so I'm following the without reanimated instructions (https://github.com/mrousavy/react-native-mmkv/blob/master/INSTALL.md), also followed #37 (otherwise it didn't compile), but I'm still getting the error. react-native-mmkv: 1.2.1
|
@mrousavy is there any way to test this ? did you publish a beta or similar ? |
Not yet, I'm very busy with work + RNEU right now. You can test this change yourself by cloning the repo and running the Example app, then revert those commits and see if it works |
@mrousavy I ran the Example app from the repo (after some hacking 😄) as you suggested and it works on android !! So, I created a clean RN project, importing only react-native-mmkv and react-native-async-storage, to replicate the Example app and it crashes !! This leads me to believe that there's something odd with the build process ?? I'm using 0.64.1 in this repro project, without reanimated, but I have another project running 0.64.1 with reanimated 2, react-native-mmkv 1.1.6 and it works 🤷♂️ This is the repro repo https://github.com/jbrodriguez/rnmmkv Hopefully this helps figure out this issue ? This lib is running in prod in my other project and it works awesomely, I'd love to use it in my new project as well. If you have any other suggestions, let me know. |
UPDATE: I downgraded to 1.1.6 in the repro project and it works. |
Tried cleaning and rebuilding everything, does v1.2.2 work for you guys? |
@mrousavy there's no android folder with this build 08:47 rnmmkv on master [!] via v16.5.0 took 2s
❯ l node_modules/react-native-mmkv/
drwxr-xr-x - jbrodriguez staff 6 Aug 08:47 -I ios
drwxr-xr-x - jbrodriguez staff 6 Aug 08:38 -I lib
drwxr-xr-x - jbrodriguez staff 6 Aug 08:47 -I node_modules
.rw-r--r-- 22k jbrodriguez staff 6 Aug 08:38 -I LICENSE
.rw-r--r-- 3.8k jbrodriguez staff 6 Aug 08:38 -I package.json
.rw-r--r-- 1.0k jbrodriguez staff 6 Aug 08:38 -I react-native-mmkv.podspec
.rw-r--r-- 5.1k jbrodriguez staff 6 Aug 08:38 -I README.md |
Okay so sorry guys I had to publish a few version before I finally got it right. Take a look at v1.2.5 and let me know if that builds for you now. |
@mrousavy thanks for working on this. v1.2.5 does build for me, but it crashes at startup on Android as v1.2.1 did. The crash log seems similar to before (attached) Perhaps it does work for somebody else, and my crash is something about my setup
|
I just released 1.3.0 (yay 🎉), can you test if that works for you guys? It does work fine in the Example App (which is on RN 0.65.1) |
While I already spent way too much of my free time on this issue to fix the crash for random people on the internet, I spent some more time on this yesterday and finally fixed it. Check out 1.3.1. 🎉🎉🎉🎉 If you appreciate my efforts, please consider buying me a coffee for my next late-night debug session: ❤️ GitHub sponsors |
just wanted to add, that android is no longer crashing on start, I have another error (attached image) but it's due to the api change Thanks a lot for fixing this ! Oh, btw, if compiles a bunch of stuff now Build reactnativemmkv_armeabi-v7a
ninja: Entering directory `/Users/unknown/tmp/rnmmkv/node_modules/react-native-mmkv/android/.cxx/cmake/debug/armeabi-v7a'
[1/38] Building CXX object core/CMakeFiles/core.dir/MMKV_OSX.cpp.o
[2/38] Building CXX object core/CMakeFiles/core.dir/CodedInputDataCrypt_OSX.cpp.o
[3/38] Building CXX object core/CMakeFiles/core.dir/MMKVLog.cpp.o
[4/38] Building CXX object core/CMakeFiles/core.dir/PBUtility.cpp.o
[5/38] Building CXX object core/CMakeFiles/core.dir/CodedInputData_OSX.cpp.o
[6/38] Building CXX object core/CMakeFiles/core.dir/MiniPBCoder_OSX.cpp.o
[7/38] Building CXX object core/CMakeFiles/core.dir/MMBuffer.cpp.o
[8/38] Building CXX object core/CMakeFiles/core.dir/KeyValueHolder.cpp.o
[9/38] Building CXX object core/CMakeFiles/core.dir/MMKVLog_Android.cpp.o
[10/38] Building CXX object core/CMakeFiles/core.dir/CodedOutputData.cpp.o
[11/38] Building CXX object core/CMakeFiles/core.dir/CodedInputData.cpp.o
[12/38] Building CXX object core/CMakeFiles/core.dir/CodedInputDataCrypt.cpp.o
[13/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock_Win32.cpp.o
[14/38] Building CXX object CMakeFiles/reactnativemmkv.dir/src/main/cpp/cpp-adapter.cpp.o
[15/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock.cpp.o
[16/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_Win32.cpp.o
[17/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock_Android.cpp.o
[18/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_OSX.cpp.o
[19/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_md5_one.cpp.o
[20/38] Building CXX object core/CMakeFiles/core.dir/ThreadLock.cpp.o
[21/38] Building ASM object core/CMakeFiles/core.dir/aes/openssl/openssl_aesv8-armx.S.o
[22/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_md5_dgst.cpp.o
[23/38] Building ASM object core/CMakeFiles/core.dir/aes/openssl/openssl_aes-armv4.S.o
[24/38] Building CXX object core/CMakeFiles/core.dir/ThreadLock_Win32.cpp.o
[25/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile.cpp.o
[26/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_cfb128.cpp.o
[27/38] Building CXX object core/CMakeFiles/core.dir/aes/AESCrypt.cpp.o
[28/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_aes_core.cpp.o
[29/38] Building CXX object core/CMakeFiles/core.dir/MMKV_Android.cpp.o
[30/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_Android.cpp.o
[31/38] Building CXX object core/CMakeFiles/core.dir/MiniPBCoder.cpp.o
[32/38] Building CXX object core/CMakeFiles/core.dir/MMKV.cpp.o
[33/38] Building CXX object core/CMakeFiles/core.dir/crc32/crc32_armv8.cpp.o
[34/38] Building CXX object core/CMakeFiles/core.dir/crc32/zlib/crc32.cpp.o
[35/38] Building CXX object core/CMakeFiles/core.dir/MMKV_IO.cpp.o
[36/38] Building CXX object CMakeFiles/reactnativemmkv.dir/src/main/cpp/MmkvHostObject.cpp.o
[37/38] Linking CXX static library core/libcore.a
[38/38] Linking CXX shared library ../../../../build/intermediates/cmake/debug/obj/armeabi-v7a/libreactnativemmkv.so
Build reactnativemmkv_arm64-v8a
ninja: Entering directory `/Users/unknown/tmp/rnmmkv/node_modules/react-native-mmkv/android/.cxx/cmake/debug/arm64-v8a'
[1/38] Building CXX object core/CMakeFiles/core.dir/MMKV_OSX.cpp.o
[2/38] Building CXX object core/CMakeFiles/core.dir/MMKVLog.cpp.o
[3/38] Building CXX object core/CMakeFiles/core.dir/PBUtility.cpp.o
[4/38] Building CXX object core/CMakeFiles/core.dir/MiniPBCoder_OSX.cpp.o
[5/38] Building CXX object core/CMakeFiles/core.dir/CodedInputDataCrypt_OSX.cpp.o
[6/38] Building CXX object core/CMakeFiles/core.dir/CodedInputData_OSX.cpp.o
[7/38] Building CXX object core/CMakeFiles/core.dir/MMBuffer.cpp.o
[8/38] Building CXX object core/CMakeFiles/core.dir/KeyValueHolder.cpp.o
[9/38] Building CXX object core/CMakeFiles/core.dir/MMKVLog_Android.cpp.o
[10/38] Building CXX object core/CMakeFiles/core.dir/CodedOutputData.cpp.o
[11/38] Building CXX object core/CMakeFiles/core.dir/CodedInputData.cpp.o
[12/38] Building CXX object core/CMakeFiles/core.dir/CodedInputDataCrypt.cpp.o
[13/38] Building CXX object CMakeFiles/reactnativemmkv.dir/src/main/cpp/cpp-adapter.cpp.o
[14/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock_Win32.cpp.o
[15/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_Win32.cpp.o
[16/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock_Android.cpp.o
[17/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock.cpp.o
[18/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_OSX.cpp.o
[19/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_md5_one.cpp.o
[20/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_md5_dgst.cpp.o
[21/38] Building CXX object core/CMakeFiles/core.dir/ThreadLock.cpp.o
[22/38] Building ASM object core/CMakeFiles/core.dir/aes/openssl/openssl_aesv8-armx.S.o
[23/38] Building ASM object core/CMakeFiles/core.dir/aes/openssl/openssl_aes-armv4.S.o
[24/38] Building CXX object core/CMakeFiles/core.dir/ThreadLock_Win32.cpp.o
[25/38] Building CXX object core/CMakeFiles/core.dir/MMKV_Android.cpp.o
[26/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile.cpp.o
[27/38] Building CXX object core/CMakeFiles/core.dir/aes/AESCrypt.cpp.o
[28/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_aes_core.cpp.o
[29/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_Android.cpp.o
[30/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_cfb128.cpp.o
[31/38] Building CXX object core/CMakeFiles/core.dir/MiniPBCoder.cpp.o
[32/38] Building CXX object core/CMakeFiles/core.dir/MMKV.cpp.o
[33/38] Building CXX object core/CMakeFiles/core.dir/crc32/zlib/crc32.cpp.o
[34/38] Building CXX object core/CMakeFiles/core.dir/crc32/crc32_armv8.cpp.o
[35/38] Building CXX object core/CMakeFiles/core.dir/MMKV_IO.cpp.o
[36/38] Building CXX object CMakeFiles/reactnativemmkv.dir/src/main/cpp/MmkvHostObject.cpp.o
[37/38] Linking CXX static library core/libcore.a
[38/38] Linking CXX shared library ../../../../build/intermediates/cmake/debug/obj/arm64-v8a/libreactnativemmkv.so
Build reactnativemmkv_x86
ninja: Entering directory `/Users/unknown/tmp/rnmmkv/node_modules/react-native-mmkv/android/.cxx/cmake/debug/x86'
[1/38] Building CXX object core/CMakeFiles/core.dir/MMKV_OSX.cpp.o
[2/38] Building CXX object core/CMakeFiles/core.dir/MMKVLog.cpp.o
[3/38] Building CXX object core/CMakeFiles/core.dir/PBUtility.cpp.o
[4/38] Building CXX object core/CMakeFiles/core.dir/CodedInputDataCrypt_OSX.cpp.o
[5/38] Building CXX object core/CMakeFiles/core.dir/CodedInputData_OSX.cpp.o
[6/38] Building CXX object core/CMakeFiles/core.dir/MiniPBCoder_OSX.cpp.o
[7/38] Building CXX object core/CMakeFiles/core.dir/MMBuffer.cpp.o
[8/38] Building CXX object core/CMakeFiles/core.dir/KeyValueHolder.cpp.o
[9/38] Building CXX object core/CMakeFiles/core.dir/MMKVLog_Android.cpp.o
[10/38] Building CXX object core/CMakeFiles/core.dir/CodedOutputData.cpp.o
[11/38] Building CXX object core/CMakeFiles/core.dir/CodedInputDataCrypt.cpp.o
[12/38] Building CXX object core/CMakeFiles/core.dir/CodedInputData.cpp.o
[13/38] Building CXX object CMakeFiles/reactnativemmkv.dir/src/main/cpp/cpp-adapter.cpp.o
[14/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock_Win32.cpp.o
[15/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock.cpp.o
[16/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_Win32.cpp.o
[17/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock_Android.cpp.o
[18/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_OSX.cpp.o
[19/38] Building CXX object core/CMakeFiles/core.dir/ThreadLock.cpp.o
[20/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_md5_one.cpp.o
[21/38] Building ASM object core/CMakeFiles/core.dir/aes/openssl/openssl_aesv8-armx.S.o
[22/38] Building ASM object core/CMakeFiles/core.dir/aes/openssl/openssl_aes-armv4.S.o
[23/38] Building CXX object core/CMakeFiles/core.dir/MMKV_Android.cpp.o
[24/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_md5_dgst.cpp.o
[25/38] Building CXX object core/CMakeFiles/core.dir/ThreadLock_Win32.cpp.o
[26/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile.cpp.o
[27/38] Building CXX object core/CMakeFiles/core.dir/aes/AESCrypt.cpp.o
[28/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_cfb128.cpp.o
[29/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_Android.cpp.o
[30/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_aes_core.cpp.o
[31/38] Building CXX object core/CMakeFiles/core.dir/MiniPBCoder.cpp.o
[32/38] Building CXX object core/CMakeFiles/core.dir/MMKV.cpp.o
[33/38] Building CXX object core/CMakeFiles/core.dir/crc32/zlib/crc32.cpp.o
[34/38] Building CXX object core/CMakeFiles/core.dir/crc32/crc32_armv8.cpp.o
[35/38] Building CXX object core/CMakeFiles/core.dir/MMKV_IO.cpp.o
[36/38] Building CXX object CMakeFiles/reactnativemmkv.dir/src/main/cpp/MmkvHostObject.cpp.o
[37/38] Linking CXX static library core/libcore.a
[38/38] Linking CXX shared library ../../../../build/intermediates/cmake/debug/obj/x86/libreactnativemmkv.so
Build reactnativemmkv_x86_64
ninja: Entering directory `/Users/unknown/tmp/rnmmkv/node_modules/react-native-mmkv/android/.cxx/cmake/debug/x86_64'
[1/38] Building CXX object core/CMakeFiles/core.dir/MMKV_OSX.cpp.o
[2/38] Building CXX object core/CMakeFiles/core.dir/PBUtility.cpp.o
[3/38] Building CXX object core/CMakeFiles/core.dir/MMKVLog.cpp.o
[4/38] Building CXX object core/CMakeFiles/core.dir/CodedInputData_OSX.cpp.o
[5/38] Building CXX object core/CMakeFiles/core.dir/CodedInputDataCrypt_OSX.cpp.o
[6/38] Building CXX object core/CMakeFiles/core.dir/MiniPBCoder_OSX.cpp.o
[7/38] Building CXX object core/CMakeFiles/core.dir/KeyValueHolder.cpp.o
[8/38] Building CXX object core/CMakeFiles/core.dir/MMBuffer.cpp.o
[9/38] Building CXX object core/CMakeFiles/core.dir/MMKVLog_Android.cpp.o
[10/38] Building CXX object core/CMakeFiles/core.dir/CodedOutputData.cpp.o
[11/38] Building CXX object core/CMakeFiles/core.dir/CodedInputData.cpp.o
[12/38] Building CXX object core/CMakeFiles/core.dir/CodedInputDataCrypt.cpp.o
[13/38] Building CXX object CMakeFiles/reactnativemmkv.dir/src/main/cpp/cpp-adapter.cpp.o
[14/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock_Win32.cpp.o
[15/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_Win32.cpp.o
[16/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock_Android.cpp.o
[17/38] Building CXX object core/CMakeFiles/core.dir/MMKV_Android.cpp.o
[18/38] Building CXX object core/CMakeFiles/core.dir/InterProcessLock.cpp.o
[19/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_md5_dgst.cpp.o
[20/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_OSX.cpp.o
[21/38] Building CXX object core/CMakeFiles/core.dir/ThreadLock_Win32.cpp.o
[22/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_md5_one.cpp.o
[23/38] Building CXX object core/CMakeFiles/core.dir/ThreadLock.cpp.o
[24/38] Building ASM object core/CMakeFiles/core.dir/aes/openssl/openssl_aesv8-armx.S.o
[25/38] Building ASM object core/CMakeFiles/core.dir/aes/openssl/openssl_aes-armv4.S.o
[26/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile.cpp.o
[27/38] Building CXX object core/CMakeFiles/core.dir/aes/AESCrypt.cpp.o
[28/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_cfb128.cpp.o
[29/38] Building CXX object core/CMakeFiles/core.dir/MemoryFile_Android.cpp.o
[30/38] Building CXX object core/CMakeFiles/core.dir/aes/openssl/openssl_aes_core.cpp.o
[31/38] Building CXX object core/CMakeFiles/core.dir/MiniPBCoder.cpp.o
[32/38] Building CXX object core/CMakeFiles/core.dir/MMKV.cpp.o
[33/38] Building CXX object core/CMakeFiles/core.dir/crc32/crc32_armv8.cpp.o
[34/38] Building CXX object core/CMakeFiles/core.dir/crc32/zlib/crc32.cpp.o
[35/38] Building CXX object core/CMakeFiles/core.dir/MMKV_IO.cpp.o
[36/38] Building CXX object CMakeFiles/reactnativemmkv.dir/src/main/cpp/MmkvHostObject.cpp.o
[37/38] Linking CXX static library core/libcore.a
[38/38] Linking CXX shared library ../../../../build/intermediates/cmake/debug/obj/x86_64/libreactnativemmkv.so |
Describe the bug
Crash on Android devices
To Reproduce
Steps to reproduce the behavior:
iOS build and archive fine, but android crash
Platform Information:
"react": "17.0.2",
"react-native": "0.64.2",
"react-native-reanimated": "^2.2.0",
"@react-navigation/native": "^5.9.4",
"react-native-mmkv": "^1.1.6",
I get same issue in here: ammarahm-ed/react-native-mmkv-storage#129
The text was updated successfully, but these errors were encountered: