61149 - Possible race condition regarding static variable in singsign.cpp

Status: RESOLVED WONTFIX

(SeaMonkey :: Passwords & Permissions, defect)

Description

[Stephen P. Morse]

signsign.cpp uses a static variable si_UserHasBeenSelected to solve certain 
problems regarding the list of multiple users that is sometimes displayed when 
more than one login is saved for the same URL.  This could result in a race 
condition if two pages are loading simultaneously in two separate windows and 
both pages have login forms and there are more than one saved logins for each of 
the two pages.  It's probably very unlikely that this will ever happen, and 
certainly very difficult to give a set of instructions that will actually 
demonstrate the problem.

Comment 1

[Brendan Eich [:brendan]]

How do you know a bad race is unlikely?  A frameset with two frames, each
containing text widgets, where the content of the two frames results in the
right combination of TCP segment boundaries, or just tag open/close relations,
to sink content in an interleaved fashion, could do it.  Cc'ing layout peers.

Does the [z] cybercrud mean "I want to mark this bug WONTFIX but I'm scared of
being criticized"?  Maybe the right answer is to use the Priority field as it
was intended to be used.

/be

Comment 2

[Marcia Knous [:marcia]]

Netscape nav triage team: based on Steve Morse's pretriage recommendation, this 
is not a beta stopper.

Comment 3

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Reassigning to new module owner.

Comment 4

[Philip Chee]

Obsoleted by Bug 390025 (Move to LoginManager and remove wallet from SeaMonkey)
Not using said file any more.