ZeusLN / zeus

A mobile Bitcoin wallet fit for the gods. ⚡️ Est. 563345
https://ZeusLN.com
Other
981 stars 145 forks source link

Error while loading Zeus Android with MTE enabled (memory tagging) #1872

Closed bedair81 closed 1 month ago

bedair81 commented 9 months ago

Describe the bug

enabling MTE to detect unsafe memory code on grapheneOS causes zeus wallet to terminate while starting the node on android 14 zeus 0.8.0 beta 4

Reproduce

enable MTE, start zeus, startup terminated

ZEUS version

v0.8.0

Node interface

Embedded LND

Network

Clearnet

Device

pixel 8 pro

Device operating system

android 14 GOS

Log output

type: crash
osVersion: google/husky/husky:14/UD1A.231105.004/2023112600:user/release-keys
uid: 10150 (u:r:untrusted_app_32:s0:c150,c256,c512,c768)
cmdline: app.zeusln.zeus
processUptime: 14s

signal: 11 (SIGSEGV), code 9 (SEGV_MTESERR), faultAddr 400cce9285b8330
threadName: Thread-4
MTE: enabled

backtrace:
    /data/app/~~vqt6U_EiZVbjVoRUaeFX1Q==/app.zeusln.zeus-BmWtxRhhpao29yIiWNKqPQ==/lib/arm64/libgojni.so (pc 456f34)
bedair81 commented 9 months ago

after updating to 0.8.0 beta 5, the same crash with MTE enabled occurs

bedair81 commented 4 months ago

Issue has been reported to React Native at: https://github.com/facebook/react-native/issues/44276

bedair81 commented 4 months ago

Issue seems to stem from native "Go" code running in embedded LND, doesn't occur if booting app and loading a remote node for management. Unsure whether to report the issue to "Go" themselves or LND

bedair81 commented 1 month ago

This seems to be resolved on 0.9.0 beta 3, not sure what fixed it but this is great news!