Closed
Bug 1266648
Opened 8 years ago
Closed 8 years ago
crash in nsHTMLEditor::SplitStyleAbovePoint
Categories
(Core :: DOM: Editor, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox48 | --- | affected |
People
(Reporter: n.nethercote, Assigned: ayg)
Details
(Keywords: crash)
Crash Data
This bug was filed from the Socorro interface and is report bp-a9353322-463c-46da-87b1-c043b2160422. ============================================================= This is a new crash that first appeared in Nightly 20160420030213. It's happened 7 times so far. It looks like it might be a null deref. > 0 xul.dll nsHTMLEditor::SplitStyleAbovePoint(nsCOMPtr<nsINode>*, int*, nsIAtom*, nsAString_internal const*, nsIContent**, nsIContent**) editor/libeditor/nsHTMLEditorStyle.cpp > 1 xul.dll nsHTMLEditRules::JoinBlocks(nsIContent&, nsIContent&, bool*) editor/libeditor/nsHTMLEditRules.cpp > 2 xul.dll nsHTMLEditRules::WillDeleteSelection(mozilla::dom::Selection*, short, short, bool*, bool*) editor/libeditor/nsHTMLEditRules.cpp > 3 xul.dll nsHTMLEditRules::WillDoAction(mozilla::dom::Selection*, nsRulesInfo*, bool*, bool*) editor/libeditor/nsHTMLEditRules.cpp > 4 xul.dll nsPlaintextEditor::DeleteSelection(short, short) editor/libeditor/nsPlaintextEditor.cpp > 5 xul.dll nsEditor::HandleKeyPressEvent(nsIDOMKeyEvent*) editor/libeditor/nsEditor.cpp > 6 xul.dll nsHTMLEditor::HandleKeyPressEvent(nsIDOMKeyEvent*) editor/libeditor/nsHTMLEditor.cpp > 7 xul.dll nsEditorEventListener::KeyPress(nsIDOMKeyEvent*) editor/libeditor/nsEditorEventListener.cpp > 8 xul.dll nsEditorEventListener::HandleEvent(nsIDOMEvent*) editor/libeditor/nsEditorEventListener.cpp It might be related to bug 1190172, which touched this code recently.
Assignee | ||
Comment 2•8 years ago
|
||
This is because I changed an NS_ENSURE_TRUE to a MOZ_ASSERT. If I were redoing these patches now, I probably wouldn't do that. The fix is just to change it back. Note: the patch seems to have been backed out for other reasons, so we just need to fix before relanding.
Assignee: nobody → ayg
Status: NEW → ASSIGNED
Flags: needinfo?(ayg)
Assignee | ||
Comment 3•8 years ago
|
||
Should be fixed in the new version of the patch: https://hg.mozilla.org/mozilla-central/rev/9c056080ee98#l3.68
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•