Data race in base::GlobalHistogramAllocator::Get |
||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=6210706660917248 Fuzzer: phoglund_webrtc_peerconnection Job Type: linux_tsan_chrome_mp Platform Id: linux Crash Type: Data race READ 8 Crash Address: 0x7f8fbe73a950 Crash State: base::GlobalHistogramAllocator::Get base::StatisticsRecorder::FindHistogram base::Histogram::Factory::Build Sanitizer: thread (TSAN) Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=445491:445525 Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv95wP7cXhAvji8L-Qj0wlFvK7hY-4tVtIL119RSPeneShxc9OjFQafiY6hd1u-3kXK5gcG9rP4xhr2CsND7B2EMwStpN09RoGSNg29BQ-aaXBKhEpjoBi8ivLYdn64fS1YZ7ccS3QcwK2FTluusUA72rGF04wJZCS-zejbdXRAWhwJj2DXdGhp4kcVd5CkNFTjW99Zylx62viQTk0EMuReMGCyOlJl-7oliF6A0CBXfJ3iZbPNMGGxayv6oOayL96Mn4_CzXWlSCvrUqVhYu8YTijwvcn0IO3KBOvbsWt25S23i61LfQKSebuTDnuejuktyPEEyANqyh3d9UIy8lYwiq42CvkTpLK_lcEs5k2aB6G4bCd8g?testcase_id=6210706660917248 Additional requirements: Requires HTTP Issue filed automatically. See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Jan 24 2017
https://codereview.chromium.org/2649813003 is a pure refactor and did not cause this.
,
Jan 24 2017
Thank You Alexclarke. Using Code Search for the file, "persistent_histogram_allocator.cc" assigning to the concern owner. Suspecting Commit# https://chromium.googlesource.com/chromium/src/+/4bc446aaa33205d3d1573cc90c8801edcef88922 @bcwhite -- Could you please look into the issue, kindly re-assign if this is not related to your changes. Thank You.
,
Jan 24 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a7d5177f1f8d47bcae75304d19b2728533e267f6 commit a7d5177f1f8d47bcae75304d19b2728533e267f6 Author: bcwhite <bcwhite@chromium.org> Date: Tue Jan 24 20:59:28 2017 Use atomic ops for managing g_allocator. BUG= 684498 Review-Url: https://codereview.chromium.org/2655673002 Cr-Commit-Position: refs/heads/master@{#445805} [modify] https://crrev.com/a7d5177f1f8d47bcae75304d19b2728533e267f6/base/metrics/persistent_histogram_allocator.cc
,
Jan 25 2017
Issue 685048 has been merged into this issue.
,
Jan 30 2017
,
Jan 30 2017
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 30 2017
Please merge your change to M57 branch 2987 ASAP.If merge happens today before 5:00 PM PT, then we can take it for tomorrow's last M57 Dev release. Thank you.
,
Jan 30 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9271f371324eb165ba5f6ec24d1f1aba0d26165d commit 9271f371324eb165ba5f6ec24d1f1aba0d26165d Author: Brian White <bcwhite@chromium.org> Date: Mon Jan 30 21:51:01 2017 Use atomic ops for managing g_allocator. BUG= 684498 Review-Url: https://codereview.chromium.org/2655673002 Cr-Commit-Position: refs/heads/master@{#445805} (cherry picked from commit a7d5177f1f8d47bcae75304d19b2728533e267f6) Review-Url: https://codereview.chromium.org/2665893002 . Cr-Commit-Position: refs/branch-heads/2987@{#191} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} [modify] https://crrev.com/9271f371324eb165ba5f6ec24d1f1aba0d26165d/base/metrics/persistent_histogram_allocator.cc
,
Jan 31 2017
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by msrchandra@chromium.org
, Jan 24 2017Components: Blink>Scheduling
Labels: Test-Predator-Correct-CLs
Owner: alexclarke@chromium.org
Status: Assigned (was: Untriaged)