Issue metadata
Sign in to add a comment
|
Crash in base::DictionaryValue::SetIntegerWithoutPathExpansion |
||||||||||||||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=4512911332737024 Fuzzer: inferno_twister Job Type: linux_asan_chrome_media Platform Id: linux Crash Type: UNKNOWN READ Crash Address: 0x0000000002a0 Crash State: base::DictionaryValue::SetIntegerWithoutPathExpansion media::VideoFrameMetadata::SetInteger media::FFmpegVideoDecoder::GetVideoBuffer Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_media&range=420372:420473 Minimized Testcase (6.37 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97-pg9ANglu8kzFzUBYYd1C-AhGfx_JnIOmikQ_43Mlcv53vKbh-7lx8yxLKCpEnTFAIg1pt-NiLQVNf353xhcexPt-GBjBNsBYXW1Pd-ffDrsFwlJ_H6W5Wr7F1oizISQiCAeyHNjWh1V2jbO8KjuuIjbNUQ?testcase_id=4512911332737024 Additional requirements: Requires HTTP Issue manually filed by: kavvaru See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Sep 26 2016
The relevant log is: [1:7:0923/104841:ERROR:video_frame_pool.cc(87)] Failed to create a video frame In https://cs.chromium.org/chromium/src/media/filters/ffmpeg_video_decoder.cc?rcl=1474860215&l=141, we don't check that we successfully created a video frame, so we end up trying to deref a null pointer when updating the video frame's metadata.
,
Sep 28 2016
hubbe: Looks like your change (TOTALLY NOT YOUR FAULT, THOUGH) to fix a fuzzer issue is causing this: commit 2ba53c9cf88833aabbb642e53de195fb150e28f0 Author: hubbe <hubbe@chromium.org> Date: Thu Sep 22 13:24:00 2016 Fix a fuzzer crash in VideoFramePool You started returning nullptr, but because of an interface design choice (one that that I warned people about in the past), the callee (FFmpegVideoDecoder::GetVideoBuffer()) is not checking for nullptr.
,
Oct 5 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6f4c79b2b5a767a56d7fc3158697cd3941e001ac commit 6f4c79b2b5a767a56d7fc3158697cd3941e001ac Author: hubbe <hubbe@chromium.org> Date: Wed Oct 05 21:14:49 2016 Fix crash in fuzz test. When provided with odd configurations, the video frame pool can return null, causing a crash. Add a simple workaround. BUG= 650191 Review-Url: https://codereview.chromium.org/2390823006 Cr-Commit-Position: refs/heads/master@{#423294} [modify] https://crrev.com/6f4c79b2b5a767a56d7fc3158697cd3941e001ac/media/filters/ffmpeg_video_decoder.cc
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6f4c79b2b5a767a56d7fc3158697cd3941e001ac commit 6f4c79b2b5a767a56d7fc3158697cd3941e001ac Author: hubbe <hubbe@chromium.org> Date: Wed Oct 05 21:14:49 2016 Fix crash in fuzz test. When provided with odd configurations, the video frame pool can return null, causing a crash. Add a simple workaround. BUG= 650191 Review-Url: https://codereview.chromium.org/2390823006 Cr-Commit-Position: refs/heads/master@{#423294} [modify] https://crrev.com/6f4c79b2b5a767a56d7fc3158697cd3941e001ac/media/filters/ffmpeg_video_decoder.cc
,
Oct 27 2016
,
Nov 4 2016
[Automated comment] removing mislabelled merge-merged-2840
,
Nov 22 2016
Removing EditIssue view restrictions from ClusterFuzz filed bugs. If you believe that this issue should still be restricted, please reapply the label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by kavvaru@chromium.org
, Sep 26 2016Owner: dcheng@chromium.org
Status: Assigned (was: Untriaged)