iOS: UnityWebRequest crash in WWWConnection.mm : WaitOnCondition()

via testflight beta, we're seeing an infrequent crash in WWWConnection.mm,
on this line:

Code (csharp):

1.  
2. static void WaitOnCondition(UnityWWWConnectionDelegate* delegate)
3. {
4.     NSCondition *condition = delegate.condition;
5.     [condition lock];          <-------------------------- crash
6.     [condition wait];
7.     [condition unlock];
8. }

others are seeing this as well:
https://answers.unity3d.com/questions/1390673/ios-crashing-on-unitywebrequest-calls.html

 

woohoo!
many thanks Aurimas-Cernius! this is great.

i'll add this to the "answers" thread as well.

 

Howdy -

this declaration actually occurs in two places:
WWConnection.h and WWConnection.mm both have this line:

Code (csharp):

1. @property (readonly, retain, nonatomic) NSURLConnection*    connection;

i assume they should both be modified ?

 

got it. thanks.

 

We're having the same problem. But you need to remove nonatomic from NSCondition* condition, not NSURLConnection* connection. And that's only in WWWConnection.mm.

When can we expect a Unity patch for this? We're using Cloud Build, which makes it harder to remove the nonatomic.

 

Do you have a reliable way to reproduce the problem ?
It's quite rare for us.

 

We use 5.6.3f1. We might upgrade to 2017 if that one gets the patch first.

Unfortunately, no. It just randomly happens sometimes when I switch view. Our Unity project is more an app than a game. We have to do a bunch of request to retrieve all images and data. Maybe it's related, but every time we close a view, we do Resources.UnloadUnusedAssets.

 

Looks like 5.6.3p4 has just arrived and should fix this issue

 

@Aurimas-Cernius Thanks for the solution. Which version of Unity 2017 should we expect the solution in?

 

@GAMEDIA_Justin We're using the following iOS PostBuild script to replace the overriden files. Just place in the appropriate folder and add a ".override" to the .mm filename. Should work with Cloudbuild.

https://gist.github.com/sapanda/a09ff0fbc49e4bb58e44cbfa758f66c2

 

I'm using Unity 2017.2 and I have this issue, crashing for ~8% of the player base.
Looked on WWWConnection.mm and the property is like this, so should be working I guess:

Code (CSharp):

1. @property (strong, atomic)               NSCondition*          condition;

Anyone know what could be causing it?


edit: added crash log

 

@Aurimas-Cernius Sorry about that, updated the file with a .txt version that seems to be working

 

It apparently shouldn't matter, but what about the same declaration in WWConnection.h ?

 

There is none

 

@Aurimas-Cernius I'm using Unity 2017.2 and Xcode 9.2

 

It's coming from inside the house !

 

Is this fix necessary in Unity 2017.4.1f1?

 

can anyone comment what is the ticket number for this? what release has the fix for it? and whether the workaround (remove nonatomic) fixes the crash? @Aurimas-Cernius