search

MetaMask / metamask-mobile

Mobile web browser providing access to websites that use the Ethereum blockchain
https://metamask.io
Other
2.17k stars 1.12k forks source link

facebook::hermes::HermesRuntime::~HermesRuntime #12266

Closed sentry-io[bot] closed 3 days ago

sentry-io[bot] commented 1 week ago

Sentry Issue: METAMASK-MOBILE-2NJS

SIGSEGV: Segfault
  ?, in facebook::hermes::HermesRuntime::~HermesRuntime
  ?, in facebook::hermes::HermesRuntime::~HermesRuntime
  ?, in facebook::hermes::HermesRuntime::~HermesRuntime
  ?, in facebook::hermes::HermesRuntime::~HermesRuntime
  ?, in facebook::hermes::HermesRuntime::~HermesRuntime
...
(123 additional frame(s) were not displayed)
sethkfman commented 1 week ago

This is a crashing Android issue related to Hermes that started during 7.34.0 release. It seems to be related to confirmation screens 

Navigation
info
07:08:14.555 PM
Navigation to Amount

{
data: {"from":"Confirm","to":"Amount"}
Cal-L commented 1 week ago

Another similar error that includes a type error https://metamask.sentry.io/issues/5706015149/events/3fc622fe2d4047248c3124a310c5bcad/

TypeError: Cannot read property 'isUpdatedAfterSecurityCheck' of undefined Probably just a breadcrumb but still should be addressed

matthewwalsh0 commented 3 days ago

Fixed by #12311.