Issue metadata
Sign in to add a comment
|
Crash in SkSpecialImage_Raster::SkSpecialImage_Raster |
||||||||||||||||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=4898955716722688 Fuzzer: sugoi_filter_fuzzer Job Type: linux_asan_filter_fuzz_stub Platform Id: linux Crash Type: UNKNOWN WRITE Crash Address: 0x000000000008 Crash State: SkSpecialImage_Raster::SkSpecialImage_Raster SkSpecialImage::MakeFromRaster SkMagnifierImageFilter::onFilterImage Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_filter_fuzz_stub&range=387336:387375 Minimized Testcase (1.52 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97tnRjwwPyu-spvY_HvdaKp3yu4fGl0oyP4j8ME5oX8Fk1G7jru1w6fip1_BbproG7Zuu-VgI_d12J-_JzR3s2tvul2goeG5h1tl4K6s3DURsNn8kL193R-gaLnYBC_vBKLy9Qfx00GGBnJqtxfkHBeTjOkKQ?testcase_id=4898955716722688 Filer: ajha See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Jul 12 2016
IIUC, what appears to be going on is that we are exceeding the the soft rss limit and allocator_may_return_null is set to 1. So, the allocator is returning null but the initialization of the C++ class that was being newed continues on - leading to the writes to 0x08 (to initialize the ref count). It seems that we either want allocator_may_return_null to be set to false (which, I believe, mirrors Chromium's use case) or make sure asan doesn't allow the constructor to be called if memory allocation fails.
,
Jul 12 2016
Issue 627047 has been merged into this issue.
,
Jul 12 2016
,
Jul 12 2016
Issue 627342 has been merged into this issue.
,
Jul 13 2016
Issue 623669 has been merged into this issue.
,
Jul 13 2016
Detailed report: https://cluster-fuzz.appspot.com/testcase?key=4620812862881792 Fuzzer: sugoi_filter_fuzzer Job Type: linux_asan_filter_fuzz_stub_32bit Platform Id: linux Crash Type: UNKNOWN WRITE Crash Address: 0x00000004 Crash State: SkPixelRef::SkPixelRef SkMallocPixelRef::NewZeroed SkMallocPixelRef::ZeroedPRFactory::create Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_filter_fuzz_stub_32bit&range=357565:358520 Minimized Testcase (0.74 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97usyx3lPAMepeP-Fr839CzatXzKcarLHqN3wYxdtJQI8KkyJi1pq89UHyV29E8FXqdiTk_a73BYcK8UKEqumC4uTy6zSUgt-gf80yshCLPGy0TyI8lysi0bzPdyUErxR4gnKn7D8YItzjOt7nMzgQJv2krZQ?testcase_id=4620812862881792 Filer: rnimmagadda See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Jul 14 2016
reed/bsalomon@, could you please help us to find a right owner if possible? Thank you!
,
Jul 14 2016
It looks like "new" has returned NULL, which it is never supposed to! Is the fuzzer building/running in a different mode than chrome, and allowing new to return null?
,
Jul 14 2016
,
Jul 14 2016
Issue 628326 has been merged into this issue.
,
Aug 24 2016
ClusterFuzz has detected this issue as fixed in range 413737:413791. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=4620812862881792 Fuzzer: sugoi_filter_fuzzer Job Type: linux_asan_filter_fuzz_stub_32bit Platform Id: linux Crash Type: UNKNOWN WRITE Crash Address: 0x00000004 Crash State: SkPixelRef::SkPixelRef SkMallocPixelRef::NewZeroed SkMallocPixelRef::ZeroedPRFactory::create Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_filter_fuzz_stub_32bit&range=357565:358520 Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_filter_fuzz_stub_32bit&range=413737:413791 Minimized Testcase (0.74 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97usyx3lPAMepeP-Fr839CzatXzKcarLHqN3wYxdtJQI8KkyJi1pq89UHyV29E8FXqdiTk_a73BYcK8UKEqumC4uTy6zSUgt-gf80yshCLPGy0TyI8lysi0bzPdyUErxR4gnKn7D8YItzjOt7nMzgQJv2krZQ?testcase_id=4620812862881792 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 ajha@chromium.org
, Jul 12 2016Labels: findit-for-crash Te-Logged M-52
Owner: robertphillips@chromium.org
Status: Assigned (was: Available)