-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Sporadic crash on app exit - iOS 13.5.1 #51
Comments
We are experiencing this issue as well. |
I think the problem is inside Runtime destructor at this line. As long as I understand ->Enter() and ->Exit() V8 API calls must be paired. That's what stack allocated instances of Isolate::Scope, Context::Scope etc actually do: they perform an ->Enter() call in their constructor and an ->Exit() call in their destructor. I don't understand why an Isolate->Exit() is manually performed in the Runtime destructor. It's not paired with an Isolate->Enter() call and leads to crashes. You should take inspiration from { N } Android Runtime. Here is an example of a proper usage of the V8 API. |
Yes, I believe this is a leftover from previous refactorings and can be removed. @adrian-branescu, would you mind submitting a PR for this? |
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
@NathanaelA, sad to report, but Reproducible only on real device, fine on simulator. Demo: https://drive.google.com/file/d/1T8eU3LQb4q9_ZDkw0ZWB4jHccEACRZov/view?usp=sharing |
Confirm this still happens, raised an issue on the main Nativescript project, but looks like a direct match to this one. Added a bit more information in my original issue report here: @EddyVerbruggen has tested this with a few versions and added some worthy insights in my original issue. Here's the trace:
|
In a current app on TestFlight, some testers are encountering this occasionally when the app exits (full crashlog attached):
crashlog.txt
The text was updated successfully, but these errors were encountered: