Null-dereference READ in GetFlag |
||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=4752983492657152 Fuzzer: inferno_layout_test_unmodified Job Type: linux_lsan_chrome_mp Platform Id: linux Crash Type: Null-dereference READ Crash Address: 0x000000000010 Crash State: GetFlag HasRareData GetLayoutObject Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_lsan_chrome_mp&range=473419:473579 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4752983492657152 Issue filed automatically. See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Jul 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a6a78b205297ad9733dbe8d828dcc43cfad7667e commit a6a78b205297ad9733dbe8d828dcc43cfad7667e Author: yoichio <yoichio@chromium.org> Date: Mon Jul 10 09:21:40 2017 Use Layoutselection::SelectionBounds() in RevealSelection() Before this patch, we use VisibleUnit::ComputeTextRect() to compute a rect to scroll. However, ComputeTextRect() calls InlineTextBox::LocalSelectionRect() which depends on InlineTextBox current SelectionState. To avoid that, update SeletionState and get a rect directly from LayoutSelection::SelectionBounds(). BUG= 739062 , 740401 TEST=LayoutTests/editing/input/scroll-with-tab-to-input-regression.html Review-Url: https://codereview.chromium.org/2979513002 Cr-Commit-Position: refs/heads/master@{#485214} [modify] https://crrev.com/a6a78b205297ad9733dbe8d828dcc43cfad7667e/third_party/WebKit/Source/core/editing/FrameSelection.cpp [modify] https://crrev.com/a6a78b205297ad9733dbe8d828dcc43cfad7667e/third_party/WebKit/Source/core/editing/FrameSelection.h
,
Jul 11 2017
,
Jul 11 2017
Issue 735853 has been merged into this issue.
,
Jul 11 2017
,
Jul 11 2017
ClusterFuzz has detected this issue as fixed in range 485213:485241. Detailed report: https://clusterfuzz.com/testcase?key=4752983492657152 Fuzzer: inferno_layout_test_unmodified Job Type: linux_lsan_chrome_mp Platform Id: linux Crash Type: Null-dereference READ Crash Address: 0x000000000010 Crash State: GetFlag HasRareData GetLayoutObject Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_lsan_chrome_mp&range=473419:473579 Fixed: https://clusterfuzz.com/revisions?job=linux_lsan_chrome_mp&range=485213:485241 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4752983492657152 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.
,
Jul 12 2017
Issue 740868 has been merged into this issue.
,
Jul 18 2017
ClusterFuzz testcase 5804677278203904 is still reproducing on tip-of-tree build (trunk). Please re-test your fix against this testcase and if the fix was incorrect or incomplete, please re-open the bug. Otherwise, ignore this notification and add ClusterFuzz-Wrong label. |
||||
►
Sign in to add a comment |
||||
Comment 1 by msrchandra@chromium.org
, Jul 10 2017Components: Blink>Editing
Labels: M-60 Test-Predator-Correct-CLs
Owner: yoichio@chromium.org
Status: Assigned (was: Untriaged)