Merge pull request #1796 from OneSignal/fix/event_producer_crash

[5.0.0] Make subscribers in EventProducer synchronized
OneSignal · Feb 6, 2024 · a741b1a · a741b1a
2 parents 0d10478 + 8b630e8
commit a741b1a
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/OneSignalSDK/onesignal/core/src/main/java/com/onesignal/common/events/EventProducer.kt b/OneSignalSDK/onesignal/core/src/main/java/com/onesignal/common/events/EventProducer.kt
@@ -3,6 +3,7 @@ package com.onesignal.common.events
 import com.onesignal.common.threading.suspendifyOnMain
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.withContext
+import java.util.*
 
 /**
  * A standard implementation that implements [IEventNotifier] and additional functionality to make
@@ -13,7 +14,7 @@ open class EventProducer<THandler> : IEventNotifier<THandler> {
     override val hasSubscribers: Boolean
         get() = _subscribers.any()
 
-    private val _subscribers: MutableList<THandler> = mutableListOf()
+    private val _subscribers: MutableList<THandler> = Collections.synchronizedList(mutableListOf())
 
     override fun subscribe(handler: THandler) {
         _subscribers.add(handler)