189530 - Trunk crash [@ nsTextServicesDocument::FirstBlock]

Status: VERIFIED DUPLICATE of bug 185251

(Core :: DOM: Editor, defect)

Description

[Jay Patel [:jay]]

This is a regression that started with 1/16 MozillaTrunk builds.  Making this a
zt4newcrash bug for now on the assumption that brade's checkin yesterday  might
have caused this crash.  Here is the latest from Talkback:

nsTextServicesDocument::FirstBlock  (14 crashes)

    BBID range: 16263330 - 16277674
    Min/Max Seconds since last crash: 39 - 20317
    Min/Max Runtime: 51 - 20443
    Crash data range: 2003-01-16 to 2003-01-16
    Build ID range: 2003011608 to 2003011608

    Stack Trace:
    nsTextServicesDocument::FirstBlock
[c:/builds/seamonkey/mozilla/editor/txtsvc/src/nsTextServicesDocument.cpp line 461]
    nsTextServicesDocument::GetCurrentTextBlock
[c:/builds/seamonkey/mozilla/editor/txtsvc/src/nsTextServicesDocument.cpp line 440]
    spellchk.dll + 0x266e (0x1000266e)
    spellchk.dll + 0x16a1 (0x100016a1)
    nsEditorSpellCheck::GetNextMisspelledWord
[c:/builds/seamonkey/mozilla/editor/composer/src/nsEditorSpellCheck.cpp line 167]
    XPTC_InvokeByIndex
[c:/builds/seamonkey/mozilla/xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp
line 102]
    XPCWrappedNative::CallMethod
[c:/builds/seamonkey/mozilla/js/src/xpconnect/src/xpcwrappednative.cpp line 2025]
    XPC_WN_CallMethod
[c:/builds/seamonkey/mozilla/js/src/xpconnect/src/xpcwrappednativejsops.cpp line
1293]
    js_Invoke [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 841]
    js_Interpret [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 2804]
    js_Invoke [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 857]
    js_InternalInvoke [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 932]
    JS_CallFunctionValue [c:/builds/seamonkey/mozilla/js/src/jsapi.c line 3433]
    nsJSContext::CallEventHandler
[c:/builds/seamonkey/mozilla/dom/src/base/nsJSEnvironment.cpp line 1043]
    nsJSEventListener::HandleEvent
[c:/builds/seamonkey/mozilla/dom/src/events/nsJSEventListener.cpp line 183]
    nsEventListenerManager::HandleEventSubType
[c:/builds/seamonkey/mozilla/content/events/src/nsEventListenerManager.cpp line
1217]
    nsEventListenerManager::HandleEvent
[c:/builds/seamonkey/mozilla/content/events/src/nsEventListenerManager.cpp line
1901]
    GlobalWindowImpl::HandleDOMEvent
[c:/builds/seamonkey/mozilla/dom/src/base/nsGlobalWindow.cpp line 805]
    DocumentViewerImpl::LoadComplete
[c:/builds/seamonkey/mozilla/content/base/src/nsDocumentViewer.cpp line 965]
    nsDocShell::EndPageLoad
[c:/builds/seamonkey/mozilla/docshell/base/nsDocShell.cpp line 4241]
    nsWebShell::EndPageLoad
[c:/builds/seamonkey/mozilla/docshell/base/nsWebShell.cpp line 776]
    nsDocShell::OnStateChange
[c:/builds/seamonkey/mozilla/docshell/base/nsDocShell.cpp line 4175]
    nsDocLoaderImpl::FireOnStateChange
[c:/builds/seamonkey/mozilla/uriloader/base/nsDocLoader.cpp line 1213]
    nsDocLoaderImpl::doStopDocumentLoad
[c:/builds/seamonkey/mozilla/uriloader/base/nsDocLoader.cpp line 867]
    nsDocLoaderImpl::DocLoaderIsEmpty
[c:/builds/seamonkey/mozilla/uriloader/base/nsDocLoader.cpp line 765]
    nsDocLoaderImpl::OnStopRequest
[c:/builds/seamonkey/mozilla/uriloader/base/nsDocLoader.cpp line 695]
    nsLoadGroup::RemoveRequest
[c:/builds/seamonkey/mozilla/netwerk/base/src/nsLoadGroup.cpp line 703]
    nsCachedChromeChannel::HandleStopLoadEvent
[c:/builds/seamonkey/mozilla/rdf/chrome/src/nsChromeProtocolHandler.cpp line 481]
    PL_HandleEvent [c:/builds/seamonkey/mozilla/xpcom/threads/plevent.c line 664]
    PL_ProcessPendingEvents [c:/builds/seamonkey/mozilla/xpcom/threads/plevent.c
line 597]
    _md_EventReceiverProc [c:/builds/seamonkey/mozilla/xpcom/threads/plevent.c
line 1386]
    USER32.dll + 0x4455 (0x77d44455)
    USER32.dll + 0x4d58 (0x77d44d58)
    nsContentTreeOwner::ShowAsModal
[c:/builds/seamonkey/mozilla/xpfe/appshell/src/nsContentTreeOwner.cpp line 447]
    nsWindowWatcher::OpenWindowJS
[c:/builds/seamonkey/mozilla/embedding/components/windowwatcher/src/nsWindowWatcher.cpp
line 791]
    GlobalWindowImpl::OpenInternal
[c:/builds/seamonkey/mozilla/dom/src/base/nsGlobalWindow.cpp line 4386]
    GlobalWindowImpl::OpenDialog
[c:/builds/seamonkey/mozilla/dom/src/base/nsGlobalWindow.cpp line 3070]
    XPTC_InvokeByIndex
[c:/builds/seamonkey/mozilla/xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp
line 102]
    XPCWrappedNative::CallMethod
[c:/builds/seamonkey/mozilla/js/src/xpconnect/src/xpcwrappednative.cpp line 2025]
    XPC_WN_CallMethod
[c:/builds/seamonkey/mozilla/js/src/xpconnect/src/xpcwrappednativejsops.cpp line
1293]
    js_Invoke [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 841]
    js_Interpret [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 2804]
    js_Invoke [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 857]
    nsXPCWrappedJSClass::CallMethod
[c:/builds/seamonkey/mozilla/js/src/xpconnect/src/xpcwrappedjsclass.cpp line 1199]
    nsXPCWrappedJS::CallMethod
[c:/builds/seamonkey/mozilla/js/src/xpconnect/src/xpcwrappedjs.cpp line 429]
    PrepareAndDispatch
[c:/builds/seamonkey/mozilla/xpcom/reflect/xptcall/src/md/win32/xptcstubs.cpp
line 117]
    SharedStub
[c:/builds/seamonkey/mozilla/xpcom/reflect/xptcall/src/md/win32/xptcstubs.cpp
line 139]
    XPTC_InvokeByIndex
[c:/builds/seamonkey/mozilla/xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp
line 102]
    XPCWrappedNative::CallMethod
[c:/builds/seamonkey/mozilla/js/src/xpconnect/src/xpcwrappednative.cpp line 2025]
    XPC_WN_CallMethod
[c:/builds/seamonkey/mozilla/js/src/xpconnect/src/xpcwrappednativejsops.cpp line
1293]
    js_Invoke [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 841]
    js_Interpret [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 2804]
    js_Invoke [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 857]
    js_InternalInvoke [c:/builds/seamonkey/mozilla/js/src/jsinterp.c line 932]
    JS_CallFunctionValue [c:/builds/seamonkey/mozilla/js/src/jsapi.c line 3433]
    nsJSContext::CallEventHandler
[c:/builds/seamonkey/mozilla/dom/src/base/nsJSEnvironment.cpp line 1043]
    nsJSEventListener::HandleEvent
[c:/builds/seamonkey/mozilla/dom/src/events/nsJSEventListener.cpp line 183]
    nsEventListenerManager::HandleEventSubType
[c:/builds/seamonkey/mozilla/content/events/src/nsEventListenerManager.cpp line
1217]
    nsEventListenerManager::HandleEvent
[c:/builds/seamonkey/mozilla/content/events/src/nsEventListenerManager.cpp line
2218]
    nsXULElement::HandleDOMEvent
[c:/builds/seamonkey/mozilla/content/xul/content/src/nsXULElement.cpp line 3361]
    PresShell::HandleDOMEventWithTarget
[c:/builds/seamonkey/mozilla/layout/html/base/src/nsPresShell.cpp line 6240]
    nsButtonBoxFrame::MouseClicked
[c:/builds/seamonkey/mozilla/layout/xul/base/src/nsButtonBoxFrame.cpp line 197]
    nsButtonBoxFrame::HandleEvent
[c:/builds/seamonkey/mozilla/layout/xul/base/src/nsButtonBoxFrame.cpp line 147]
    PresShell::HandleEventInternal
[c:/builds/seamonkey/mozilla/layout/html/base/src/nsPresShell.cpp line 6209]

    Source File :
c:/builds/seamonkey/mozilla/editor/txtsvc/src/nsTextServicesDocument.cpp line : 461

    16277674 OS: Windows NT 5.1 build 2600
    16277604 OS: Windows NT 5.1 build 2600
    16276442 OS: Windows NT 5.1 build 2600
    16276421 OS: Windows NT 5.1 build 2600
    16276407 OS: Windows NT 5.1 build 2600
    16276394 OS: Windows NT 5.1 build 2600
    16276335 OS: Windows NT 5.0 build 2195
    16276185 OS: Windows NT 5.0 build 2195
    16266447 OS: Windows NT 5.0 build 2195
    16264709 OS: Windows NT 5.1 build 2600
    16263915 OS: Windows NT 5.0 build 2195
    16263686 OS: Windows NT 5.0 build 2195
    16263608 OS: Windows 98 4.10 build 67766446
    16263330 OS: Windows NT 5.0 build 2195

Comment 1

[Joe Francis]

Kathy, do you think this is you?  If not assign it back to me.

Comment 2

[timeless]

perhaps someone changed the spellchecker interface?

jpatel: do we know whose spellchk.dll this is?

Comment 3

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

(See also bug 185251.)

Comment 4

[David Bradley]

Almost certainly this is due to the added method to the interface, and thus a
dupe of bug 185251.

For future reference, if possible try and add new methods to the end of the
interface. This allows old code to work with updated interfaces. Not sure if
it's worth moving it, since that will creat it's own set of problems.

People who have the spellchk.dll will need a version compiled with the new
version of this interface for it work properly.

*** This bug has been marked as a duplicate of 185251 ***

Comment 5

[Jay Patel [:jay]]

verifying invalid.  i still think we should somehow inform people about
recompiling the spellchk.dll with the new interface to avoid these crashes.