Closed
Bug 824637
Opened 11 years ago
Closed 9 years ago
nsStorageStream crash with modified test_streamcopier.js
Categories
(Core :: XPCOM, defect)
Core
XPCOM
Tracking
()
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
firefox20 | - | --- |
People
(Reporter: WeirdAl, Unassigned)
Details
(Keywords: crash, testcase)
Crash Data
Attachments
(1 file)
2.92 KB,
patch
|
Details | Diff | Splinter Review |
Steps to reproduce: (1) Apply the testcase patch. (2) In your objdir with tests enabled: make -C netwerk/test/ SOLO_FILE="test_streamcopier.js" libs check-one ###!!! ASSERTION: index out of bounds: 'indx < segCount', file c:\repos\trunk\mozilla\xpcom\io\nsSegmentedBuffer.h, line 58
Reporter | ||
Updated•11 years ago
|
tracking-firefox20:
--- → ?
Updated•11 years ago
|
Attachment #695654 -
Attachment is patch: true
Reporter | ||
Comment 1•11 years ago
|
||
OK, I found something very interesting: the storage input stream isn't reading from the beginning of the storage stream.
Reporter | ||
Comment 2•11 years ago
|
||
Apparently it matters when I initialize the storage input stream: if I do so before there's any data in the output stream, then the crash happens.
Comment 3•11 years ago
|
||
When do we think this regressed? Are we certain this test represents a normal user scenario? If this is a longstanding issue it's unlikely we'd track for a specific release.
Keywords: regressionwindow-wanted
Reporter | ||
Comment 4•11 years ago
|
||
I'd be very surprised if this was a regression of any kind.
Updated•11 years ago
|
Crash Signature: xul!nsStorageInputStream::ReadSegments+0x00000000000000E3 (c:\repos\trunk\mozilla\xpcom\io\nsstoragestream.cpp, line 424)
xul!nsStorageInputStream::Read+0x000000000000001D (c:\repos\trunk\mozilla\xpcom\io\nsstoragestream.cpp line 397)
xul!nsScriptableIn… → [@ nsStorageInputStream::ReadSegments]
Comment 5•11 years ago
|
||
(In reply to Alex Vincent [:WeirdAl] from comment #4) > I'd be very surprised if this was a regression of any kind. Given that, if you think this should still be pursued asap, please nominate with more context as to why you think this should track for a specific release.
Comment 6•9 years ago
|
||
Does this still reproduce?
Flags: needinfo?(ajvincent)
Keywords: regressionwindow-wanted
Reporter | ||
Comment 7•9 years ago
|
||
No, it looks like it's working now. :-)
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(ajvincent)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•