Last Comment Bug 703882 - crash nsExternalAppHandler::nsExternalAppHandler
: crash nsExternalAppHandler::nsExternalAppHandler
Status: RESOLVED FIXED
: crash
Product: Core
Classification: Components
Component: Document Navigation (show other bugs)
: Trunk
: x86 Windows 7
: -- critical (vote)
: mozilla12
Assigned To: Makoto Kato [:m_kato]
:
: Andrew Overholt [:overholt]
Mentors:
: 549257 674007 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-11-19 07:44 PST by Wayne Mery (:wsmwk, NI for questions)
Modified: 2012-02-21 20:19 PST (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
fix (14.28 KB, patch)
2012-01-11 02:47 PST, Makoto Kato [:m_kato]
bzbarsky: review+
Details | Diff | Splinter Review

Description Wayne Mery (:wsmwk, NI for questions) 2011-11-19 07:44:11 PST
second highest crash with MIME in signature

This bug was filed from the Socorro interface and is 
report bp-67669cdf-2b62-4bd1-a56f-df6922111110 .
=============================================================

0	xul.dll	nsExternalAppHandler::nsExternalAppHandler	uriloader/exthandler/nsExternalHelperAppService.cpp:1256
1	xul.dll	nsExternalHelperAppService::DoContent	uriloader/exthandler/nsExternalHelperAppService.cpp:853
2	xul.dll	nsDocumentOpenInfo::DispatchContent	uriloader/base/nsURILoader.cpp:614
3	xul.dll	nsDocumentOpenInfo::OnStartRequest	uriloader/base/nsURILoader.cpp:294
4	xul.dll	nsStreamConverter::FirePendingStartRequest	mailnews/mime/src/nsStreamConverter.cpp:1187
5	xul.dll	mime_output_fn	mailnews/mime/src/mimemoz2.cpp:971
6	xul.dll	MimeOptions_write	mailnews/mime/src/mimei.cpp:1758
7	xul.dll	MimeObject_write	mailnews/mime/src/mimei.cpp:1793
8	xul.dll	MimeExternalObject_parse_decoded_buffer	mailnews/mime/src/mimeeobj.cpp:252
9	xul.dll	mime_decode_base64_buffer	mailnews/mime/src/mimeenc.cpp:333
10	xul.dll	MimeDecoderWrite	mailnews/mime/src/mimeenc.cpp:854
11	xul.dll	MimeLeaf_parse_buffer	mailnews/mime/src/mimeleaf.cpp:179
Comment 1 Makoto Kato [:m_kato] 2012-01-10 20:32:19 PST
gExtProtSvc shouldn't have global.  This pointer is clear by destructor of nsExternalHelperAppService.  But nsExternalAppHandler may still use it.
Comment 2 Makoto Kato [:m_kato] 2012-01-11 02:47:52 PST
Created attachment 587639 [details] [diff] [review]
fix
Comment 3 Boris Zbarsky [:bz] (still a bit busy) 2012-01-12 13:47:27 PST
Comment on attachment 587639 [details] [diff] [review]
fix

r=me
Comment 5 Justin Wood (:Callek) 2012-01-16 19:37:30 PST
https://hg.mozilla.org/mozilla-central/rev/fb92bdd1ab2d
Comment 6 Makoto Kato [:m_kato] 2012-02-21 20:16:41 PST
*** Bug 674007 has been marked as a duplicate of this bug. ***
Comment 7 Makoto Kato [:m_kato] 2012-02-21 20:19:34 PST
*** Bug 549257 has been marked as a duplicate of this bug. ***

Note You need to log in before you can comment on or make changes to this bug.