Skip to content

Commit

Permalink
AudioUnitManager: Lock mutex before sending wakeAll
Browse files Browse the repository at this point in the history
  • Loading branch information
fwcd committed Nov 17, 2024
1 parent 210f502 commit c24f979
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/effects/backends/audiounit/audiounitmanager.mm
Original file line number Diff line number Diff line change
Expand Up @@ -49,14 +49,14 @@
}

bool AudioUnitManager::waitForAudioUnit(int timeoutMs) {
auto locker = lockMutex(&m_instantiatedWcMutex);
auto locker = lockMutex(&m_isInstantiatedMutex);

Check failure on line 52 in src/effects/backends/audiounit/audiounitmanager.mm

View workflow job for this annotation

GitHub Actions / macOS 12 x64

use of undeclared identifier 'm_isInstantiatedMutex'; did you mean 'm_instantiatedWcMutex'?

Check failure on line 52 in src/effects/backends/audiounit/audiounitmanager.mm

View workflow job for this annotation

GitHub Actions / macOS 12 arm64

use of undeclared identifier 'm_isInstantiatedMutex'; did you mean 'm_instantiatedWcMutex'?

AudioUnit _Nullable audioUnit = getAudioUnit();
if (audioUnit != nil) {
return true;
}

return m_instantiatedWc.wait(&m_instantiatedWcMutex, timeoutMs);
return m_instantiatedWc.wait(&m_isInstantiatedMutex, timeoutMs);

Check failure on line 59 in src/effects/backends/audiounit/audiounitmanager.mm

View workflow job for this annotation

GitHub Actions / macOS 12 x64

use of undeclared identifier 'm_isInstantiatedMutex'; did you mean 'm_instantiatedWcMutex'?

Check failure on line 59 in src/effects/backends/audiounit/audiounitmanager.mm

View workflow job for this annotation

GitHub Actions / macOS 12 arm64

use of undeclared identifier 'm_isInstantiatedMutex'; did you mean 'm_instantiatedWcMutex'?
}

void AudioUnitManager::instantiateAudioUnitAsync(
Expand Down Expand Up @@ -126,5 +126,7 @@

m_audioUnit = audioUnit;
m_isInstantiated.store(true);

auto locker = lockMutex(&m_instantiatedWcMutex);
m_instantiatedWc.wakeAll();
}

0 comments on commit c24f979

Please sign in to comment.