Status

Core Graveyard
Security: UI
P1
normal
VERIFIED DUPLICATE of bug 85451
17 years ago
a year ago

People

(Reporter: Bob Lord, Assigned: Javier Delgadillo)

Tracking

({relnote})

1.0 Branch
psm2.1
relnote

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: crasher)

Attachments

(2 attachments)

(Reporter)

Description

17 years ago
Talkback: TB32961255E

I crashed in the Cert Viewer.
(Reporter)

Updated

17 years ago
Priority: -- → P1
Whiteboard: crasher
Target Milestone: --- → 2.0

Comment 1

17 years ago
-> ddrinan
Assignee: ssaux → ddrinan

Comment 2

17 years ago
1. Open the Cert Manager
2. Select ABA.ECOM Root CA from the Authorities Tab
3. Click the 'View' button
4. Click the 'Details' Tab
5. Select 'Certificate Signature Value' under Certificate Fields
6. Select some of the 'Field Values' for 'Certificate Signature Value'
7. Select 'Certificate Signature Algorithm' 
(Reporter)

Comment 3

17 years ago
Walker's script does indeed crash for me on Win2K.

Comment 4

17 years ago
Re-assigning to Javi.
Assignee: ddrinan → javi

Comment 5

17 years ago
crahs on linux branch 0716200104.0.9.2

Comment 6

17 years ago
Changing platform to all.
OS: Windows 2000 → All
Hardware: PC → All

Comment 7

17 years ago
linux talkback is 32962260 and shows the same stack trace as on TB32961255E

(Assignee)

Comment 8

17 years ago
Created attachment 42417 [details] [diff] [review]
Patch to fix crash.

Comment 9

17 years ago
This isn't the right fix for this. Basically this is fixing the symptom, not the
problem. Generally this crash is triggered by putting style="overflow:auto" on a
<tree/>, which causes it to be put in a nsIScrollableFrame, thus causing the QI
to nsITreeFrame to fail. Adding null checks here will subtly break code, not fix
it.

The real fix is to just not use style="overflow:auto" on a <tree/> (generally
used to get scrollbars). Instead, use the attribute rows="X" where X is the
number of rows you want visible (min and max) in the tree. Currently this has a
(small) bug though where the initial height will be 0 if you begin with an empty
tree (<tree/>) and fill in its children through JS later (bug 88138).
(Reporter)

Updated

17 years ago
Keywords: relnote

Comment 10

17 years ago
Yes, why on earth is a tree being given a style of overflow: auto?  That is
nonsensical and meaningless for a tree widget.  I don't mind bulletproofing
crashes of course, but your tree is going to massively malfunction as long as
you've artificially imposed this extra scrolling frame between the tree's parent
and the tree itself.

In fact, we should add assertions in addition to the returning of the failure
codes when the tree frame is null.

Comment 11

17 years ago
target->2.1
Target Milestone: 2.0 → 2.1
(Assignee)

Comment 12

17 years ago
This seems like a dup of 85451

*** This bug has been marked as a duplicate of 85451 ***
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → DUPLICATE

Comment 13

17 years ago
Verified dupe, and the crash is gone, at least from the branch.
Status: RESOLVED → VERIFIED

Updated

13 years ago
Component: Security: UI → Security: UI
Product: PSM → Core

Updated

9 years ago
Version: psm2.0 → 1.0 Branch

Comment 14

4 years ago
Created attachment 822169 [details]
bluetooth.zip
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.