1051173 - crash in nsEditorSpellCheck::CheckCurrentDictionary()

Status: RESOLVED DUPLICATE of bug 1056322

(Core :: Spelling checker, defect)

Description

[Nils Maier [:nmaier]]

This bug was filed from the Socorro interface and is 
report bp-a4e7eb4b-f2d3-4798-b2c4-7e1cc2140809.
=============================================================

STR:
Run the following:
var editorSpellCheck = Cc["@mozilla.org/editor/editorspellchecker;1"].createInstance(Ci.nsIEditorSpellCheck);
editorSpellCheck.checkCurrentDictionary();

A cursorary check reveals that the crash seems to happen because |::mSpellChecker| was not initialized and that particular method lacks a |NS_ENSURE_TRUE(mSpellChecker, NS_ERROR_NOT_INITIALIZED);| check (that other methods use).

"Found" via StackOverflow, see URL.

Comment 1

[Wayne Mery (:wsmwk)]

bp-a4e7eb4b-f2d3-4798-b2c4-7e1cc2140809
 0 	XUL	nsEditorSpellCheck::CheckCurrentDictionary()	editor/composer/src/nsEditorSpellCheck.cpp
1 	XUL	NS_InvokeByIndex	xpcom/reflect/xptcall/src/md/unix/xptcinvoke_x86_64_unix.cpp
2 	XUL	XPCWrappedNative::CallMethod(XPCCallContext&, XPCWrappedNative::CallMode)	js/xpconnect/src/XPCWrappedNative.cpp 

right in the beginning of the method...
mail@78229 624 NS_IMETHODIMP
mail@78229 625 nsEditorSpellCheck::CheckCurrentDictionary()
mail@78229 626 {
mail@78229 627 mSpellChecker->CheckCurrentDictionary(); 

example windows crash...
bp-fa1181c9-36e0-4b54-8966-68dd22140808 kekc " Tried to call checkCurrentDictionary() in nsIEditorSpellCheck from an addon. Seems to crash the FF immediately."

Comment 2

[Wayne Mery (:wsmwk)]

duplicate of bug 1056322?

Comment 3

[Makoto Kato [:m_kato]]

(In reply to Wayne Mery (:wsmwk, NI for questions) from comment #2)
> duplicate of bug 1056322?

Yes.  Spell check should be initialized.