Issue metadata
Sign in to add a comment
|
Integer-overflow in blink::operator-= |
||||||||||||||||||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=6636323442262016 Fuzzer: inferno_twister Job Type: linux_ubsan_chrome Platform Id: linux Crash Type: Integer-overflow Crash Address: Crash State: blink::operator-= blink::PaintLayer::updateLayerPosition blink::PaintLayer::updateLayerPositionsAfterScrollRecursive Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_ubsan_chrome&range=418843:418863 Minimized Testcase (1.11 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94cYPEY50pZ_ZhgMpPh-2TVM_EdypvQl68DzsjgK8poxfnZMjs_9rdsGfOVzr5nX04BPh4BhVyfz7sAa_Pw3DVVBpiJUIyiU95myGD7r9T5H_sgJOPNi1bRwXdduvRweT7q_4L-R1bTmYxXoUoOfQzZBRDL6g?testcase_id=6636323442262016 Issue manually filed by: kavvaru See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Sep 22 2016
,
Sep 22 2016
I would have to suspect https://chromium.googlesource.com/chromium/src/+/6d73ab5ed5d9b00bc48da255dc6599e90e72f599 or, less likely, this https://chromium.googlesource.com/chromium/src/+/4567d59b87472d29a1fc660a55cd8aa56c83f280
,
Sep 22 2016
I'm not sure if this is caused by my patch, since my patch doesn't touch any of the functions on the stack trace. Besides, I cannot easily reproduce it by opening the minimized testcase in chromium.
,
Sep 22 2016
skobes@, can you take a look at it?
,
Sep 22 2016
Emil I think you were looking at the integer overflow problem recently? What is the right thing to do here?
,
Sep 25 2016
Still TBD, if we decide to fix it it should be at the level of IntRect and friends (just like we do for the Layout* units).
,
Oct 6 2016
ClusterFuzz has detected this issue as fixed in range 422899:423265. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=6636323442262016 Fuzzer: inferno_twister Job Type: linux_ubsan_chrome Platform Id: linux Crash Type: Integer-overflow Crash Address: Crash State: blink::operator-= blink::PaintLayer::updateLayerPosition blink::PaintLayer::updateLayerPositionsAfterScrollRecursive Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_ubsan_chrome&range=418843:418863 Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_ubsan_chrome&range=422899:423265 Minimized Testcase (1.11 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94cYPEY50pZ_ZhgMpPh-2TVM_EdypvQl68DzsjgK8poxfnZMjs_9rdsGfOVzr5nX04BPh4BhVyfz7sAa_Pw3DVVBpiJUIyiU95myGD7r9T5H_sgJOPNi1bRwXdduvRweT7q_4L-R1bTmYxXoUoOfQzZBRDL6g?testcase_id=6636323442262016 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 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 22 2016Labels: M-55 Te-Logged
Owner: schenney@chromium.org
Status: Assigned (was: Untriaged)