Don't use createCleaner for native Lock.

Author: MohammedKHC

okio/src/commonMain/kotlin/okio/CommonPlatform.kt

@@ -31,6 +31,7 @@ expect class Lock
 expect inline fun <T> Lock.withLock(action: () -> T): T
 
 internal expect fun newLock(): Lock
+internal expect inline fun Lock.destroy()
 
 expect open class IOException(message: String?, cause: Throwable?) : Exception {
   constructor(message: String?)

okio/src/commonMain/kotlin/okio/FileHandle.kt

@@ -289,6 +289,7 @@ abstract class FileHandle(
       closed = true
       if (openStreamCount != 0) return
     }
+    lock.destroy()
     protectedClose()
   }
 

okio/src/jsMain/kotlin/okio/JsPlatform.kt

@@ -57,6 +57,7 @@ internal val tmpdir: String
 actual typealias Lock = Unit
 
 internal actual fun newLock(): Lock = Unit
+internal actual inline fun Lock.destroy() = Unit
 
 actual inline fun <T> Lock.withLock(action: () -> T): T {
   contract {

okio/src/jvmMain/kotlin/okio/-JvmPlatform.kt

@@ -31,6 +31,7 @@ actual typealias ArrayIndexOutOfBoundsException = java.lang.ArrayIndexOutOfBound
 actual typealias Lock = ReentrantLock
 
 internal actual fun newLock(): Lock = ReentrantLock()
+internal actual inline fun Lock.destroy() = Unit
 
 actual inline fun <T> Lock.withLock(action: () -> T): T {
   contract {

okio/src/mingwX64Main/kotlin/okio/WindowsPlatform.kt

@@ -17,29 +17,26 @@ package okio
 
 import kotlin.contracts.InvocationKind
 import kotlin.contracts.contract
-import kotlin.experimental.ExperimentalNativeApi
-import kotlin.native.ref.createCleaner
 import platform.windows.CloseHandle
 import platform.windows.CreateMutexA
 import platform.windows.INFINITE
 import platform.windows.ReleaseMutex
 import platform.windows.WaitForSingleObject
 
-actual class Lock {
+actual class Lock : Closeable {
   val mutex = CreateMutexA(
     null,
     0,
     null
   ) ?: throw lastErrorToIOException()
 
-  @Suppress("unused")
-  @OptIn(ExperimentalNativeApi::class)
-  private val cleaner = createCleaner(mutex) {
+  override fun close() {
     CloseHandle(mutex)
   }
 }
 
 internal actual fun newLock(): Lock = Lock()
+internal actual inline fun Lock.destroy() = close()
 
 actual inline fun <T> Lock.withLock(action: () -> T): T {
   contract {

okio/src/unixMain/kotlin/okio/UnixPlatform.kt

@@ -17,8 +17,6 @@ package okio
 
 import kotlin.contracts.InvocationKind
 import kotlin.contracts.contract
-import kotlin.experimental.ExperimentalNativeApi
-import kotlin.native.ref.createCleaner
 import kotlinx.cinterop.alloc
 import kotlinx.cinterop.free
 import kotlinx.cinterop.nativeHeap
@@ -30,22 +28,21 @@ import platform.posix.pthread_mutex_lock
 import platform.posix.pthread_mutex_t
 import platform.posix.pthread_mutex_unlock
 
-actual class Lock {
+actual class Lock : Closeable {
   val mutex = nativeHeap.alloc<pthread_mutex_t>().apply {
     if (pthread_mutex_init(ptr, null) != 0) {
       throw errnoToIOException(errno)
     }
   }
 
-  @Suppress("unused")
-  @OptIn(ExperimentalNativeApi::class)
-  private val cleaner = createCleaner(mutex) {
-    pthread_mutex_destroy(it.ptr)
-    nativeHeap.free(it)
+  override fun close() {
+    pthread_mutex_destroy(mutex.ptr)
+    nativeHeap.free(mutex)
   }
 }
 
 internal actual fun newLock(): Lock = Lock()
+internal actual inline fun Lock.destroy() = close()
 
 actual inline fun <T> Lock.withLock(action: () -> T): T {
   contract {

okio/src/wasmMain/kotlin/okio/WasmPlatform.kt

@@ -25,6 +25,7 @@ internal actual val PLATFORM_DIRECTORY_SEPARATOR: String
 actual typealias Lock = Unit
 
 internal actual fun newLock(): Lock = Unit
+internal actual inline fun Lock.destroy() = Unit
 
 @OptIn(ExperimentalContracts::class)
 actual inline fun <T> Lock.withLock(action: () -> T): T {