50312 - cannot import p12 certificate files

Status: VERIFIED FIXED

(Core :: XUL, defect, P2)

Description

[bill]

I have a bunch of .p12 files of certificates exported from previous builds of
COmmunicator.  I'd like to import those certs into N6, but the file dialog box I
get after pressing the "restore" button doesn't show any files in the
directory.  And I know they're in there.

How do you import older certificates into PSM/Mac?

Comment 1

[John Unruh]

Nominating for nsbeta3.

Comment 2

[Javier Delgadillo]

Who's the owner of file picker for the Mac?  This works on all the other 
platforms, so my hunch is there's been a regression in the file picker code for 
the Mac.

Comment 3

[Javier Delgadillo]

pinkerton:

Who else should be included on this bug?

Comment 4

[Mike Pinkerton (not reading bugmail)]

cc'ing simon.



my guess is that it's a regression as well, as I just added file filtering code. 

I'll take this bug, but I need to know how to repro it and where the code is.

Comment 5

[Javier Delgadillo]

Attachment: attaching a p12 blob to test against

Comment 6

[Javier Delgadillo]

To reproduce dowload the attached file and unzip it somewhere on your Mac.

With the Commercial build, click on the security icon.  When Security Advisor 
comes up, click on the "Certificates" tab at the top.  Click on the 
"Restore" button.  This will eventually bring up the file picker dialog.

Set a breakpoint at nsPSMUIHandlerImpl::PromptForFile 
(http://lxr.mozilla.org/seamonkey/source/extensions/psm-glue/src/nsPSMUICallback
s.cpp#137)

That's where we invoke the file picker.

Comment 7

[Mike Pinkerton (not reading bugmail)]

thanks. I also need to know the location of the code that calls the file picker 
that is failing.

Comment 8

[Mike Pinkerton (not reading bugmail)]

I checked in a fix for this, but didn't test the exact case reported here (I just 
tried the new extension filtering code in other dialogs and it appears to work 
dandy).

Try this on Monday's builds and see if it works. If not, please reopen and I'll 
build commercial and try it out.

Comment 9

[John Unruh]

I cannot verify this until the Security Manager can be opened.

Comment 10

[John Unruh]

Verifying this depends on bug 54207.

Comment 11

[John Unruh]

Verified fixed with the 10/29 MN6 build.