Merge pull request #17472 from wordpress-mobile/issue/17463-prevent-invalid-mutex-unlock

Guard mutex unlock in isLocked check to avoid errors during local data uploads

Author: ovitrif

WordPress/src/main/java/org/wordpress/android/ui/uploads/UploadStarter.kt

@@ -178,7 +178,13 @@ class UploadStarter @Inject constructor(
                         )
                     }
         } finally {
-            mutex.unlock()
+            // If the job of the current coroutine is cancelled while the `lock()` call is suspended,
+            // it results in the mutex ending up unlocked.
+            // We introduced this check to prevent IllegalStateExceptions when `unlock` is called in those cases.
+            // See: https://github.com/wordpress-mobile/WordPress-Android/issues/17463
+            if (mutex.isLocked) {
+                mutex.unlock()
+            }
         }
     }