Lothario
/
axmol
mirror of https://github.com/axmolengine/axmol.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix AudioEngine thread concurrency crash between myAlSourceNotificationCallback and _play2d iterating over _audioPlayers. (#18083)

This commit is contained in:
Felippe Durán 2017-07-18 00:12:20 -03:00 committed by minggo
parent a6773f408a
commit 78818ca275
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cocos/audio/apple/AudioEngine-inl.mm
View File

@ -215,6 +215,7 @@ ALvoid AudioEngineImpl::myAlSourceNotificationCallback(ALuint sid, ALuint notifi
return; return;
AudioPlayer* player = nullptr; AudioPlayer* player = nullptr;
s_instance->_threadMutex.lock();
for (const auto& e : s_instance->_audioPlayers) for (const auto& e : s_instance->_audioPlayers)
{ {
player = e.second; player = e.second;
@ -223,6 +224,7 @@ ALvoid AudioEngineImpl::myAlSourceNotificationCallback(ALuint sid, ALuint notifi
player->wakeupRotateThread(); player->wakeupRotateThread();
} }
} }
s_instance->_threadMutex.unlock();
} }
AudioEngineImpl::AudioEngineImpl() AudioEngineImpl::AudioEngineImpl()