-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
il2cpp::gc::GCHandle::GetTarget #1442
Comments
Thanks for raising. Took a look and doing a dump here to help folks (and also because shared event expired in 90 days): 11 different events, all Android 13 but 3 different device models (Samsung, OnePlus, etc) Looks like a Then goes into:
Somehow in there, two frames in, it crashes.
Finally seems to crash here:
Error captured from from Sentry Native @Swatinem should we be looking at C# instead of C++ given all symbols were uploaded? |
The problem is rather that we are crashing trying to resolve the C# provided stack trace. In particular, converting the C# object into its underlying C++ object/pointer so that we can call unity internal C++ methods on it. |
That might be possible. I haven’t looked at this in detail in quite a while. As a workaround, if this is causing problems for you, you can disable Sentrys own il2cpp line numbers integration, that should avoid running the code that is causing the crash here. |
With Unity 2023 the signature for
to
With I managed to get it working locally. |
@bitsandfoxes my baby boy, I think this issue might have come back with unity 6. Its crashing exactly the same way, exactly on the same line, and disabling the line thingy makes it not crash anymore |
Crashes caused by sentry are extremely frustrating because at a glance you first think that sentry its just trying to report a crash caused by another circumstance, not crashing the app itself having sentry actually crashing the app is such a curveball, I really would suggest this feature to be disabled by default and marked as experimental @bitsandfoxes |
Damn.. thanks @dganzella for pointing that out. We're going to have to update our CI to include Unity 6 at the very least! |
Working on it! This has not been forgotten. |
Using Sentry 1.5.2 on Android IL2CPP build Unity 2023.1.12f1 I'm getting a few of these reports.
https://odd-games.sentry.io/share/issue/0696247855b64ca9b9b89104b37e9dd5/
The text was updated successfully, but these errors were encountered: