Reviewed by John
authorkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 Jan 2005 18:55:53 +0000 (18:55 +0000)
committerkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 Jan 2005 18:55:53 +0000 (18:55 +0000)
commitbbfc7bb4558772b2b9ae2e6498990ca8b7ece497
tree8ef5cf88e7b4327cfe158e09783acb2626625dd3
parentf3011e0151d0e1e7502f4027f9b74c17741e621e
    Reviewed by John

        Fix for this bug:

        <rdar://problem/3957204> Mail crashed when sending a message in ~InsertParagraphSeparatorInQuotedContentCommand

        * khtml/editing/htmlediting.cpp:
        (khtml::InsertParagraphSeparatorInQuotedContentCommand::InsertParagraphSeparatorInQuotedContentCommand):
        InsertParagraphSeparatorInQuotedContentCommand has this member variable: DOM::ElementImpl *m_breakNode;
        I failed to initialize this variable to null in the constructor, and there is an uncommonly-traveled
        code path which does not set this variable to something good. In the destructor, we check for null,
        and deref if non-null. Obvious "BOOM" potential. Fixed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@8426 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog-2005-08-23
WebCore/khtml/editing/htmlediting.cpp