Segfault Crash

[AliSoftware]

Sentry Url: https://sentry.io/share/issue/ab1e2ab600d540aa964a361d54a1dd5d/ User Count: 582 Count: 1431 First Release: 17.7-rc-2 First Seen: 2021-07-04T09:26:28.771000Z Last Seen: 2021-08-24T09:24:50Z 24 Hours: 24 30 Days: 1234

null: Segfault

Saw this one on Sentry today, quite an obscure one with not much data nor stacktrace to help 😒 … but seems the events are still steadily happening and building up since 17.7, so figured it could be worth tracking and checking.

[ravishanker]

It is hard to reproduce issue. Need to monitor and investigate further.

Most likely suspect is a null pointer dereference. Possibly in parsing a Gson somewhere using Java classes

[fluiddot]

Not sure if the original issue was related to the Sentry library but I recently noticed that the stacktrace references the method sentry_get_modules_list which is defined by Sentry.

I looked for opened/closed issues on the Sentry repository (getsentry/sentry-java) that might be related and found this one with a similar problem. It states that the crash was addressed in version 5.1.0 by bumping sentry-native to 0.4.12. The current Sentry version defined in Automattic-Tracks-Android is 5.0.1, so I'm wondering if bumping Sentry to version 5.1.0 or newer might help.

[fluiddot]

I see a reference to Hermes (facebook::hermes::detail::hermesFatalErrorHandler) in a recent Sentry event, so potentially this crash could be related to React Native library. It might be a coincidence but, the first event was originated in version 17.7 which is also the one where the React Native upgrade to version 0.64 was released 🤔 .

[ParaskP7]

FYI: It seems that this event has been spiking for the past 3 months. Maybe it is worth to investigate it again as part of groundskeeping, wdyt? 🤔

[ovitrif]

@thehenrybyrd Following up on the internal discussion (↗️ ref: pbArwn-52u-p2#comment-6335) we can see in the graph of the past 3 months that the Sentry event has indeed started to show a recent downturn, therefore we might consider re-evaluating its priority in the near future 👍 .

[thehenrybyrd]

Thanks for following up on this and giving an update @ovitrif! As we're seeing a significant decrease, I'll lower the priority on this one to low.

[antonis]

The issue exists since version 17.7 and the crash rate remains low