Race Condition in CentralManagerImpl

[LionZXY]

I found that the following code is used inside the CentralManagerImpl class

lateinit var internalScope: CoroutineScope

init {
 closeJob = scope.launch {
  internalScope = this
  //...
 }
}
//...
protected fun peripheral(id: ID, factory: (ID) -> P): P {
// ...
 state
  .launchIn(internalScope)
}

The problem with this code is that if internalScope has not been initialised, we will get an exception. For example this code:

val centralManager = CentralManagerImpl(scope)
centralManager.peripheral(ID) { } 

Can throw UninitializedPropertyAccessException if internalScope will not have time to reach execution

[LionZXY]

What about this code?

internalScope = CoroutineScope(scope.newCoroutineContext(EmptyCoroutineContext))

closeJob = internalScope.launch {
  try { awaitCancellation() }
  finally { withContext(NonCancellable) { close() } }
}

[philips77]

Thanks for spotting this. The CentralManagerImpl is internal class, so should not be used ourside, but I understand that one could use just final classes the same way.
Thanks for pointing into how we can have it fixed and I'm looking forward for more feedback from you!

Have a great day!