Null-dereference READ in blink::SubresourceIntegrityHelper::DoReport |
|||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=6087073131134976 Fuzzer: inferno_twister Job Type: mac_asan_content_shell Platform Id: mac Crash Type: Null-dereference READ Crash Address: 0x000000000000 Crash State: blink::SubresourceIntegrityHelper::DoReport blink::FetchManager::Loader::SRIVerifier::DidGetReadable content::SharedMemoryDataConsumerHandle::Context::NotifyInternal Sanitizer: address (ASAN) Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=6087073131134976 Additional requirements: Requires HTTP Issue filed automatically. See https://github.com/google/clusterfuzz-tools for more information.
,
Sep 6 2017
Reproduced locally on Linux. The execution context is null.
,
Sep 7 2017
,
Sep 7 2017
,
Sep 12 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/13e0a33c0de604128233dd9e293e946070aa2f22 commit 13e0a33c0de604128233dd9e293e946070aa2f22 Author: Hiroshige Hayashizaki <hiroshige@chromium.org> Date: Tue Sep 12 03:17:18 2017 Cancel SRIVerifier when the corresponding context is destroyed Otherwise, DidGetReadable() is called with null ExecutionContext. Bug: 758883 Change-Id: I904d0c946a1be928c1fe2bebb37efec384a7bd59 Reviewed-on: https://chromium-review.googlesource.com/655807 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by: Yutaka Hirano <yhirano@chromium.org> Reviewed-by: Kouhei Ueno <kouhei@chromium.org> Cr-Commit-Position: refs/heads/master@{#501166} [modify] https://crrev.com/13e0a33c0de604128233dd9e293e946070aa2f22/third_party/WebKit/Source/modules/fetch/FetchManager.cpp
,
Sep 12 2017
ClusterFuzz has detected this issue as fixed in range 501164:501180. Detailed report: https://clusterfuzz.com/testcase?key=6087073131134976 Fuzzer: inferno_twister Job Type: mac_asan_content_shell Platform Id: mac Crash Type: Null-dereference READ Crash Address: 0x000000000000 Crash State: blink::SubresourceIntegrityHelper::DoReport blink::FetchManager::Loader::SRIVerifier::DidGetReadable content::SharedMemoryDataConsumerHandle::Context::NotifyInternal Sanitizer: address (ASAN) Fixed: https://clusterfuzz.com/revisions?job=mac_asan_content_shell&range=501164:501180 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=6087073131134976 Additional requirements: Requires HTTP 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.
,
Sep 13 2017
Fixed. Do we need to merge to M-62? DoReport is introduced recently, but the root cause has existed since before.
,
Sep 13 2017
ClusterFuzz testcase 6087073131134976 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 msrchandra@chromium.org
, Aug 25 2017Labels: M-62 Test-Predator-Wrong
Owner: hirosh...@chromium.org
Status: Assigned (was: Untriaged)