Closed Bug 35862 Opened 25 years ago Closed 25 years ago

Categories

(SeaMonkey :: General, defect, P3)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: sspitzer, Assigned: rickg)

References

()

Details

NS_NOTREACHED("no mime type set"); in CParserContext.cpp is getting hit. NTDLL! 77f7629c() nsDebug::Assertion(const char * 0x01806198, const char * 0x100a3dd4, const char * 0x01806160, int 143) line 191 + 13 bytes nsDebug::NotReached(const char * 0x01806198, const char * 0x01806160, int 143) line 302 + 22 bytes CParserContext::SetMimeType(const nsString & {...}) line 143 + 21 bytes nsParser::OnStartRequest(nsParser * const 0x05b9b654, nsIChannel * 0x05c651d0, nsISupports * 0x00000000) line 1390 + 31 bytes nsDocumentOpenInfo::OnStartRequest(nsDocumentOpenInfo * const 0x05c65110, nsIChannel * 0x05c651d0, nsISupports * 0x00000000) line 257 + 34 bytes nsHTTPServerListener::FinishedResponseHeaders() line 958 + 48 bytes nsHTTPServerListener::OnDataAvailable(nsHTTPServerListener * const 0x05b8c9a0, nsIChannel * 0x05b8d824, nsISupports * 0x05c651d0, nsIInputStream * 0x05b8ec5c, unsigned int 0, unsigned int 0) line 343 + 8 bytes nsOnDataAvailableEvent::HandleEvent(nsOnDataAvailableEvent * const 0x05b88aa0) line 406 + 47 bytes nsStreamListenerEvent::HandlePLEvent(PLEvent * 0x05b8d230) line 97 + 12 bytes PL_HandleEvent(PLEvent * 0x05b8d230) line 563 + 10 bytes PL_ProcessPendingEvents(PLEventQueue * 0x026664f0) line 508 + 9 bytes _md_EventReceiverProc(HWND__ * 0x01a90116, unsigned int 49341, unsigned int 0, long 40264944) line 1018 + 9 bytes USER32! 77e71820() 026664f0()
Ok, I added that assertion last night (hopefully not erroneously) because I found a case where the content type was getting passed through as "" (empty string) and this caused the parser to bail later. Not having a recognizable content type seemed wrong.
I've removed the assertion. Thanks.
Status: NEW → ASSIGNED
Rick: When is it valid to call SetMimeType and ignore the value passed in? Shouldn't this case be handled explicitly. Leaving the hole in there is asking for problems.
The method is on the parsercontext, which (should only) be accessible from the parser. It sets the mimetype as part of the DTD detection process. No one else should ever call it (or even be able to call it).
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Sorry for the spam. New QA Contact for Browser General. Thanks for your help Joseph (good luck with the new job) and welcome aboard Doron Rosenberg
QA Contact: jelwell → doronr
warren or rick, can one of you verify this bug. Neither Doron or I are going to be able to. Thanks.
Works for Me Platform: PC OS: Windows 98 Mozilla Version: 2000100508 Marking as Verified
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.