Null-dereference READ in I422AlphaToARGBRow_SSSE3 |
|||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=5457325784301568 Fuzzer: attekett_surku_fuzzer Job Type: linux_ubsan_chrome Platform Id: linux Crash Type: Null-dereference READ Crash Address: 0x000000000000 Crash State: I422AlphaToARGBRow_SSSE3 libyuv::I420AlphaToARGBMatrix I420AlphaToARGB Sanitizer: undefined (UBSAN) Regressed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=583622:583623 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5457325784301568 Issue filed automatically. See https://github.com/google/clusterfuzz-tools for more information.
,
Aug 20
Almost the same as crbug.com/875158
,
Aug 20
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ecd725db6011e2b5a6f8f569e8585c7eb0ab38f2 commit ecd725db6011e2b5a6f8f569e8585c7eb0ab38f2 Author: Miguel Casas <mcasas@chromium.org> Date: Mon Aug 20 22:10:46 2018 GMBVFPool: bail if VideoFrame format forces change of OutputFormat ClusterFuzz encountered an issue if a given VideoFrame comes and causes a change of OutputFormat: in the bug below, an I420 VF was received after a series of I420A ones, and caused a crash when the alpha channel was accessed. This might happen more generally if the received VideoFrames cause a change of OutputFormat and we access what we don't have. This CL addresses that by comparing the current |output_format_| with what is derived out of the incoming VideoFrame. If they differ, we just bail. Bug: 875158 , 875670 Change-Id: I961b5bf95651a3888482957ee0b7acd48149d467 Reviewed-on: https://chromium-review.googlesource.com/1180543 Commit-Queue: Miguel Casas <mcasas@chromium.org> Reviewed-by: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#584574} [modify] https://crrev.com/ecd725db6011e2b5a6f8f569e8585c7eb0ab38f2/media/video/gpu_memory_buffer_video_frame_pool.cc [modify] https://crrev.com/ecd725db6011e2b5a6f8f569e8585c7eb0ab38f2/media/video/gpu_memory_buffer_video_frame_pool_unittest.cc
,
Aug 21
ClusterFuzz has detected this issue as fixed in range 584573:584574. Detailed report: https://clusterfuzz.com/testcase?key=5457325784301568 Fuzzer: attekett_surku_fuzzer Job Type: linux_ubsan_chrome Platform Id: linux Crash Type: Null-dereference READ Crash Address: 0x000000000000 Crash State: I422AlphaToARGBRow_SSSE3 libyuv::I420AlphaToARGBMatrix I420AlphaToARGB Sanitizer: undefined (UBSAN) Regressed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=583622:583623 Fixed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=584573:584574 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5457325784301568 See https://github.com/google/clusterfuzz-tools for more information. If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
,
Aug 21
ClusterFuzz testcase 5457325784301568 is verified as fixed, so closing issue as verified. If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
,
Aug 21
|
|||
►
Sign in to add a comment |
|||
Comment 1 by ClusterFuzz
, Aug 19Owner: mcasas@chromium.org
Status: Assigned (was: Untriaged)