[xlib] crash on cnn page nsFontXlibNormal::GetWidth(const PRUnichar* aString, PRUint32 aLength)

VERIFIED FIXED

Status

()

--
critical
VERIFIED FIXED
18 years ago
18 years ago

People

(Reporter: tomi.leppikangas, Assigned: timecop)

Tracking

({crash})

Trunk
x86
Linux
crash
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

18 years ago
Xlib port crashes on cnn page, maybe something to do with unicode fonts?

Crash line in nsFontXlibNormal::GetWidth is:
ENCODER_BUFFER_ALLOC_IF_NEEDED(p, mCharSetInfo->mConverter,
    aString, aLength, buf, sizeof(buf), bufLen);

print *mCharSetInfo;
*mCharSetInfo = {
   nsFontCharSetXlibInfo::mCharSet = (nil)  
   nsFontCharSetXlibInfo::Convert = 0xfd2f3ed8 =

&`libgfx_xlib.so`nsFontMetricsXlib.cpp`ISO10646Convert(nsFontCharSetXlibInfo*,XFontStruct*,const
 unsigned short*,int,char*,int) 
   nsFontCharSetXlibInfo::mSpecialUnderline = '\001' 
   nsFontCharSetXlibInfo::mMap = (nil)  
   nsFontCharSetXlibInfo::mConverter = (nil)  
   nsFontCharSetXlibInfo::mLangGroup = (nil) 
}

mConverter shoudn't be null?

Updated

18 years ago
Severity: normal → critical
Keywords: crash

Comment 1

18 years ago
Yeah, I've noticed this too, at Yahoo! Mail.
(Assignee)

Updated

18 years ago
Depends on: 88486
(Assignee)

Comment 2

18 years ago
tomi: looks like GTK people are having the same issue in 88486 and when that
gets checked in, this will be automaticaly fixed here, too :)
Status: NEW → ASSIGNED

Comment 3

18 years ago
The fix for bug 88486 is in. 

Kindly see if that fixes this bug as will.
(Reporter)

Comment 4

18 years ago
cnn page works for me, marking as fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 5

18 years ago
It works for me on Yahoo! Mail now, too.  Marking verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.