Skip to content
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

sentry crash by [sentry_get_modules_list] #1549

Closed
10 tasks done
donlan opened this issue Jun 22, 2021 · 12 comments
Closed
10 tasks done

sentry crash by [sentry_get_modules_list] #1549

donlan opened this issue Jun 22, 2021 · 12 comments
Labels

Comments

@donlan
Copy link

donlan commented Jun 22, 2021

Platform:

  • Android ->
    min_sdk = 21
    target_sdk = 30
    build_tools = "30.0.2"

  • Java -> JavaVersion.VERSION_1_8

  • Kotlin -> 1.5.0

  • NDK -> 21.3.6528143

IDE:

  • Android Studio -> 4.2.1

Build system:

  • Gradle -> 6.5.1

Android Gradle Plugin:

  • Yes -> com.android.tools.build:gradle:4.0.1

Sentry Android Gradle Plugin:

  • Yes -> 2.0.0

Proguard/R8:

  • Enabled

Platform installed with:

  • Maven Central

The version of the SDK:
5.0.1


I have the following issue:

sentry native crash from sentry dashboard issue, original crash info as follow:

OS Version: Android 10 (NOP-AN01P 2.0.0.116(C786E115R6P2))
Report Version: 104

Exception Type: Unknown (SIGSEGV)

Application Specific Information:
Segfault

Thread 0 Crashed:
0   libsentry.so                    0xc04c6adc          sentry_get_modules_list
1   libsentry.so                    0xc04c24d7          <unknown> + 3226215639
2   libsentry.so                    0xc04bf8dd          <unknown> + 3226204381
3   libsentry.so                    0xc04c70b7          <unknown> + 3226235063
4   libsentry.so                    0xc04c6ecd          <unknown> + 3226234573
5   libc.so                         0xf268ea90          <unknown> + 4066962064
6   libc.so                         0xf26944fe          abort
7   liblog.so                       0xef6edb2f          __android_log_assert
8   libandroid_runtime.so           0xf065fe33          android::bitmap::toBitmap
9   libandroid_runtime.so           0xf062037b          <unknown> + 4032955259
10  boot-framework.oat              0x7098e181          <unknown> + 1889067393



EOF

sentry gradle plugin config :

sentry {

    autoUpload = true

    uploadNativeSymbols = true

    includeNativeSources = false
}

more crash info:
image
image

Steps to reproduce:

  • None

Actual result:

  • Actual

Expected result:

  • Result
@marandaneto
Copy link
Contributor

@donlan thanks for reporting.

cc @Swatinem , wondering if its related to #1477

@donlan sentry android gradle plugin 2.0.1 does not exist, can you confirm the versioning?
also, are you using sentry.io or self hosted sentry?

@donlan
Copy link
Author

donlan commented Jun 22, 2021

@donlan thanks for reporting.

cc @Swatinem , wondering if its related to #1477

@donlan sentry android gradle plugin 2.0.1 does not exist, can you confirm the versioning?
also, are you using sentry.io or self hosted sentry?

Sorry, i wrote a wrong version number. The correct sentry android gradle plugin is 2.0.0. And we using sentry.io .

@Swatinem
Copy link
Member

I do have a hunch, but I’m unsure if these things are related.

The interesting thing here is also:

6   libc.so                         0xf26944fe          abort
7   liblog.so                       0xef6edb2f          __android_log_assert

The code is triggering an abort following a failed assertion (or so it seems), and sentry seems to crash while handling that. So it would seem that at least we did not cause the crash.

@donlan
Copy link
Author

donlan commented Jun 22, 2021

I have no more info about this crash. But we don't have the crash before sentry SDK version of 5.0.1. And we only catch one crash as so far.

@donlan
Copy link
Author

donlan commented Jun 23, 2021

OS Version: Android 10 (POT-AL00a 10.0.0.168(C00E70R5P6))
Report Version: 104

Exception Type: Unknown (SIGSEGV)

Application Specific Information:
Segfault

Thread 0 Crashed:
0   libsentry.so                    0xc3380adc          sentry_get_modules_list
1   libsentry.so                    0xc337c4d7          <unknown> + 3275211991
2   libsentry.so                    0xc33798dd          <unknown> + 3275200733
3   libsentry.so                    0xc33810b7          <unknown> + 3275231415
4   libsentry.so                    0xc3380ecd          <unknown> + 3275230925
5   libc.so                         0xf3e0ea8e          <unknown> + 4091603598

We catch a new crash issue reported. @Swatinem

@marandaneto
Copy link
Contributor

@donlan you cannot reproduce this issue either, right?

@Swatinem
Copy link
Member

Swatinem commented Jul 1, 2021

I landed some improvements related to the sentry_get_modules_list here: getsentry/sentry-native#559

I hope to get these fixes into the Android SDK soon.

@donlan
Copy link
Author

donlan commented Jul 1, 2021

@donlan you cannot reproduce this issue either, right?

yeah, i can't reproduce. But we received 13 crashed issues with our recent release version.

image

@donlan
Copy link
Author

donlan commented Jul 1, 2021

I landed some improvements related to the sentry_get_modules_list here: getsentry/sentry-native#559

I hope to get these fixes into the Android SDK soon.

Thanks, we would try the next release version soon.

@marandaneto
Copy link
Contributor

@donlan any news here? thanks

@Abbondanzo
Copy link

It looks like this was fixed in #1651

@marandaneto
Copy link
Contributor

Closing the issue as a part of large repository cleanup, due to it being inactive and/or outdated. There's a large chance that this issue has been solved since.
Please do not hesitate to ping me if it is still relevant, and I will happily reopen and work on it.
Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants