Issue metadata
Sign in to add a comment
|
Crash in SkAAClipBlitter::blitV |
||||||||||||||||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=4846459234287616 Fuzzer: miaubiz_svg_fuzzer Job Type: linux_asan_chrome_v8_arm Platform Id: linux Crash Type: UNKNOWN READ Crash Address: 0x00000000 Crash State: SkAAClipBlitter::blitV SkScan::aaa_fill_path SkScan::AAAFillPath Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_v8_arm&range=430188:430191 Minimized Testcase (4.66 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94AUOsigVkTLc15oD1WMA-bAlXYxdm90CyQSdThzKBUwC9Zpxyxfni5eaqViUlgPgcl756nA8MVIEt9s1S4M3Xjh5hHpF9xzOUd5ZXrjurlIENwd4hK6H-lbfoljq5KLCvDoHLfycphYaQ6OFZCv6lI20ICgw?testcase_id=4846459234287616 Issue filed automatically. See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Nov 7 2016
Note- adding the Skia component is sufficient so we can triage and get to the appropriate owner, reed & bsalomon are getting too much bug noise.
,
Nov 7 2016
,
Nov 8 2016
ClusterFuzz has detected this issue as fixed in range 430262:430287. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=4846459234287616 Fuzzer: miaubiz_svg_fuzzer Job Type: linux_asan_chrome_v8_arm Platform Id: linux Crash Type: UNKNOWN READ Crash Address: 0x00000000 Crash State: SkAAClipBlitter::blitV SkScan::aaa_fill_path SkScan::AAAFillPath Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_v8_arm&range=430188:430191 Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_v8_arm&range=430262:430287 Minimized Testcase (4.66 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94AUOsigVkTLc15oD1WMA-bAlXYxdm90CyQSdThzKBUwC9Zpxyxfni5eaqViUlgPgcl756nA8MVIEt9s1S4M3Xjh5hHpF9xzOUd5ZXrjurlIENwd4hK6H-lbfoljq5KLCvDoHLfycphYaQ6OFZCv6lI20ICgw?testcase_id=4846459234287616 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
,
Nov 9 2016
The following revision refers to this bug: https://skia.googlesource.com/skia.git/+/3ce89dad5bc324dad9c4b77393e16f3bcb7396a7 commit 3ce89dad5bc324dad9c4b77393e16f3bcb7396a7 Author: liyuqian <liyuqian@google.com> Date: Wed Nov 09 16:53:39 2016 Fix the meaning of stop_y stop_y means that we should stop exactly at stop_y, so the last row should be [stop_y - 1, stop_y], not [stop_y, stop_y + 1]. Somehow this misunderstanding didn't trigger any issue until Chrome exercises SkAAClip with some websites (e.g., http://www.lemonde.fr/elections-americaines/article/2016/11/07/deux-programmes-deux-visions-de-l-amerique_5026444_829254.html). When we blitter the extra row [stop_y, stop_y + 1], the SkAAClip will return nullptr by findRow. Later when that nullptr row is used to findX, the crash happened. BUG=chromium:662925, chromium:662776 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2481703004 Review-Url: https://codereview.chromium.org/2481703004 [modify] https://crrev.com/3ce89dad5bc324dad9c4b77393e16f3bcb7396a7/src/core/SkScan_AAAPath.cpp
,
Nov 9 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
, Nov 7 2016Components: Internals>Skia
Labels: -Pri-1 -Type-Bug Test-Predator-Wrong-CLs M-56 Pri-2 Type-Bug-Regression
Owner: reed@chromium.org
Status: Assigned (was: Untriaged)