Closed Bug 763395 Opened 9 years ago Closed 6 years ago

crash in nsHttpConnection::OnReadSegment @ nlsp with NetOp School

Categories

(Core :: Networking: HTTP, defect)

14 Branch
x86
Windows XP
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox39 + fixed
firefox40 + fixed
firefox41 --- fixed

People

(Reporter: scoobidiver, Assigned: dmajor)

References

Details

(Keywords: crash)

Crash Data

Attachments

(1 file)

It's #72 top browser crasher in 14.0b6.

Here are correlations per module:
nlsp.dll@0x17e40|EXCEPTION_ACCESS_VIOLATION_READ (32 crashes)
    100% (32/32) vs.   0% (48/23847) nlsp.dll (NetOp School 6.2.2009.258)

Signature 	nlsp.dll@0x17e40 More Reports Search
UUID	218f9a4f-10fa-404f-adb0-998912120609
Date Processed	2012-06-09 05:12:09
Uptime	387
Last Crash	6.5 minutes before submission
Install Age	2.5 hours since version was first installed.
Install Time	2012-06-09 02:41:31
Product	Firefox
Version	14.0
Build ID	20120605113340
Release Channel	beta
OS	Windows NT
OS Version	6.1.7601 Service Pack 1
Build Architecture	x86
Build Architecture Info	GenuineIntel family 6 model 30 stepping 5
Crash Reason	EXCEPTION_ACCESS_VIOLATION_READ
Crash Address	0x173c1000
User Comments	crash....help
App Notes 	
AdapterVendorID: 0x10de, AdapterDeviceID: 0x0e22, AdapterSubsysID: 00000000, AdapterDriverVersion: 8.17.12.7533
D2D? D2D+ DWrite? DWrite+ D3D10 Layers? D3D10 Layers+ 
EMCheckCompatibility	True
Total Virtual Memory	2147352576
Available Virtual Memory	1699102720
System Memory Use Percentage	33
Available Page File	5933989888
Available Physical Memory	2449227776

Frame 	Module 	Signature 	Source
0 	nlsp.dll 	nlsp.dll@0x17e40 	
1 	nlsp.dll 	nlsp.dll@0x16179 	
2 	ws2_32.dll 	send 	
3 	nspr4.dll 	SocketWrite 	nsprpub/pr/src/io/prsocket.c:701
4 	xul.dll 	nsSocketOutputStream::Write 	netwerk/base/src/nsSocketTransport2.cpp:587
5 	xul.dll 	nsHttpConnection::OnReadSegment 	netwerk/protocol/http/nsHttpConnection.cpp:1190
6 	xul.dll 	nsHttpPipeline::ReadFromPipe 	netwerk/protocol/http/nsHttpPipeline.cpp:682
7 	xul.dll 	nsPipeInputStream::ReadSegments 	xpcom/io/nsPipe3.cpp:799
8 	xul.dll 	nsHttpPipeline::ReadSegments 	netwerk/protocol/http/nsHttpPipeline.cpp:726
9 	xul.dll 	nsHttpConnection::OnSocketWritable 	netwerk/protocol/http/nsHttpConnection.cpp:1251
10 	xul.dll 	nsHttpConnection::OnOutputStreamReady 	netwerk/protocol/http/nsHttpConnection.cpp:1530
11 	xul.dll 	nsHttpConnection::Activate 	netwerk/protocol/http/nsHttpConnection.cpp:396
12 	xul.dll 	nsHttpConnectionMgr::DispatchTransaction 	netwerk/protocol/http/nsHttpConnectionMgr.cpp:1512
13 	xul.dll 	nsHttpConnectionMgr::TryDispatchTransaction 	netwerk/protocol/http/nsHttpConnectionMgr.cpp:1432
14 	xul.dll 	nsHttpConnectionMgr::ProcessNewTransaction 	netwerk/protocol/http/nsHttpConnectionMgr.cpp:1617
15 	xul.dll 	nsHttpConnectionMgr::OnMsgNewTransaction 	netwerk/protocol/http/nsHttpConnectionMgr.cpp:1781
16 	xul.dll 	nsHttpConnectionMgr::nsConnEvent::Run 	netwerk/protocol/http/nsHttpConnectionMgr.h:522
17 	xul.dll 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:656
18 	xul.dll 	NS_ProcessNextEvent_P 	obj-firefox/xpcom/build/nsThreadUtils.cpp:245
19 	xul.dll 	nsSocketTransportService::Run 	netwerk/base/src/nsSocketTransportService2.cpp:652
20 	xul.dll 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:656
21 	xul.dll 	nsThread::ThreadFunc 	xpcom/threads/nsThread.cpp:289
22 	nspr4.dll 	_PR_NativeRunThread 	nsprpub/pr/src/threads/combined/pruthr.c:426
23 	nspr4.dll 	pr_root 	nsprpub/pr/src/md/windows/w95thred.c:122
24 	msvcr100.dll 	_callthreadstartex 	f:\dd\vctools\crt_bld\self_x86\crt\src\threadex.c:314
25 	msvcr100.dll 	_threadstartex 	f:\dd\vctools\crt_bld\self_x86\crt\src\threadex.c:292
26 	kernel32.dll 	BaseThreadInitThunk 	
27 	ntdll.dll 	__RtlUserThreadStart 	
28 	ntdll.dll 	_RtlUserThreadStart

More reports at:
https://crash-stats.mozilla.com/report/list?signature=nlsp.dll%400x17e40
Depends on: 671468
I think this is a dup of 671468.. The pipeline changes in FF14 have changed the stack a little bit, but the underlying pattern is still the same. not a new issue.
(In reply to Patrick McManus [:mcmanus] from comment #1)
> I think this is a dup of 671468.. The pipeline changes in FF14 have changed
> the stack a little bit, but the underlying pattern is still the same. not a
> new issue.

There are different filters involved so I wouldn't say it's strictly a duplicate.
This is still around and is pretty high volume on 39 beta.

According to http://www.netop.com/classroom-management-software/products/netop-school.htm, this product was discontinued in 2012.
Crash Signature: [@ nlsp.dll@0x17e40] → [@ nlsp.dll@0x17e40] [@ nlsp.dll@0x18330 ]
Attached patch Block nlsp.dllSplinter Review
Assignee: nobody → dmajor
Attachment #8616149 - Flags: review?(benjamin)
Attachment #8616149 - Flags: review?(benjamin) → review+
Given the current tree state, I'll check back next week!
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/40affb6a994f
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
[Tracking Requested - why for this release]:
Given comment #3, requesting tracking for 39 and 40. This should be an easy fix to uplift.
Comment on attachment 8616149 [details] [diff] [review]
Block nlsp.dll

Approval Request Comment
[Feature/regressing bug #]: Old version of NetOp School
[User impact if declined]: crashes
[Describe test coverage new/current, TreeHerder]: none
[Risks and why]: low, the product is discontinued
[String/UUID change made/needed]: none
Attachment #8616149 - Flags: approval-mozilla-beta?
Attachment #8616149 - Flags: approval-mozilla-aurora?
Comment on attachment 8616149 [details] [diff] [review]
Block nlsp.dll

Approved for uplift to aurora and beta. Nice!
Attachment #8616149 - Flags: approval-mozilla-beta?
Attachment #8616149 - Flags: approval-mozilla-beta+
Attachment #8616149 - Flags: approval-mozilla-aurora?
Attachment #8616149 - Flags: approval-mozilla-aurora+
Tracked for 39 & 40, inline with the fix & uplift comments above.
You need to log in before you can comment on or make changes to this bug.