63893 - PSM crashes if FIPS-mode is enabled.

Status: VERIFIED FIXED

(Core :: Security: PSM, defect, P1)

Description

[Stephen Moehle]

PSM will crash if FIPS-mode (on the Advanced tab of the Personal Security
Manager) is enabled and then disabled.

To reproduce:
1) Go to the Modules section on the Advanced tab of the Personal Security
Manager dialog.
2) Click on the Enable button next to the "Enable FIPS-mode" text. The button
should now say Disable and the text say "Disable FIPS-mode".
3) Close the Personal Security Manager dialog.
4) psm.exe will crash in a couple of minutes.  Unfortunately since it is psm
crashing and not Mozilla, no Talkback incident is created.

In addition, if after step 2, but before psm.exe crashes, you click on the
Disable button, the button disapears and is replaced by the text: "Not
Implemented. The requested feature has not been implemented, or is not
functioning properly."

Tested with Mozilla 122805 on NT4.

Comment 1

[John Unruh]

Confirmed in the 12/31 commercial trunk WinNT build.

Comment 2

[John Unruh]

*** Bug 64308 has been marked as a duplicate of this bug. ***

Comment 3

[John Unruh]

Setting platforms to All.

Comment 4

[ratman]

using 2001011804 win95 mtrunk.  

psm crashes upon first browser action after closing psm after enabling fips
mode.  rectifying the crash then prevents psm from loading subsequently, until
the browser is restarted.  after a browser restart following this crash, loading
psm shows that fips is no longer enabled.

here's what norton crashguard picks up:
Windows 95 Version 4.00.1111
PSM caused an invalid page fault in
module PSM.EXE at 014f:00449c2f.
Registers:
EAX=00000000 CS=014f EIP=00449c2f EFLGS=00010202
EBX=00899510 SS=0157 ESP=012afb14 EBP=00000000
ECX=00000001 DS=0157 ESI=008a3bf0 FS=2b2f
EDX=00000008 ES=0157 EDI=00000000 GS=0000
Bytes at CS:EIP:
8b 00 50 e8 19 e5 ff ff 83 c4 04 c3 8b 48 28 8b 
Stack dump:
00449c8f 012afb50 008a6840 008a3410 008102a0 008a3410 004498dc 008a3bf0 00424b4a
008a3bf0 00899510 008a69a8 008a6900 012aff94 81650db8 00000000 


suggest keyword crash.  if there's a such thing as a psm blocker, this'd be it.

Comment 5

[John Unruh]

Adding crash keyword

Comment 6

[John Unruh]

Milestone 2.0

Comment 7

[Javier Delgadillo]

This should be fixed now.

Comment 8

[John Unruh]

Verified fixed.