950 - ASSERT in CAbstractCX via mkgeturl.c

Status: VERIFIED WONTFIX

(Core Graveyard :: Viewer App, defect, P2)

Description

[Bryce Harrington]

Access Violation crash while viewing CNN & Slashdot.  I had just closed
a window.  I have a lot of programs loaded into memory.

Call Stack:
-----------
CAbstractCX::IsFrameContext() line 185 + 3 bytes
GetFrame(MWContext_ * 0x00ba5710) line 112 + 24 bytes
FE_URLEcho(URL_Struct_ * 0x00c26230, int 0x00000001, MWContext_ * 0x00ba5710)
line 47 + 14 bytes
net_CallExitRoutine(void (URL_Struct_ *, int, MWContext_ *)* 0x00792cc0
il_netgeturldone(URL_Struct_ *, int, MWContext_ *), URL_Struct_ * 0x00c26230,
int 0x00000001, int 0x00000002, MWContext_ * 0x00ba5710) line 1082 + 17 bytes
NET_ProcessNet(PRFileDesc * 0x00000000, int 0x00000001) line 3472 + 39 bytes
net_process_slow_net_timer_callback(void * 0x00000000) line 216 + 9 bytes
wfe_ProcessTimeouts(unsigned long 0x0b07a289) line 303 + 12 bytes
FireTimeout(HWND__ * 0x006f031e, unsigned int 0x00000113, unsigned int
0x00000309, unsigned long 0x0b07a289) line 60 + 9 bytes
USER32! 77e7128c()
CNetscapeApp::Run() line 1675 + 8 bytes
AfxWinMain(HINSTANCE__ * 0x00400000, HINSTANCE__ * 0x00000000, char *
0x00142595, int 0x00000001) line 52 + 11 bytes
WinMain(HINSTANCE__ * 0x00400000, HINSTANCE__ * 0x00000000, char * 0x00142595,
int 0x00000001) line 33 + 21 bytes
WinMainCRTStartup() line 330 + 57 bytes

Cause:
------
Here is the assert that failed, on line 194 of cxabstra.h, a member
function of CAbstractCX:

	BOOL IsFrameContext() const {
/* >> */	ASSERT(m_cxType);
		switch (m_cxType) {
		case Window:
			return TRUE;

		default:
			return FALSE;
		}
	}

Tracing this back, an appropriate place to address this problem
might be net_CallExitRoutine, in mkgeturl.c:

#if defined(XP_WIN) || defined (XP_MAC) || defined (XP_OS2)
	FE_URLEcho(URL_s, status, window_id);
#endif /* XP_WIN/MAC/OS2 */

I guess you'd want to add an if statement to only call the FE_URLEcho
function if window_id is valid.  In this case, the window_id pointer
is okay (it equals 0x00ba5710), but its contents are all set to
0xdddddddd or some such.  I'm not sure how you'd construct the if
statement to address this...


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, PowerPoint, Netscape 4.06,
and a notepad-like editor.

Comment 1

[Gagan]

Windows... reassiging.

Comment 2

[Garrett Arch Blythe]

the assert is harmless in this situation.

Comment 3

[leger]

Old bug...these sites load now with latest builds.