[TIMOB-26852] iOS: Random JSCore crashes in SDK 8+
| GitHub Issue | n/a |
|---|---|
| Type | Bug |
| Priority | Critical |
| Status | Closed |
| Resolution | Fixed |
| Resolution Date | 2019-06-17T17:03:18.000+0000 |
| Affected Version/s | Release 8.0.0, Release 8.1.0 |
| Fix Version/s | Release 8.0.1, Release 8.0.2 |
| Components | iOS |
| Labels | engSchedule |
| Reporter | Hans Knöchel |
| Assignee | Jan Vennemann |
| Created | 2019-02-23T13:52:42.000+0000 |
| Updated | 2019-06-17T17:03:26.000+0000 |
Description
We noticed some random JSCore crashes in latest SDK 8 builds (not sure if also happening on < SDK 8 since we already updated to 8 a while ago). One of the crash logs attached.
Attachments
| File | Date | Size |
|---|---|---|
| crash-20190416.crash | 2019-04-16T14:22:14.000+0000 | 156354 |
| MyApp_2019-02-23-144905_Hanss-MacBook-Pro.crash | 2019-02-23T13:52:25.000+0000 | 175108 |
| MyApp_2019-02-27-104648_Hanss-MacBook-Pro.crash | 2019-02-27T09:47:56.000+0000 | 179358 |
| MyApp_2019-03-17-124204_Hanss-MacBook-Pro.crash | 2019-03-17T11:43:12.000+0000 | 180667 |
| wral_news_crash | 2019-04-23T15:52:01.000+0000 | 9626 |
| wral_wx_crash_1 | 2019-04-23T15:52:02.000+0000 | 15797 |
| wral_wx_crash_2 | 2019-04-23T15:52:01.000+0000 | 14823 |
[~amukherjee] [~lchoudhary] Can you check this one..
I just attached another crash log.
[~jvennemann] You now have access to all out crashlogs reported via Firebase.
[~rmitro], please share the following build with users that report this issue. It contains a possible fix for this issue. Due to lack of reproducible test cases we depend on customer feedback here. https://axwaysoftware-my.sharepoint.com/:u:/g/personal/jvennemann_axway_com/EV4qFmnFFrFFgVg8UUJNTrEB6vnWbspI49Ik4Sa7YfRaYw?e=wiQMio
PR (master): https://github.com/appcelerator/titanium_mobile/pull/10771 PR (8_0_X): https://github.com/appcelerator/titanium_mobile/pull/10772
The changes have been applied but it is still crashing in some occurrences. [~amukherjee] [~jvennemann] Please escalate this asap, this is really a SDK 8 update blocker.
Unfortunately even the latest fixes could not fix the crashes, although it seems to become more stable now. I hope more core-devs can be allocated to eliminate this.
Returning this ticket to "In Progress"
[~lchoudhary], [~vijaysingh] can you also please help with the debug?
This issue causes another possible crash in native modules (see [here](https://github.com/AndreaVitale/av.imageview/issues/72) for reference). To reproduce it in our app, we used an iPhone 6 (device, not Simulator. Simulator does not crash at all). You are free to create stuff in our app to trigger it. The reason why this is not reported too many tines so far is that probably not many users moved to 8.0.0 already. If I could guess, I would now pinpoint it to be related to firing events that cause a bad access / race condition inside JSCore. We now have 20+ unique crash reports for this. Please get more staff on this.
Pushed an updated fix to the PR. [~smohammed], can you test this again with the project provided by Hans? Thanks!
Added one more crash occurring after the latest changes from [~jvennemann].
Guys, you cannot be serious moving this ticket to 8.1.0. Every app using 8.0.0 is affected by this and there is no workaround available. Please finally more staff on this to resolve this. Edit: Thanks for the clarification and active efforts, I hope the team will find a solution soon.
[~hknoechel] I just sent you email. We can do a 8.0.2 immediately after we get the fix for this.
I believe I am also experiencing this issue with my app (support told me about this Jira issue). The app was using 7.x SDK (not sure which, perhaps 7.5) on iOS 10.1.x. Seemed to break when app iOS was updated to 10.2, but wasn't able to verify because my devices are all 10.2. Updated to use SDK 8, but didn't seem to change it. Crash on my app only happens on real hardware (simulator is fine, so same as this issue). It seems like once my app gets running, it's good to go. However, the crash is like 8 out of 10. Will try to upload my crashlog. Edit: I don't see how to upload my crashlog in chrome on my Mac. :-/
[~dlevine], if you can't upload directly to JIRA you can also share a link to the crash report via Dropbox, Google Drive or similar.
[~vijaysingh] this can now be reviewed again.
We are seeing a ton of crashes in JavaScriptCore in our client's apps as well because of the upgrade to 8.0. We are now considering downgrading to 7.5 unless a fix can be provided very soon. Here is a link to an example crash report...we get hundreds of these a day. https://drive.google.com/open?id=1pw6M3CzpJf53X2RSlmliUSlJdaAEAWSW
Please test the following pre-release build of 8.0.1 that includes a fix: https://www.dropbox.com/s/eidzetqd0z7cmv0/mobilesdk-8.0.1.v20190429083235-osx.zip?dl=0 Please note that due to a tight schedule this fix might be postponed to 8.0.2 in a final release. However, this will follow shortly after the 8.0.1.GA.
@Jan Vennemann - your link to the pre-release build requires a username/password.
[~btknorr], sorry, try again. I've updated the link!
I'd love to see if this resolves my issue, but I'm not quite sure how I would install this in my Studio build process. I'm on a Mac.
[~dlevine], just download the zip and run
ti sdk install <zip-file-path>. You should now be able to select this version in Studio.Ok got it. For completeness, I needed to run:
appc ti sdk install <zip-file-path>Nice job guys! This build resolves my crashing problem. Can't wait for 8.0.2. ;-)We even see some crashes in the current pull request state, but those may be different kind of leaks. [~jvennemann] is informed about those.
FR Passed, Waiting on Jenkins to merge.
[~smohammed] I heard of more people still having crashes in regards to memory usage over time, so I think this is not finished, yet.
[~hknoechel], we don't want do postpone 8.0.1 any longer so we will merge the current fixes. This is still being worked on and any additional fixes will be released in 8.0.2.
[~jvennemann] Feel free to reopen for any additional fixes for 8.0.2 or create a new ticket for the additional fixes.
Reopening to allow further PR's for this ticket.