Issue metadata
Sign in to add a comment
|
Data race in blink::UserGestureIndicator::UserGestureIndicator |
||||||||||||||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=5785701691162624 Fuzzer: inferno_layout_test_unmodified Job Type: linux_tsan_chrome_mp Platform Id: linux Crash Type: Data race READ 8 Crash Address: 0x7fd929bd43a0 Crash State: blink::UserGestureIndicator::UserGestureIndicator blink::DOMTimer::fired blink::TimerBase::runInternal Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=429267:429319 Minimized Testcase (0.55 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97GjiI4z9eAHBino-3gsPtDgOlm-x-cBiiuOxjWX4e7UXpCpUbFbcjjjscYMMy9Ekxn0wDGMn-j26A9w1FxIAQcEiBjc3mNn9PnZgKZkKZf9H7lpSvGltJT97kI2H44PPX-ztHMUu8nLFLNxxPzwPH1LyWHGg?testcase_id=5785701691162624 Additional requirements: Requires Gestures Issue filed automatically. See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Nov 8 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/28da3bf9d966cb3048f4dff0284f2173ecb5c5e7 commit 28da3bf9d966cb3048f4dff0284f2173ecb5c5e7 Author: rdevlin.cronin <rdevlin.cronin@chromium.org> Date: Tue Nov 08 00:51:32 2016 [Blink] Check thread access in UserGestureIndicator We silently ignore UserGestureIndicators on non-main threads, but tried checking equality against s_rootToken before checking the thread. Check the thread before token equality. BUG= 662710 Review-Url: https://codereview.chromium.org/2483983002 Cr-Commit-Position: refs/heads/master@{#430450} [modify] https://crrev.com/28da3bf9d966cb3048f4dff0284f2173ecb5c5e7/third_party/WebKit/Source/platform/UserGestureIndicator.cpp
,
Nov 8 2016
I think this should be fixed by #2, but am hoping that clusterfuzz will automatically update with a fix detection.
,
Nov 8 2016
ClusterFuzz has detected this issue as fixed in range 430398:430495. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5785701691162624 Fuzzer: inferno_layout_test_unmodified Job Type: linux_tsan_chrome_mp Platform Id: linux Crash Type: Data race READ 8 Crash Address: 0x7fd929bd43a0 Crash State: blink::UserGestureIndicator::UserGestureIndicator blink::DOMTimer::fired blink::TimerBase::runInternal Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=429267:429319 Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=430398:430495 Minimized Testcase (0.55 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97GjiI4z9eAHBino-3gsPtDgOlm-x-cBiiuOxjWX4e7UXpCpUbFbcjjjscYMMy9Ekxn0wDGMn-j26A9w1FxIAQcEiBjc3mNn9PnZgKZkKZf9H7lpSvGltJT97kI2H44PPX-ztHMUu8nLFLNxxPzwPH1LyWHGg?testcase_id=5785701691162624 Additional requirements: Requires Gestures See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information. If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
,
Nov 8 2016
ClusterFuzz testcase is verified as fixed, closing issue. If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
,
Nov 8 2016
Yay! Good job, clusterfuzz.
,
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 nyerramilli@chromium.org
, Nov 7 2016Labels: -Type-Bug M-56 Test-Predator-Correct-CLs Type-Bug-Regression
Owner: rdevlin....@chromium.org
Status: Assigned (was: Untriaged)