Closed Bug 616300 Opened 9 years ago Closed 9 years ago

crash [@ nsPrintEngine::DoCommonPrint(int, nsIPrintSettings*, nsIWebProgressListener*)]

Categories

(Core :: Printing: Output, defect, critical)

x86
All
defect
Not set
critical

Tracking

()

RESOLVED FIXED

People

(Reporter: wsmwk, Assigned: timeless)

References

Details

(Keywords: crash, Whiteboard: [tbird crash])

Crash Data

Attachments

(1 file, 2 obsolete files)

crash [@ nsPrintEngine::DoCommonPrint(int, nsIPrintSettings*, nsIWebProgressListener*)]

bp-7436940f-cd56-48bd-8dca-f2c812101114 (laura)
Mac OS X
10.4.11 8S2167
x86
GenuineIntel family 6 model 14 stepping 8
EXC_BAD_ACCESS / KERN_PROTECTION_FAILURE
0x44
0	XUL	nsPrintEngine::DoCommonPrint	
1	XUL	nsPrintEngine::CommonPrint	layout/printing/nsPrintEngine.cpp:447
2	XUL	nsPrintEngine::Print	layout/printing/nsPrintEngine.cpp:755
3	XUL	DocumentViewerImpl::Print	layout/base/nsDocumentViewer.cpp:3746
4	XUL	nsGlobalWindow::Print	dom/base/nsGlobalWindow.cpp:4597 

bp-74e4dbbe-42f9-4e43-8ed2-9b0102101122
Windows NT
5.1.2600 Service Pack 2
x86
AuthenticAMD family 6 model 10 stepping 0
EXCEPTION_ACCESS_VIOLATION_READ
0x44
0	xul.dll	nsPrintEngine::DoCommonPrint	layout/printing/nsPrintEngine.cpp:629
1	xul.dll	nsPrintEngine::CommonPrint	layout/printing/nsPrintEngine.cpp:447
2	xul.dll	nsPrintEngine::Print	layout/printing/nsPrintEngine.cpp:755
3	xul.dll	DocumentViewerImpl::Print	layout/base/nsDocumentViewer.cpp:3746
4	xul.dll	nsGlobalWindow::Print	dom/base/nsGlobalWindow.cpp:4597 

bp-e43ee349-2093-4e0a-b623-999662101114 thunderbird
628        // The user might have changed shrink-to-fit in the print dialog, so update our copy of its state
629        mPrt->mPrintSettings->GetShrinkToFit(&mPrt->mShrinkToFit);
630      } else {
631        rv = NS_ERROR_GFX_NO_PRINTROMPTSERVICE;
632      }
633    } else {
634      // Call any code that requires a run of the event loop.
635      rv = mPrt->mPrintSettings->SetupSilentPrinting();

I believe mPrintSettings is null
Blocks: 193001
actually,
        rv = printPromptService->ShowPrintDialog(domWin, wbp,
                                                 mPrt->mPrintSettings);

shows a dialog which spins an event loop, which means that the assumptions about objects here cease to be valid.
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #494970 - Flags: review?(roc)
Attached patch compiling (obsolete) — Splinter Review
Attachment #494970 - Attachment is obsolete: true
Attachment #494971 - Flags: review?(roc)
Attachment #494970 - Flags: review?(roc)
sorry, i didn't realize qrefresh hadn't finished *sigh*
Attachment #494971 - Attachment is obsolete: true
Attachment #494972 - Flags: review?(roc)
Attachment #494972 - Flags: approval2.0?
Attachment #494971 - Flags: review?(roc)
Attachment #494972 - Flags: review?(roc)
Attachment #494972 - Flags: review+
Attachment #494972 - Flags: approval2.0?
Attachment #494972 - Flags: approval2.0+
Keywords: checkin-needed
Pushed http://hg.mozilla.org/mozilla-central/rev/916b65373ba2
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Crash Signature: [@ nsPrintEngine::DoCommonPrint(int, nsIPrintSettings*, nsIWebProgressListener*)]
You need to log in before you can comment on or make changes to this bug.