590584 - Firefox 4.0b4 Crash Report [@ pk11_GetNewSession ]

Status: NEW

(NSS :: Libraries, defect, P5)

Description

[Carsten Book [:Tomcat]]

Crash Report for Firefox 4 Beta 4 and earlier - http://crash-stats.mozilla.com/report/list?signature=pk11_GetNewSession

Windows Crash only

Frame  	Module  	Signature [Expand]  	Source
0 	nss3.dll 	pk11_GetNewSession 	security/nss/lib/pk11wrap/pk11obj.c:372
1 	nss3.dll 	pk11_getKeyFromList 	security/nss/lib/pk11wrap/pk11skey.c:131
2 	nss3.dll 	pk11_CreateSymKey 	security/nss/lib/pk11wrap/pk11skey.c:178
3 	nss3.dll 	pk11_ImportSymKeyWithTempl 	security/nss/lib/pk11wrap/pk11skey.c:445
4 	nss3.dll 	PK11_ImportSymKey 	security/nss/lib/pk11wrap/pk11skey.c:496
5 	xul.dll 	nsKeyObjectFactory::KeyFromString 	security/manager/ssl/src/nsKeyModule.cpp:217
6 	xul.dll 	nsUrlClassifierDBServiceWorker::BeginStream 	
7 	xul.dll 	NS_InvokeByIndex_P 	xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp:102
8 	xul.dll 	nsProxyObjectCallInfo::Run 	xpcom/proxy/src/nsProxyEvent.cpp:181
9 	xul.dll 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:547
10 	xul.dll 	nsThread::ThreadFunc 	xpcom/threads/nsThread.cpp:263
11 	nspr4.dll 	_PR_NativeRunThread 	nsprpub/pr/src/threads/combined/pruthr.c:426
12 	nspr4.dll 	pr_root 	nsprpub/pr/src/md/windows/w95thred.c:122
13 	mozcrt19.dll 	_callthreadstartex 	obj-firefox/memory/jemalloc/crtsrc/threadex.c:348
14 	mozcrt19.dll 	_threadstartex 	obj-firefox/memory/jemalloc/crtsrc/threadex.c:326

Comment 1

[timeless]

Crash Address	0x30

366 pk11_GetNewSession(PK11SlotInfo *slot,PRBool *owner)
367 {
368   CK_SESSION_HANDLE session;
369   *owner = PR_TRUE;
370   if (!slot->isThreadSafe) PK11_EnterSlotMonitor(slot); 

this is roughly slot=0, but i'd have expected the code to crash earlier when it called PK11_ReferenceSlot:

278 context->slot = PK11_ReferenceSlot(slot); <- why didn't it crash here?
279 context->session = pk11_GetNewSession(slot,&context->ownSession);

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

Since the crash volume is low (less than 5 per week), the severity is downgraded to S3. Feel free to change it back if you think the bug is still critical.

For more information, please visit auto_nag documentation.