Integer-overflow in WebRtcSpl_FilterARFastQ12 |
|||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=5914469531385856 Fuzzer: libFuzzer_neteq_signal_fuzzer Job Type: libfuzzer_chrome_ubsan Platform Id: linux Crash Type: Integer-overflow Crash Address: Crash State: WebRtcSpl_FilterARFastQ12 webrtc::Expand::Process webrtc::NetEqImpl::DoExpand Sanitizer: undefined (UBSAN) Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_ubsan&range=510927:510949 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5914469531385856 Issue filed automatically. See https://chromium.googlesource.com/chromium/src/+/master/testing/libfuzzer/reference.md for more information.
,
Nov 13 2017
,
Dec 7 2017
The following revision refers to this bug: https://webrtc.googlesource.com/src.git/+/6f54e7ab679fe13c62e33c969d3b75d6f71eb4eb commit 6f54e7ab679fe13c62e33c969d3b75d6f71eb4eb Author: Henrik Lundin <henrik.lundin@webrtc.org> Date: Thu Dec 07 16:56:47 2017 Avoid integer-overflow in WebRtcSpl_FilterARFastQ12 Simply use int64_t instead of int32_t for two local variables. This avoids integer-overflow in some rare cases. Bug: chromium:784216 Change-Id: Ie96835d6dc04e338b157469b74ead29e8bd403dd Reviewed-on: https://webrtc-review.googlesource.com/26580 Reviewed-by: Karl Wiberg <kwiberg@webrtc.org> Commit-Queue: Henrik Lundin <henrik.lundin@webrtc.org> Cr-Commit-Position: refs/heads/master@{#21141} [modify] https://crrev.com/6f54e7ab679fe13c62e33c969d3b75d6f71eb4eb/common_audio/signal_processing/filter_ar_fast_q12.c
,
Dec 8 2017
Note: There was a slight performance regression for 32-bit platforms with this fix. See crbug.com/793226 .
,
Dec 9 2017
ClusterFuzz has detected this issue as fixed in range 522890:522915. Detailed report: https://clusterfuzz.com/testcase?key=5914469531385856 Fuzzer: libFuzzer_neteq_signal_fuzzer Job Type: libfuzzer_chrome_ubsan Platform Id: linux Crash Type: Integer-overflow Crash Address: Crash State: WebRtcSpl_FilterARFastQ12 webrtc::Expand::Process webrtc::NetEqImpl::DoExpand Sanitizer: undefined (UBSAN) Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_ubsan&range=510927:510949 Fixed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_ubsan&range=522890:522915 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5914469531385856 See https://chromium.googlesource.com/chromium/src/+/master/testing/libfuzzer/reference.md for more information. If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
,
Dec 9 2017
ClusterFuzz testcase 5914469531385856 is verified as fixed, so closing issue as verified. If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue. |
|||
►
Sign in to add a comment |
|||
Comment 1 by ClusterFuzz
, Nov 13 2017Labels: Test-Predator-Auto-CC