If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

XP_ASSERT in PA_DropDocData pa_parse.c

VERIFIED FIXED

Status

MozillaClassic
P3
normal
VERIFIED FIXED
19 years ago
19 years ago

People

(Reporter: Bryce Harrington, Assigned: Chris Toshok)

Tracking

1998-09-04
x86
Windows NT

Details

(URL)

(Reporter)

Description

19 years ago
'Nother assert...  This one while viewing http://sagan.earthspace.net/~esr/ecsl/

Line 541 of pa_parse.c:

pa_DocData *
PA_DropDocData(NET_StreamClass *stream)
{
    pa_DocData *doc_data=(pa_DocData *)stream->data_object;
    XP_ASSERT(doc_data &&
/* >> */     (doc_data->stream_count >= 1));
    if (!doc_data)
	return NULL;
    doc_data->stream_count--;
    return pa_check_doc_data_count(stream);
}

doc_data equals NULL.  I've tried tracing it back, but I can't tell
how the NULL enters the picture.

Stack trace (trimmed):
----------------------
1.  XP_AssertAtLine(char * 0x00906cc8, int 0x00000213) line 2687
2.  PA_DropDocData(_NET_StreamClass * 0x0012fc54) line 531 + 78 bytes
3.  lo_FlushBlockage(MWContext_ * 0x00c5a4a0, lo_DocState_struct * 0x01c98190,
lo_DocState_struct * 0x01c98190) line 2742 + 9 bytes
4.  lo_set_image_info(MWContext_ * 0x00c5a4a0, long 0x000003da, long 0x00000070,
long 0x000000a4) line 5624 + 17 bytes
5.  lo_process_deferred_image_info(void * 0x00000000) line 5724 + 32 bytes
6.  wfe_ProcessTimeouts(unsigned long 0x01788359) line 303 + 12 bytes
7.  FireTimeout(HWND__ * 0x00040272, unsigned int 0x00000113, unsigned int
0x00000309, unsigned long 0x01788359) line 60 + 9 bytes
8.  USER32! 77e7128c()
9.  CNetscapeApp::Run() line 1675 + 8 bytes
<Snip>


System:
-------
I'm running WinNT4.0 SR3, on a 166MHz Dell Pentium with 64MB RAM and
200MB swap.  In addition to WinNT processes and mozilla, I am running
cdplayer.exe, MSACCESS, TASKMGR, MSDEV, CRT, and a notepad-like editor.

Updated

19 years ago
Assignee: gagan → toshok
Component: NetLib → Layout

Comment 1

19 years ago
Nisheeth, do you handle the parser as well? If not reassign.
(Assignee)

Comment 2

19 years ago
toshok@netscape.com => toshok@hungry.com

Updated

19 years ago
Status: NEW → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED

Comment 3

19 years ago
old code base

Updated

19 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.