314354 - Print from Message Window with .EML file => crash [@ nsMailboxService::FetchMessage]

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect)

Description

[Mike Cowperthwaite]

TB 1.5b2-1006, TB 1.6a1-1028, Win2K

Open any .EML file.  From the message window, select   File | Print   or
File | Print Preview.  Program crashes.

Talkback (Print Preview, 1.6a1): TB11226422G

Comment 1

[timeless]

Incident ID: 11226422
Stack Signature	nsMailboxService::FetchMessage b2c95563
Product ID	ThunderbirdTrunk
Build ID	2005102808
Trigger Time	2005-10-29 13:50:02.0
Platform	Win32
Operating System	Windows NT 5.0 build 2195
Module	thunderbird.exe + (004f2c53)
URL visited	
User Comments	Opened a .EML file from disk; selected File|Print Preview
Since Last Crash	1393 sec
Total Uptime	23921 sec
Trigger Reason	Access violation
Source File, Line No.	e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/mailnews/local/src/nsMailboxService.cpp, line 215
Stack Trace 	
nsMailboxService::FetchMessage  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/mailnews/local/src/nsMailboxService.cpp, line 215]
nsMailboxService::DisplayMessageForPrinting  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/mailnews/local/src/nsMailboxService.cpp, line 591]
nsMsgPrintEngine::FireThatLoadOperation  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/mailnews/base/src/nsMsgPrintEngine.cpp, line 553]
nsMsgPrintEngine::Observe  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/mailnews/base/src/nsMsgPrintEngine.cpp, line 887]
XPCWrappedNative::CallMethod  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/js/src/xpconnect/src/xpcwrappednative.cpp, line 2139]
XPC_WN_CallMethod  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/js/src/xpconnect/src/xpcwrappednativejsops.cpp, line 1444]
js_Invoke  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/js/src/jsinterp.c, line 1177]
js_Interpret  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/js/src/jsinterp.c, line 3507]
js_Invoke  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/js/src/jsinterp.c, line 1197]
js_InternalInvoke  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/js/src/jsinterp.c, line 1274]
JS_CallFunctionValue  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/js/src/jsapi.c, line 4163]
nsJSContext::CallEventHandler  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/dom/src/base/nsJSEnvironment.cpp, line 1422]
nsGlobalWindow::RunTimeout  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/dom/src/base/nsGlobalWindow.cpp, line 6212]
nsGlobalWindow::TimerCallback  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/dom/src/base/nsGlobalWindow.cpp, line 6570]
nsAppStartup::Run  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/toolkit/components/startup/src/nsAppStartup.cpp, line 162]
main  [e:/builds/tinderbox/thunderbird-trunk/WINNT_5.0_Depend/mozilla/mail/app/nsMailApp.cpp, line 62]
KERNEL32.dll + 0x28989 (0x7c598989)

Comment 2

[David :Bienvenu]

taking

Comment 3

[Hermann Schwab]

regressed between 
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.9a1) Gecko/2005091204 SeaMonkey/1.1a
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.9a1) Gecko/2005091706 SeaMonkey/1.1a

TB11279943H,TB11280751W,TB11281169E,TB11282587H

Comment 4

[Hermann Schwab]

wfm Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.9a1) Gecko/20050915 SeaMonkey/1.1a
BuildId 2005091504

http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=SeaMonkeyAll&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2005-09-15+03%3A00&maxdate=2005-09-17+06%3A00&cvsroot=%2Fcvsroot

may have been the fix for Bug 268746, add ability to reply/forward edit message as new for .eml files opened in stand-alone window:

http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=SeaMonkeyAll&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2005-09-16+08%3A00&maxdate=2005-09-16+08%3A20&cvsroot=%2Fcvsroot

Comment 5

[David :Bienvenu]

Attachment: proposed fix

this fixes the crash, and print/print preview both work with the change.

Comment 6

[David :Bienvenu]

I think the 1.8 branch has this bug too, so if we respin, I think we should take a fix for this.

Comment 7

[David :Bienvenu]

fixed on trunk.

Comment 8

[Scott MacGregor]

ride along candidate. No impact to Firefox.

Comment 9

[Scott MacGregor]

flag clean up.

Comment 10

[Hermann Schwab]

crash WIN98SE BuildID 2005103110  
WFM Tinderbox BuildID 2005103121
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.9a1) Gecko/20051031 SeaMonkey/1.1a