678706 - Deadlock after adding cert override

Status: RESOLVED DUPLICATE of bug 678440

(Core :: Security: PSM, defect)

Description

[Kai Engert (:KaiE:)]

- fresh profile
- go to https://www.cacert.org
- "I understant the risks"
- add exception, confirm

The dialog closes.
The browser attempts to reload the page.

It stalls forever.
I can use "stop", I can enter another URL into the location bar, but it fails to load.

Note that after a restart of the application the page loads fine.
I believe this regression was caused by bug 674571.

Comment 1

[Luke Wagner [:luke]]

I tried several times to reproduce on OSX with a nightly and no luck.  Is it every time for you?  Also, as the patches were just the "obvious" translation from (previously unsafe) proxies, I would appreciate any advice you have on this patch since you certainly understand what's actually going on in the code much better than I do.

Comment 2

[Kai Engert (:KaiE:)]

Maybe it's Linux specific?
Happened to me with my own debug build.

If I backout your patch, I crash with the expected assertion (GetProxyForObject on wrapped JS not allowed).

I apply your patch, and I get your deadlock.


Could someone else running Linux please try to reproduce?

Comment 3

[Kai Engert (:KaiE:)]

I am no longer able to reproduce myself.
Sorry for the noise :(

Maybe it was the side effect of another patch I had in my tree.
I'll reopen if it reoccurs.

Comment 4

[Kai Engert (:KaiE:)]

Well, I don't know yet.
I tried the other potential culprit (my own work-in-progress-patch),
but I still cannot reproduce.

Maybe there is a random component to reproducing this bug.

I propose, let's try to find someone running Linux, who is able to reproduce this bug.

I'm changing to UNCONFIRMED and asking for QAWANTED.

Comment 5

[Brian Smith (:briansmith, :bsmith, use NEEDINFO?)]

Luke, try unchecking the "Permanently store this exception" checkbox in the exception window, when trying to reproduce. If you get a crash instead, see bug 678710.

Comment 6

[Virgil Dicu [:virgil] [QA]]

Mozilla/5.0 (X11; Linux i686; rv:9.0a1) Gecko/20110818 Firefox/9.0a1

Attempted to reproduce this issue several times using the steps in the description on the latest nightly. Worked fine for me.

Comment 7

[Luke Wagner [:luke]]

This reflects the backout for the 8/18 nightly:
http://hg.mozilla.org/mozilla-central/rev/5f0596a0b81e