Bug 220630 – Crash from CompositeEditCommand::moveParagraphs() being passed null end

• Status: RESOLVED
• Resolution: FIXED
• Component: HTML Editing

[Julian Gonzalez]

frame #0: WebCore`WTF::Optional<WebCore::SimpleRange>::operator*() &&+46
frame #1: WebCore`WebCore::CompositeEditCommand::moveParagraphs(WebCore::VisiblePosition const&, WebCore::VisiblePosition const&, WebCore::VisiblePosition const&, bool, bool)+2041
frame #2: WebCore`WebCore::InsertListCommand::unlistifyParagraph(WebCore::VisiblePosition const&, WebCore::HTMLElement*, WebCore::Node*)+2703
frame #3: WebCore`WebCore::InsertListCommand::doApplyForSingleParagraph(bool, WebCore::HTMLQualifiedName const&, WebCore::SimpleRange&)+3029
frame #4: WebCore`WebCore::InsertListCommand::doApply()+3303
frame #5: WebCore`WebCore::CompositeEditCommand::applyCommandToComposite(WTF::Ref<WebCore::EditCommand, WTF::RawPtrTraits<WebCore::EditCommand> >&&)+107
frame #6: WebCore`WebCore::IndentOutdentCommand::outdentParagraph()+860
frame #7: WebCore`WebCore::IndentOutdentCommand::outdentRegion(WebCore::VisiblePosition const&, WebCore::VisiblePosition const&)+1386
frame #8: WebCore`WebCore::IndentOutdentCommand::formatSelection(WebCore::VisiblePosition const&, WebCore::VisiblePosition const&)+48
frame #9: WebCore`WebCore::ApplyBlockElementCommand::doApply()+1114
frame #10: WebCore`WebCore::CompositeEditCommand::apply()+535
frame #11: WebCore`WebCore::executeOutdent(WebCore::Frame&, WebCore::Event*, WebCore::EditorCommandSource, WTF::String const&)+204

<rdar://problem/72223506>

[Julian Gonzalez]

Created attachment 417641 [details]
Patch

[Ryosuke Niwa]

Comment on attachment 417641 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=417641&action=review

> Source/WebCore/editing/InsertListCommand.cpp:303
> +    // Make sure there are valid positions in the paragraph to move.

I don't think this comment is necessary. It's pretty evident from the code why we'd have to check it here.

[Julian Gonzalez]

(In reply to Ryosuke Niwa from comment #2)
> Comment on attachment 417641 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=417641&action=review
> 
> > Source/WebCore/editing/InsertListCommand.cpp:303
> > +    // Make sure there are valid positions in the paragraph to move.
> 
> I don't think this comment is necessary. It's pretty evident from the code
> why we'd have to check it here.

Removing the comment.

[Julian Gonzalez]

Created attachment 417672 [details]
Patch

[EWS]

Committed r271510: <https://trac.webkit.org/changeset/271510>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 417672 [details].

[Ryosuke Niwa]

*** Bug 218495 has been marked as a duplicate of this bug. ***