Sentry Native Android SDK Crashing - sentry__string_eq Unhandled Segfault #859

Activity

1. getsantry

   added

   Status: Untriaged

   Waiting for: Product Owner

   on Jun 28, 2023
2. getsantry

   added this to GitHub Issues with 👀on Jun 28, 2023
3. getsantry

   moved this to Waiting for: Product Owner in GitHub Issues with 👀on Jun 28, 2023
4. will commented on Jun 28, 2023

   

   will

   on Jun 28, 2023 via email

   More actions

   I would but I don’t think I’m in a slack with you On Jun 28, 2023, at 15:07, William Capozzoli ***@***.***> wrote: Description Message me @will in Slack for link to the Customer's Issue in Sentry. When does the problem happen During build [ x] During run-time When capturing a hard crash Environment Steps To Reproduce The issue in sentry has this on it: err.type looks like sentry__string_eq it also says Unhandled Segfault and Log output no additional log output —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
5. supervacuus commented on Jun 29, 2023

   

   supervacuus

   on Jun 29, 2023

   Collaborator

   More actions

   Hi, thanks for the report. I can gather from the stack trace that this must have been in a sentry-android version that came bundled with sentry-native < 0.6.2. The segfault comes from passing NULL to strcmp() inside sentry__string_eq().

   Adding ptr/len interfaces to the API, I rewrote many string functions and specifically added early checks for NULLs. Especially the code in sentry_value_set_by_key() has been made safer. This was released with sentry-android 6.19.0.
6. supervacuus

   added

   Bugsomething isn't working as it shouldsomething isn't working as it should

   outdated

   and removed

   Status: Untriaged

   Waiting for: Product Owner

   on Jun 29, 2023
7. thinkocapo commented on Jun 29, 2023

   

   thinkocapo

   on Jun 29, 2023

   Author

   More actions

   is the suggestion to upgrade to 6.19.0? If so, what is the the new expected behavior? So I can communicate back to the customer.

   For example:

   1. upgrade to 6.19.0
   2. the sdk should not error on itself anymore, therefore no event for sending to Sentry.io.
   3. or
   4. upgrade to 6.19.0
   5. the sdk might still error on itself, but the event will not get sent to Sentry.io? the sdk will fail more silently this time?
8. supervacuus commented on Jun 29, 2023

   

   supervacuus

   on Jun 29, 2023

   Collaborator

   More actions

   I would always recommend upgrading if there seems to be a bug in an SDK because we do not maintain separate release branches, so even if there is still an issue in the latest release, I can only supply a fix on top of that. Going deeper, this stack trace is probably from version 0.4.18 of the Native SDK, released over a year ago (and packaged with sentry-android 6.3.1).

   The crash in the Native SDK is probably caused by the same memory corruption which caused the application to crash in the first place. We can make sure that we don't crash in that particular situation. However, the resulting crash event will still be affected by that. The stack trace shows that the segfault occurs when setting the event_id of the crash-event, meaning that either the constant string "event_id" is NULL or that the event internal structure has been corrupted, leaving us to try to iterate over non-existent key/value pairs of a fresh object (which has no entries yet).

   What I mean by that: fingers crossed that we can send a halfway decent crash report in that case. Understanding how that situation is possible would be interesting, but I cannot gather that from the stack trace. Maybe @Swatinem can look at the event and check if he sees something relevant.
9. supervacuus

   added

   Platform: Android

   on Apr 2, 2024
10. kahest

    added

    Platform: Native

    on Aug 29, 2024
11. kahest

    closed this as not plannedon Dec 10, 2024