mirror of
https://github.com/mollyim/webrtc.git
synced 2025-05-13 22:00:47 +01:00
d2b5b1f5ba
This change handles a special case in NoiseSuppression. The special case was found by the AudioProcessing fuzzer. A const copy of the capture audio stream is sent to NoiseSuppression::AnalyzeCaptureAudio. Then audio undergoes processing by e.g. the echo canceller. Then it's processed by NoiseSuppression::ProcessCaptureAudio. The special case is when the following conditions are all satisfied: * All stream samples are constantly zero in the call to AnalyzeCaptureAudio * a processing component modifies it to be nonzero before the call to ProcessCaptureAudio * The array NoiseSuppressionC::magnPrevAnalyze is filled with zeros. This holds after initialization. In this case, there is a division by zero in WebRtcNs_ProcessCore. The resulting NaN values pollute the output signal. They are only detected several submodules later in the process chain. The NaN values cause the EchoDetector to crash in debug mode. There is special handling of the case when the signal is constant zero in ProcessCore. This change avoids zero division by handling this issue the same way. Bug: chromium:803810 chromium:804634 Change-Id: I6d698dd0cd27e6d550b42085124300ce58533125 Reviewed-on: https://webrtc-review.googlesource.com/41282 Commit-Queue: Alex Loiko <aleloi@webrtc.org> Reviewed-by: Henrik Lundin <henrik.lundin@webrtc.org> Cr-Commit-Position: refs/heads/master@{#21745} |
||
|---|---|---|
| .. | ||
| defines.h | ||
| noise_suppression.c | ||
| noise_suppression.h | ||
| noise_suppression_x.c | ||
| noise_suppression_x.h | ||
| ns_core.c | ||
| ns_core.h | ||
| nsx_core.c | ||
| nsx_core.h | ||
| nsx_core_c.c | ||
| nsx_core_mips.c | ||
| nsx_core_neon.c | ||
| nsx_defines.h | ||
| windows_private.h | ||