Closed
Bug 110771
Opened 23 years ago
Closed 23 years ago
assertion: NS_ENSURE_TRUE(unknownOther) failed: 'unknownOther', file nsStandardURL.cpp, line 1048
Categories
(Core :: Networking, defect)
Core
Networking
Tracking
()
VERIFIED
DUPLICATE
of bug 110685
People
(Reporter: Brade, Assigned: darin.moz)
Details
Attachments
(1 file)
1.09 KB,
patch
|
Details | Diff | Splinter Review |
assertion: NS_ENSURE_TRUE(unknownOther) failed: 'unknownOther', file
nsStandardURL.cpp, line 1048
I am seeing the above assertions every time I open a window (or a dialog comes
up). It is very annoying.
nsStandardURL::Equals appears to be called from nsDocShell::OnNewURI. A member
nsIURI there seems to be null and causing the assertion.
i wanted to file this bug yesterday but my browser kept on crashing for other
reasons. same problem on w2k cvs builds.
OS: Mac System 9.x → All
Hardware: Macintosh → All
Comment 3•23 years ago
|
||
see also bug 110686
This line is the culprit:
http://lxr.mozilla.org/seamonkey/source/docshell/base/nsDocShell.cpp#4731
Seems as though it should check that the current URI isn't nsnull before trying
to compare it against the incoming one.
This patch puts a pointer check in so Equals doesn't get called when there is
no current URI.
I've also moved the Equals test at the end so it doesn't run unless all the
other conditions are met.
Reviews?
Assignee | ||
Comment 6•23 years ago
|
||
*** This bug has been marked as a duplicate of 110685 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 7•23 years ago
|
||
bug 110685 also has a patch, but either look fine to me... the one in this one
is probably better... sr=darin on either.
but, i suspect this may point us at something more serious... is it OK for
mCurrentURI to be NULL?
Reporter | ||
Comment 8•23 years ago
|
||
verify dup; apologies about the dup filing
I did query bugzilla descriptions for "unknownother" but it didn't match
anything (but now it matches two); temporary glitch?
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•