Closed Bug 242990 Opened 20 years ago Closed 20 years ago

Crash when LDAP query results content none UTF8 characters

Categories

(MailNews Core :: LDAP Integration, defect)

defect
Not set
critical

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: wind.li, Assigned: sspitzer)

References

Details

(Keywords: crash, fixed1.7)

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6) Gecko/20040113
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6) Gecko/20040113

in mozilla/directory/xpcom/base/src/nsLDAPMessage.cpp:516 we use
UTF8ToNewUnicode to convert the result strings to unicode, but the LDAP qeury
result may not be UTF8 chars, such as "Ulrich Rüße". If LDAP server return these
chars, mozilla will crash

Reproducible: Always
Steps to Reproduce:
Attachment #147958 - Flags: review?(sspitzer)
Attachment #147958 - Flags: superreview?(dmose)
Blocks: 213274
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment on attachment 147958 [details] [diff] [review]
If the result strings IsUTF8 use UTF8ToNewUnicode, otherwise use general ToNewUnicode

r+sr=dmose
Attachment #147958 - Flags: superreview?(dmose)
Attachment #147958 - Flags: superreview+
Attachment #147958 - Flags: review?(sspitzer)
Attachment #147958 - Flags: review+
Keywords: crash
Comment on attachment 147958 [details] [diff] [review]
If the result strings IsUTF8 use UTF8ToNewUnicode, otherwise use general ToNewUnicode

This fixed a crash bug, nice to have in 1.7.
Attachment #147958 - Flags: approval1.7?
fixed in trunk, still waiting for 1.7 branch approval.
Is there a crash point or stack for this so we can search for dups?
#0  0x4047421d in free () from /lib/libc.so.6
#1  0x400ae488 in PR_Free () from ./libnspr4.so
#2  0x406242bd in nsMemoryImpl::Free ()
   from /cws/cws/mozab04/mozilla/dist/bin/libxpcom.so
#3  0x08063633 in nsMemory::Free ()
#4  0x41657275 in NSGetModule ()
   from /cws/cws/mozab04/mozilla/dist/bin/components/libmozldap.so
#5  0x411d0770 in NSGetModule ()
   from /cws/cws/mozab04/mozilla/dist/bin/components/libaddrbook.so
#6  0x411d2526 in NSGetModule ()
   from /cws/cws/mozab04/mozilla/dist/bin/components/libaddrbook.so
#7  0x411d1217 in NSGetModule ()
   from /cws/cws/mozab04/mozilla/dist/bin/components/libaddrbook.so
#8  0x406314e5 in XPTC_InvokeByIndex ()
   from /cws/cws/mozab04/mozilla/dist/bin/libxpcom.so
#9  0x4061e628 in nsProxyObject::Post ()
   from /cws/cws/mozab04/mozilla/dist/bin/libxpcom.so
#10 0x406171da in PL_HandleEvent ()
   from /cws/cws/mozab04/mozilla/dist/bin/libxpcom.so
#11 0x40617111 in PL_ProcessPendingEvents ()
   from /cws/cws/mozab04/mozilla/dist/bin/libxpcom.so
#12 0x40618bab in nsEventQueueImpl::ProcessPendingEvents ()
   from /cws/cws/mozab04/mozilla/dist/bin/libxpcom.so
---Type <return> to continue, or q <return> to quit---
#13 0x40d6a71f in ?? ()
   from /cws/cws/mozab04/mozilla/dist/bin/components/libwidget_gtk.so
#14 0x0810ca98 in ?? ()
#15 0x4050dcc0 in __morecore () from /lib/libc.so.6
#16 0xbfffeec8 in ?? ()
#17 0x40d6a341 in ?? ()
   from /cws/cws/mozab04/mozilla/dist/bin/components/libwidget_gtk.so
#18 0x0810ca98 in ?? ()
#19 0x00000008 in ?? ()
#20 0x00000001 in ?? ()
#21 0x4047426b in free () from /lib/libc.so.6
Previous frame inner to this frame (corrupt stack?)
Comment on attachment 147958 [details] [diff] [review]
If the result strings IsUTF8 use UTF8ToNewUnicode, otherwise use general ToNewUnicode

a=chofmann for 1.7
Attachment #147958 - Flags: approval1.7? → approval1.7+
fixed on the 1.7 branch
Keywords: fixed1.7
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Product: MailNews → Core
Is this fix included in TBird-1.0? I just had it crash in libmozldap.so (under
Fedora Core 3) and I am wondering whether I am seeing this bug or a different
one (I was running a non-debug build, so I did not get a meaningfull stack
trace). Thank you!
Yes, this was in Thunderbird 1.0.
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: