commit | 421c85935126c5c529149236fb5de4caaf5b925f | [log] [tgz] |
---|---|---|
author | Sam Zackrisson <saza@webrtc.org> | Mon Feb 11 12:39:46 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Mon Feb 11 14:14:40 2019 |
tree | 4bdca2d8cb30ee4ff63ad6c0138720d37bff970e | |
parent | 00f9400d8272772808d78120224635344f0b4f33 [diff] |
Remove crit_render_ lock from webrtc::GainControlImpl The lock is unnecessary and potentially unsafe: 1) All gain_control accesses in AudioProcessingImpl happen - and are intended to happen - while holding the crit_capture_ lock, and all external API calls take the same lock once inside GainControlImpl. 2) If ProcessCaptureStreamLocked (locked by crit_capture) calls a gain_control function that takes crit_render, the mandated locking order (render before capture) is violated and we might get a deadlock with the render thread. Bug: b/123456404 Change-Id: Id7a888827e347e5e1d50e2f87d90e8b68f52b7b8 Reviewed-on: https://webrtc-review.googlesource.com/c/122087 Reviewed-by: Alessio Bazzica <alessiob@webrtc.org> Commit-Queue: Sam Zackrisson <saza@webrtc.org> Cr-Commit-Position: refs/heads/master@{#26637}
WebRTC is a free, open software project that provides browsers and mobile applications with Real-Time Communications (RTC) capabilities via simple APIs. The WebRTC components have been optimized to best serve this purpose.
Our mission: To enable rich, high-quality RTC applications to be developed for the browser, mobile platforms, and IoT devices, and allow them all to communicate via a common set of protocols.
The WebRTC initiative is a project supported by Google, Mozilla and Opera, amongst others.
See http://www.webrtc.org/native-code/development for instructions on how to get started developing with the native code.
Authoritative list of directories that contain the native API header files.