Closed Bug 71399 Opened 24 years ago Closed 23 years ago

parser causes string assertion loading NYTimes

Categories

(Core :: DOM: HTML Parser, defect)

Product:
Core
Component:
DOM: HTML Parser
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: dbaron, Assigned: harishd)

References

(
URL
)

Details

When loading http://www.nytimes.com/ I see the assertion

###!!! ASSERTION: |First()| on an empty string: 'Length()>0', file
../../dist/include/string/nsAReadableString.h, line 630
###!!! Break: at file ../../dist/include/string/nsAReadableString.h, line 630

which is caused by nsHTMLTokenizer.cpp, line 583.  The code

        if((kForwardSlash==key.First()) && (0==text.Length())){

should probably be replaced by

        if (key.Length() && (kForwardSlash==key.First()) && (0==text.Length())){
ya I have fixed that in my tree.
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
FIXED.
*** Bug 71459 has been marked as a duplicate of this bug. ***
Verified on:
build: 2001-03-29-09-Mtrunk
Platform: Win NT

The url loads fine.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.