479438 - Can't fetch SMIME certificate from LDAP in Compose and Message Security windows

Status: RESOLVED FIXED

(MailNews Core :: Security, defect)

Description

[Raphaël Fairise]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.6) Gecko/2009020911 Ubuntu/8.10 (intrepid) Firefox/3.0.6
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3pre) Gecko/20090218 Shredder/3.0b2pre

This bug is a similar LDAP issue but in another location, resolved in bug 473715.
nsILDAPURL.spec = url doesn't work as before.

[Exception... "Component returned failure code: 0xc1f30001 (NS_ERROR_NOT_INITIALIZED) [nsILDAPURL.spec]"  nsresult: "0xc1f30001 (NS_ERROR_NOT_INITIALIZED)"  location: "JS frame :: chrome://messenger-smime/content/certFetchingStatus.js :: search :: line 93"  data: no] exception creating ldap connection

Reproducible: Always

Comment 1

[Ludovic Hirlimann [:Usul]]

(In reply to comment #0)

> This bug is a similar LDAP issue but in another location, resolved in bug
> 473715.
> nsILDAPURL.spec = url doesn't work as before.

Raphaël could you give us the last version where this worked for you ?

Comment 2

[Raphaël Fairise]

Attachment: Patch to use nsIIOService to get a nsILDAPURL

Tested successfully in Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3pre)
Gecko/20090218 Shredder/3.0b2pre

Comment 3

[Mark Banner (:standard8)]

(In reply to comment #1)
> (In reply to comment #0)
> 
> > This bug is a similar LDAP issue but in another location, resolved in bug
> > 473715.
> > nsILDAPURL.spec = url doesn't work as before.
> 
> Raphaël could you give us the last version where this worked for you ?

That's easy, look at bug 473715... anyway in between b1 and b2.

Comment 4

[Mark Banner (:standard8)]

Comment on attachment 363307 [details] [diff] [review]
Patch to use nsIIOService to get a nsILDAPURL

I'll take the review on this.

Comment 5

[Bryan Clark (DevTools PM) [:clarkbw]]

While this bug was found during the testing of tb3b2 (bug 477222) I don't think we're going to re-spin the tb3b2 release for this fix.  Mark, let me know if you feel otherwise and we can look into it.

Comment 6

[Mark Banner (:standard8)]

Comment on attachment 363307 [details] [diff] [review]
Patch to use nsIIOService to get a nsILDAPURL

Thanks for the patch. I'll grant sr as well given its a small patch.

Comment 7

[Mark Banner (:standard8)]

Patch checked in: http://hg.mozilla.org/comm-central/rev/610cae7fc174