9052 - crash when starting apprunner (debug build June 29)

Status: VERIFIED FIXED

(Core :: XPConnect, defect, P3)

Description

[ppandit]

Built seamonkey on Windows NT 4.0
Debug build 6:10 p.m.
Crash when apprunner is started:

D:\mozilla\dist\WIN32_D.OBJ\bin>apprunner
nsComponentManager: Autoregistration begins. dir = D:\mozilla\dist\WIN32_D.OBJ\b
in\components
***  Libjar is being registered
nsFindComponent registration successful
nsFindComponent added to appshell component list
nsUnknownContentTypeHandler registration successful
nsUnknownContentTypeHandler added to appshell component list
nsStreamTransfer registration successful
nsStreamTransfer added to appshell component list
Registering the PrefsWindow
*** Registering html library
*** Register XPConnect
*** Register XPConnect test components
*** XPInstall is being registered
nsComponentManager: Autoregistration ends. dir = D:\mozilla\dist\WIN32_D.OBJ\bin
\components


Trace:
NTDLL! DbgBreakPoint@0 address 0x77f76148
nsDebug::Break(const char * 0x01045714
??_C@_0CG@HCJI@?4?4?2?4?4?2dist?2include?2nsISupportsUt@, int 743) line 134
nsDebug::PreCondition(const char * 0x010456f4
??_C@_0P@IFHP@null?5parameter?$AA@, const char * 0x01045708
??_C@_07FDKL@aSource?$AA@, const char * 0x01045714
??_C@_0CG@HCJI@?4?4?2?4?4?2dist?2include?2nsISupportsUt@, int 743) line 143 + 13
bytes
CallQueryInterface(nsISupports * 0x00000000, nsISupportsArray * * 0x0012fbe4)
line 743 + 32 bytes
nsCOMPtr<nsISupportsArray>::Assert_NoQueryNeeded() line 414 + 15 bytes
nsCOMPtr<nsISupportsArray>::operator=(nsISupportsArray * 0x00000000) line 464
nsDocLoaderImpl::nsDocLoaderImpl() line 416
nsDocLoaderImpl::Create(nsISupports * 0x00000000, const nsID & {...}, void * *
0x0012fcb0) line 2126 + 27 bytes
nsGenericFactory::CreateInstance(nsGenericFactory * const 0x00c550c0,
nsISupports * 0x00000000, const nsID & {...}, void * * 0x0012fcb0) line 54
nsComponentManagerImpl::CreateInstance(nsComponentManagerImpl * const
0x00bd45c0, const nsID & {...}, nsISupports * 0x00000000, const nsID & {...},
void * * 0x0012fcb0) line 1125 + 24 bytes
nsComponentManager::CreateInstance(const nsID & {...}, nsISupports * 0x00000000,
const nsID & {...}, void * * 0x0012fcb0) line 68
nsServiceManagerImpl::GetService(nsServiceManagerImpl * const 0x00bd4760, const
nsID & {...}, const nsID & {...}, nsISupports * * 0x00c55e18,
nsIShutdownListener * 0x00000000) line 243 + 19 bytes
nsServiceManager::GetService(const nsID & {...}, const nsID & {...}, nsISupports
* * 0x00c55e18, nsIShutdownListener * 0x00000000) line 446
nsPICS::Init() line 530 + 25 bytes
nsPICS::nsPICS() line 400
nsPICS::GetPICS(nsIPICS * * 0x0012fd98) line 440 + 27 bytes
NS_NewPICS(nsIPICS * * 0x0012fd98) line 377 + 9 bytes
nsPICSFactory::CreateInstance(nsPICSFactory * const 0x00c55e60, nsISupports *
0x00000000, const nsID & {...}, void * * 0x0012fe10) line 1592 + 9 bytes
nsComponentManagerImpl::CreateInstance(nsComponentManagerImpl * const
0x00bd45c0, const nsID & {...}, nsISupports * 0x00000000, const nsID & {...},
void * * 0x0012fe10) line 1125 + 24 bytes
nsComponentManager::CreateInstance(const nsID & {...}, nsISupports * 0x00000000,
const nsID & {...}, void * * 0x0012fe10) line 68
nsAppShellService::Initialize(nsAppShellService * const 0x00c51970,
nsICmdLineService * 0x00c08b90) line 213 + 22 bytes
main(int 1, char * * 0x00bd14b0) line 511 + 16 bytes
mainCRTStartup() line 338 + 17 bytes
KERNEL32! BaseProcessStart@4 + 64 bytes

Comment 1

[lchiang]

marking blocker for QA testing.

Comment 2

[John Bandhauer]

I can't see anything in the stack trace that has anything to do with xpconnect.
It looks like nsDocLoaderImpl::nsDocLoaderImpl() that is getting into trouble.
Based on cvsblame, I'm reassigning to warren. This code is in flux. Are you sure
it is not already fixed?

Comment 3

[John Bandhauer]

It looks to me like Assert_NoQueryNeeded in nsCOMPtr.h needs to return
immediately if mRawPtr is null. I'll check in a fix for now. scc can change it
if it bothers him or close this bug.

Comment 4

[ppandit]

Was able to build on 7/1 and apprunner started without crashing. Assuming this
is fixed. Change status to FIXED.

Comment 5

[ppandit]

VERIFIED since this particular crash no longer occurs