Workaround for KT-58685

Kotlin · Sep 6, 2023 · 751364f · 751364f
1 parent 862b554
commit 751364f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/kotlinx-coroutines-core/common/src/sync/Mutex.kt b/kotlinx-coroutines-core/common/src/sync/Mutex.kt
@@ -123,11 +123,14 @@ public suspend inline fun <T> Mutex.withLock(owner: Any? = null, action: () -> T
     }
 
     lock(owner)
-    try {
-        return action()
-    } finally {
+    val result = try {
+        action()
+    } catch (e: Throwable) {
         unlock(owner)
+        throw e
     }
+    unlock(owner)
+    return result
 }
 
 

diff --git a/kotlinx-coroutines-core/common/src/sync/Semaphore.kt b/kotlinx-coroutines-core/common/src/sync/Semaphore.kt
@@ -84,11 +84,14 @@ public suspend inline fun <T> Semaphore.withPermit(action: () -> T): T {
     }
 
     acquire()
-    try {
-        return action()
-    } finally {
+    val result = try {
+        action()
+    } catch (e: Throwable) {
         release()
+        throw e
     }
+    release()
+    return result
 }
 
 @Suppress("UNCHECKED_CAST")