ASSERT_NOT_REACHED in blink::PaintController::processNewItem |
|||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=6678992886693888 Fuzzer: inferno_twister Job Type: linux_debug_content_shell_drt Platform Id: linux Crash Type: ASSERT_NOT_REACHED Crash Address: Crash State: blink::PaintController::processNewItem void blink::PaintController::createAndAppend<blink::DrawingDisplayItem, blink::D blink::DrawingRecorder::~DrawingRecorder Minimized Testcase (0.99 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94djk5zr3lEHR2hhq6Mct6cbU5ePxqQ0kvrpO4oPjbyqeHu7qAXkQ_UvjNyQODZgbtjXut7NrBZbpaVoKODkJ-92e9SULXvKz3EkQX1wDrq5fKCva23HMGo3mzDGMkxeT5jnK0wTHhOc8W_AXP9OFtUSR4XVA Filer: ssamanoori See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Apr 18 2016
Minimized testcase
,
Apr 26 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6c961fe1112914b9d63d6551e31b96c415dfb83f commit 6c961fe1112914b9d63d6551e31b96c415dfb83f Author: pdr <pdr@chromium.org> Date: Tue Apr 26 03:17:41 2016 Rewrite overhanging float arithmetic to work near numeric limits This patch fixes a bug where floats near LayoutUnit::max() would incorrectly get added to two LayoutObjects which would lead to duplicate display items when the floats were painted by both parents. Examples sometimes make these easier to review: logicalTop = LayoutUnit::max() prevLogicalTop = (LayoutUnit::max() - 4) prevLowestFloatLogicalBottom = 8 Previous logic: if (prevLowestFloatLogicalBottom > logicalTop - prevLogicalTop) { ... New logic: if (logicalTop < prevLowestFloatLogicalBottom + prevLogicalTop) { ... BUG= 604331 Review URL: https://codereview.chromium.org/1916693002 Cr-Commit-Position: refs/heads/master@{#389679} [add] https://crrev.com/6c961fe1112914b9d63d6551e31b96c415dfb83f/third_party/WebKit/LayoutTests/fast/block/float/overhanging-float-overflow-crash-expected.txt [add] https://crrev.com/6c961fe1112914b9d63d6551e31b96c415dfb83f/third_party/WebKit/LayoutTests/fast/block/float/overhanging-float-overflow-crash.html [modify] https://crrev.com/6c961fe1112914b9d63d6551e31b96c415dfb83f/third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp
,
Apr 26 2016
This is marked as M50 but I'd like to let this roll through the regular release process.
,
Apr 26 2016
ClusterFuzz has detected this issue as fixed in range 389675:389686. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=6678992886693888 Fuzzer: inferno_twister Job Type: linux_debug_content_shell_drt Platform Id: linux Crash Type: ASSERT_NOT_REACHED Crash Address: Crash State: blink::PaintController::processNewItem void blink::PaintController::createAndAppend<blink::DrawingDisplayItem, blink::D blink::DrawingRecorder::~DrawingRecorder Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_content_shell_drt&range=389675:389686 Minimized Testcase (0.99 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94djk5zr3lEHR2hhq6Mct6cbU5ePxqQ0kvrpO4oPjbyqeHu7qAXkQ_UvjNyQODZgbtjXut7NrBZbpaVoKODkJ-92e9SULXvKz3EkQX1wDrq5fKCva23HMGo3mzDGMkxeT5jnK0wTHhOc8W_AXP9OFtUSR4XVA 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 ssamanoori@chromium.org
, Apr 18 2016Owner: pdr@chromium.org
Status: Assigned (was: Available)