Integer-overflow in blink::IntPoint::move |
||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=6036424619196416 Fuzzer: inferno_twister Job Type: linux_ubsan_chrome Platform Id: linux Crash Type: Integer-overflow Crash Address: Crash State: blink::IntPoint::move blink::LocalDOMWindow::moveBy blink::DOMWindowV8Internal::moveByMethodCallback Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_ubsan_chrome&range=370022:370027 Minimized Testcase (0.11 Kb): Download: https://cluster-fuzz.appspot.com/download/AMIfv96uQgE-hw7wrJ2XpTee6kPM0CMenpg-ZOOd-JxZSmyzwbGt2iP6NtoLLactfUCsyMq4lE90ZeFQBCHqkxYeao3q3wQr4xCnGqpNfCcxxb4KjfzixxT4KQOcw25KXQspo1x1-Pfwncp2in4UG95ziUhoC8JE8Q?testcase_id=6036424619196416 <script> y = 47; x = 65428; window.moveTo(x, y); x = 2147483543; window.moveBy(x, y); </script> Issue manually filed by: nyerramilli See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Aug 12 2016
Regression range has no likely suspect; the least unlikely is the clang roll...
,
Aug 19 2016
eae@ could you please help us here to find the culprit and feel free to assigned back if needed.Thanks in advance
,
Aug 19 2016
Not a regression but should be easy to fix.
,
Aug 22 2016
Removing Needs-triage label.
,
Aug 23 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/47e39c323732b04b595f7a46a225b3cd7e768567 commit 47e39c323732b04b595f7a46a225b3cd7e768567 Author: eae <eae@chromium.org> Date: Tue Aug 23 23:06:32 2016 Fix int overflow in window.moveBy Fix an int overflow bug in window.moveBy by changing it to saturate when performing addition though a new saturatedMove method on IntPoint, Rect. BUG= 635799 R=pdr@chromium.org Review-Url: https://codereview.chromium.org/2264723002 Cr-Commit-Position: refs/heads/master@{#413885} [modify] https://crrev.com/47e39c323732b04b595f7a46a225b3cd7e768567/third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp [modify] https://crrev.com/47e39c323732b04b595f7a46a225b3cd7e768567/third_party/WebKit/Source/platform/geometry/IntPoint.h [modify] https://crrev.com/47e39c323732b04b595f7a46a225b3cd7e768567/third_party/WebKit/Source/platform/geometry/IntRect.h
,
Aug 23 2016
,
Aug 25 2016
ClusterFuzz has detected this issue as fixed in range 413791:414128. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=6036424619196416 Fuzzer: inferno_twister Job Type: linux_ubsan_chrome Platform Id: linux Crash Type: Integer-overflow Crash Address: Crash State: blink::IntPoint::move blink::LocalDOMWindow::moveBy blink::DOMWindowV8Internal::moveByMethodCallback Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_ubsan_chrome&range=370022:370027 Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_ubsan_chrome&range=413791:414128 Minimized Testcase (0.11 Kb): Download: https://cluster-fuzz.appspot.com/download/AMIfv96uQgE-hw7wrJ2XpTee6kPM0CMenpg-ZOOd-JxZSmyzwbGt2iP6NtoLLactfUCsyMq4lE90ZeFQBCHqkxYeao3q3wQr4xCnGqpNfCcxxb4KjfzixxT4KQOcw25KXQspo1x1-Pfwncp2in4UG95ziUhoC8JE8Q?testcase_id=6036424619196416 <script> y = 47; x = 65428; window.moveTo(x, y); x = 2147483543; window.moveBy(x, y); </script> 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.
,
Oct 18 2016
,
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
, Aug 9 2016Components: Tools>Test>FindIt>WrongResult Blink
Labels: Needs-triage findit-wrong Te-Logged M-53