/[blink]
Chromium logo

Revision 157256


Jump to revision: Previous Next
Author: haraken@chromium.org
Date: Thu Sep 5 01:24:48 2013 UTC (9 years, 8 months ago)
Changed paths: 2
Log Message:
Fix threading races on AudioDSPKernelProcessor::m_kernels

AudioDSPKernelProcessor::m_kernels is accessed by the main thread and
the Audio Device thread, and thus should be protected by mutex.

This mutex implementation is consistent with that of ConvolverNode::m_reverb.

See the crash report for more details: https://cluster-fuzz.appspot.com/testcase?key=6662160935550976

BUG=271161
No tests because the crash depends on threading races and thus not reproducible.

Review URL: https://chromiumcodereview.appspot.com/23931002

Changed paths

Path Details
Directorytrunk/Source/core/platform/audio/AudioDSPKernelProcessor.cpp modified , text changed , props changed
Directorytrunk/Source/core/platform/audio/AudioDSPKernelProcessor.h modified , text changed , props changed

Properties

Name Value
commit-bot commit-bot@chromium.org

Powered by ViewVC 1.1.26 ViewVC Help