[webkit-changes] [WebKit/WebKit] b83027: Potential crash fix by bailing from DeleteSelectio...
Ahmad Saleem
noreply at github.com
Wed Nov 30 16:40:59 PST 2022
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: b83027e6d7d01b7ec451b6086430aed994c2ed12
https://github.com/WebKit/WebKit/commit/b83027e6d7d01b7ec451b6086430aed994c2ed12
Author: Ahmad Saleem <ahmad.saleem792+github at gmail.com>
Date: 2022-11-30 (Wed, 30 Nov 2022)
Changed paths:
A LayoutTests/editing/execCommand/delete-non-editable-range-crash-expected.txt
A LayoutTests/editing/execCommand/delete-non-editable-range-crash.html
M Source/WebCore/editing/DeleteSelectionCommand.cpp
Log Message:
-----------
Potential crash fix by bailing from DeleteSelectionCommand::doApply when selection isn't editable
Potential crash fix by bailing from DeleteSelectionCommand::doApply when selection isn't editable
https://bugs.webkit.org/show_bug.cgi?id=248486
Reviewed by Ryosuke Niwa.
Merge - https://src.chromium.org/viewvc/blink?view=revision&revision=190121
This patch is to harden the function doApply() by bailing early by ensuring the selection to delete is not editable.
* Source/WebCore/editing/DeleteSelectionCommand.cpp:
(DeleteSelectionCommand::doApply): Add condition about selection being non-editable
* LayoutTests/editing/execCommand/delete-non-editable-range-crash.html: Add Test Case
* LayoutTests/editing/execCommand/delete-non-editable-range-crash-expected.txt: Add Test Case Expectations
Canonical link: https://commits.webkit.org/257203@main
More information about the webkit-changes
mailing list