Closed Bug 2568 Opened 26 years ago Closed 26 years ago

Crash when switching between test app 9 and 2

Categories

(Core :: DOM: HTML Parser, defect, P1)

PowerPC
All
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: davidm, Assigned: rickg)

Details

Mac 604e/300 sys 8.5 Steps Command 9 to start loading sample 9 Before the load finishes hit command 2 to load sample 2 crash in nsParser.cpp in ResumeParse since mDTD is nil. Reproduced 100% of the time for me.
Assignee: rickg → pierre
OS: All
Please take a look at this and let me know if you see it.
Assignee: pierre → rickg
I'm seeing it too. It crashes when ResumeParse is called from OnStopBinding. Reassigned to rickg since it is in the parser.
Status: NEW → ASSIGNED
Setting all current Open/Normal to M4.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Summary: Crash when switching between test app 9 and 2 → Crash when switching between test app 9 and 2
I was looking at this bug this weekend and here's what I found: nsHttpUrlImpl::GetContainer always returns with NS_OK even if the container is nil. This is bad because the function getNetSupport in nsStubContext.cpp is making a call to get the container, checking the result and then calling QueryInterface on the container. This is probably bad on all platforms but crashes the Mac. After discussing it with rpotts I've changed GetContainer to return NS_ERROR_UNEXPECTED if mContainer is nil which fixed the Mac crash.
QA Contact: 4082
Status: RESOLVED → VERIFIED
Verified fixed with Mac March 5 builds.
You need to log in before you can comment on or make changes to this bug.