51415 - View stored passwords crashes psm on win2000,98

Status: VERIFIED WORKSFORME

(Core Graveyard :: Security: UI, defect, P2)

Description

[nitinp]

try to view the stored passwords with the latest windows build and psm1.3 build.
the moment you enter the db password , psm will crash. This happens on
windows2000 only . works for NT and linux.

Comment 1

[nitinp]

adding keyword nsbeta3 and nominating for nsbeta3+

Comment 2

[Terry Hayes]

Does this fail just for "view stored" or does it fail for normal password use 
like forms-fill and remembered passwords?

Can we get a stack trace?

Comment 3

[nitinp]

I get this crash for "view stored passwords" . It works sometimes , but if view
it again after a successful view it crashes psm.
the stack trace is

PSM! 004be7af()
PSM! 004be85c()
PSM! 00494ae6()
PSM! 0047e8e6()
PSM! 00476770()
PSM! 00414c99()
PSM! 00435ca0()
NSPR4! 30013131()
KERNEL32! 77e837cd()

Comment 4

[John Unruh]

Reproducible with the 9/11 commercial builds on Win2000 and Win98. WinNT and 
Linux are OK.
1.) With a new profile, enter info at mail.yahoo.com and say yes to save the 
data.
2.) Change the master password (create a password).
3.) Encrypt the data.
4.) View the stored data
What happens: crash in psm.exe

Comment 5

[nitinp]

still happens for the 09/18 10:43pm build.

Comment 6

[Blake Ross]

*** This bug has been marked as a duplicate of 53116 ***

Comment 7

[Terry Hayes]

Reopen - duplicate reference is incorrect.  There is a possible dup at 53197.

Comment 8

[Blake Ross]

Please look more closely.  53197 is a dup of 53116, and Dan comments in 53116 
that it's really a dup of 52970...and 52970 covers this screen as well.  It 
should all be the same problem, and it should be fixed now.

*** This bug has been marked as a duplicate of 53116 ***

Comment 9

[Terry Hayes]

I wish you would look more closely.  This bug refers to a crash in PSM, not in 
Mozilla itself.  This bug has a relatively short stack trace, where the your 
"dup" has an infinite recursion.  Just because the same screen is involved 
doesn't mean it's the same bug.

Since we've already fixed this, I'm not going to fight about it anymore, except 
to note (for the record) that the resolution of this is wrong.

Comment 10

[John Unruh]

Reopening. Still not fixed in the 9/22 Win98 and Win2000 builds.

Comment 11

[clayton]

PDT:  Not a ship stopper for PR3.  Marking P2, not a crash on high profile
functionality.  If our backs were up to the wall, could we ship RTM with this?

Comment 12

[Terry Hayes]

I'm surprised this isn't a "high profile" feature, given the amount of 
discussion about how to present encryption to the end user.

In any case, I believe this bug will be fixed with the latest drop of PSM.  We 
should retest this bug against the next build of SeaMonkey that includes that 
drop.

This bug is probably related to 49106

Comment 13

[nitinp]

The fix for this bug is the new drop of psm (9/22) which leaf will have to put 
in the netcape6 daily builds. ddrinan will talk to leaf and he will update the 
build scripts soon to get the new psm1.3.

Comment 14

[Phil Peterson]

Marking nsbeta3- since we wouldn't pull PR3 off the wire for this. We'll be 
happy to get the fix if/when we get a new PSM drop.

Comment 15

[John Unruh]

Worksforme with the 92808 branch builds on Win95,98,2000.

Comment 16

[John Unruh]

Verified.

Comment 17

[John Unruh]

Mass changing Security:Crypto to PSM

Comment 18

[John Unruh]

Mass changing Security:Crypto to PSM