New issue
Advanced search Search tips

Issue 616068 link

Starred by 0 users

Issue metadata

Status: Verified
Owner: ----
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Data race in blink::StringCacheMapTraits::Dispose

Project Member Reported by ClusterFuzz, May 31 2016

Issue description

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=4558499862544384

Fuzzer: ochang_domfuzzer
Job Type: linux_tsan_chrome_mp
Platform Id: linux

Crash Type: Data race READ 4
Crash Address: 0x7d2400035dc0
Crash State:
  blink::StringCacheMapTraits::Dispose
  v8::PersistentValueMapBase<WTF::StringImpl*, v8::String, blink::StringCacheMapTr
  blink::StringCache::dispose
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=396083:396125

Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv97CHaVVHBdvHLG3O9pyijC7i9EPR08j3FPSW_YbgOg4Vl_gqjnhKE4Wnkj3lcrRr69vBmB9FljwQ2trozx0Vk1YS-qOWdf3pNw9KXcwqjPvsmdOGpID8Lnzv7WGYhysOAbp8eiKhl5tBT79MNjanMnzECW60w


Filer: ajha

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 

Comment 1 by ajha@chromium.org, May 31 2016

Cc: haraken@chromium.org yutak@chromium.org
Components: Tools>Test>FindIt>NoResult
Labels: -Type-Bug -Pri-2 Te-Logged M-53 Pri-1 Type-Bug-Regression
Owner: koten...@yandex-team.ru
Status: Assigned (was: Available)
Based on the code search on 'StringImpl.h', kotenkov@: Could this be related to https://codereview.chromium.org/2016223002.

Please help in investigating this further if the change is not related.

Cc'ing haraken@ and yutak@ as the reviewer of the above CL. 
This should not be related, since it is a revert of a CL that was introduced a couple of days earlier and both CLs don't do anything except renaming RELEASE_ASSERTS.
Unfortunately, I don't have access to clusterfuzz, so I cannot se the testcase and investigate it further.

Comment 3 by yutak@chromium.org, Jun 1 2016

Cc: koten...@yandex-team.ru
Components: Blink>Workers
Owner: ----
Status: Untriaged (was: Assigned)
Indeed kotenkov's change has nothing to do with this.

This appears worker-related.
Status: Available (was: Untriaged)
Project Member

Comment 5 by ClusterFuzz, Jun 2 2016

ClusterFuzz has detected this issue as fixed in range 397130:397162.

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=4558499862544384

Fuzzer: ochang_domfuzzer
Job Type: linux_tsan_chrome_mp
Platform Id: linux

Crash Type: Data race READ 4
Crash Address: 0x7d2400035dc0
Crash State:
  blink::StringCacheMapTraits::Dispose
  v8::PersistentValueMapBase<WTF::StringImpl*, v8::String, blink::StringCacheMapTr
  blink::StringCache::dispose
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=396083:396125
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=397130:397162

Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv97CHaVVHBdvHLG3O9pyijC7i9EPR08j3FPSW_YbgOg4Vl_gqjnhKE4Wnkj3lcrRr69vBmB9FljwQ2trozx0Vk1YS-qOWdf3pNw9KXcwqjPvsmdOGpID8Lnzv7WGYhysOAbp8eiKhl5tBT79MNjanMnzECW60w


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.
Project Member

Comment 6 by ClusterFuzz, Jun 13 2016

Labels: ClusterFuzz-Verified
Status: Verified (was: Available)
ClusterFuzz testcase is verified as fixed, closing issue.

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
Components: -Tools>Test>FindIt>NoResult
Project Member

Comment 8 by sheriffbot@chromium.org, Nov 22 2016

Labels: -Restrict-View-EditIssue
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