Hang in blank entering secure site dialog after viewing cert for domain name mismatch

RESOLVED FIXED in mozilla1.8.1beta1

Status

Core Graveyard
Widget: Mac
--
critical
RESOLVED FIXED
11 years ago
7 years ago

People

(Reporter: tracy, Assigned: Mark Mentovai)

Tracking

({regression, verified1.8.1})

1.8 Branch
mozilla1.8.1beta1
PowerPC
All
regression, verified1.8.1
Bug Flags:
blocking1.8.1 +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

11 years ago
This regressed between 2006-05-08-03 and 2006-05-09-09 builds of Bon Echo on Mac (both ppc and intel are affected)

steps to reproduce:
-ensure security.warn_entering_secure is set to *true*
(this gets turned off by default after the first https site visited)
-go to the url given: https://verisign.com
dialog:
-Click View Certificate
View Cert Window:
-Click Close
next dialog:
-Click OK

tested results:
a blank dialog slides out... we're hung.  force quit. or sometimes able to actually Firefox | Quit but then crash.

expected results:
Entering Securing Site warning appears and is dismissable.

work-around is to ensure security.warn_entering_secure is set to *false*  or don't view the cert.

note: I tested the regression window builds on the trunk. they don't show this bug.
(Assignee)

Comment 1

11 years ago
Sounds like my event handling changes.  I'll investigate.
(Assignee)

Comment 2

11 years ago
Event handling changes seem to be the trigger - I also see this in the 1.5.0.4rc3+ event handling test build at http://jackassofalltrades.com/tmp/firefox-1504rc3-332579v8-337199v4.dmg .  I also see this on the trunk now, probably following bug 338153.
Blocks: 332579
(Assignee)

Comment 3

11 years ago
Looks like we're re-showing the last sheet on a kEventWindowActivated event oddly sent from inside HideSheetWindow.  I'm going to compare this to how we handled activation with WaitNextEvent.
Assignee: nobody → mark

Comment 4

11 years ago
Perhaps I'm seeing the same:

Mac blue G3, OSX 10.3.9.latest-updates

Bon Echo A3

http://usfed.org/   (a financial institution site)

- login: a half dozen numerals
- password: whatever-you-wish
- next screen: it tries to transfer data from the site,
a blank "sheet" pops down, Hangs!

Ff 1.5.0.3 & 1.5.0.4.RC4 work.

Eddie Maddox
(Assignee)

Comment 5

11 years ago
OK, I see the problem.  ShowSheetWindow is being called multiple times to attach a sheet to the same top-level window.  In this case, it's the same sheet being attached twice in a row.  The first attachment is happening in nsMacWindow::Show, the second is happening when reentering nsMacWindow::Show on a kEventWindowActivated event caused by the first ShowSheetWindow call after hiding a parent sheet.  The solution to this bug is to only call ShowSheetWindow to attach a sheet, not if Show is called for a sheet that's already attached.

I'm marking this bug as dependent on bug 312586, because the patch I'm about to post depends on the more robust sheet support that bug 312586 provides.  (Bug 312586 fixes some similar problems.)
Status: NEW → ASSIGNED
Component: Security → Widget: Mac
Depends on: 312586
Product: Firefox → Core
Target Milestone: --- → mozilla1.8.1beta1
Version: 2.0 Branch → 1.8 Branch
(Assignee)

Comment 6

11 years ago
*** Bug 339359 has been marked as a duplicate of this bug. ***
(Assignee)

Updated

11 years ago
No longer depends on: 312586
(Assignee)

Comment 7

11 years ago
Created attachment 223697 [details] [diff] [review]
Patch (requires 312586)

This patch was prepared against the trunk, which already has the patch to bug 312586.  I intend to take that patch on the 1.8 branch as well.
Attachment #223697 - Flags: review?(joshmoz)

Updated

11 years ago
Flags: blocking1.8.1+

Updated

11 years ago
Attachment #223697 - Flags: review?(joshmoz) → review+
(Assignee)

Comment 8

11 years ago
Checked in on trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

11 years ago
Comment on attachment 223697 [details] [diff] [review]
Patch (requires 312586)

Will consolidate with 312586 checkin on branch once it's proven.
Attachment #223697 - Flags: approval-branch-1.8.1?(mark)

Comment 10

11 years ago
Mark, was this the same behavior that was causing 338501?
(Assignee)

Comment 11

11 years ago
*** Bug 338501 has been marked as a duplicate of this bug. ***
(Assignee)

Comment 12

11 years ago
Yes, I'll do some branch checkins shortly that will fix all sorts of sheet problems.
(Assignee)

Comment 13

11 years ago
Created attachment 223816 [details] [diff] [review]
1.8 branch version (includes 312586)

Checked in on MOZILLA_1_8_BRANCH.
Attachment #223816 - Flags: approval-branch-1.8.1+
(Assignee)

Updated

11 years ago
Attachment #223697 - Flags: approval-branch-1.8.1?(mark)
(Assignee)

Updated

11 years ago
Keywords: fixed1.8
(Assignee)

Updated

11 years ago
Keywords: fixed1.8 → fixed1.8.1

Comment 14

11 years ago
Still broken, but different now.

Nightly: firefox-2.0a3.en-US.mac.dmg  31-May-2006 16:28   17M
On: Mac blue G3, OSX 10.3.9.all-updates

URL: usfed.org (Financial institution.)

After giving my acct # and PW,
the next screen Hangs "Transferring data from ...". (Same.)
No sheet drops down. (Different.)
Page is all white. (Same.)

Cannot make BonEcho Stop the transfer,
nor Quit except via Force Quit. (All the same.)


Thank you,
Eddie Maddox

Comment 15

11 years ago
Tested with Bon echo updated on 20060822 with Intel Mac OSX 10.4.7
No hang or crash observed. 

After following the steps in Description, Click OK, the Secure site warning is closed and Verisign webpage is downloaded and displayed.
(Reporter)

Updated

11 years ago
Keywords: fixed1.8.1 → verified1.8.1

Updated

7 years ago
Component: Widget: Mac → Widget: Mac
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.