Password Manager removes incorrect entries when multiple are entered for the same domain/user




13 years ago
10 years ago


(Reporter: glen, Unassigned)


Firefox Tracking Flags

(Not tracked)




13 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv: Gecko/20060111 Firefox/
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv: Gecko/20060111 Firefox/

nsiPasswordManager::removeUser only takes domain and user as arguments, but it's possible to have multiple password manager entries with the same domain and user, but different passwords (and userFieldNames and passFieldNames) - so when deleting a user, the one that gets deleted is arbitrary.

Reproducible: Sometimes

Steps to Reproduce:
1. Go to
2. Enter 'one' as username, and 'one' as password
3. Go to
4. Enter 'one' as username, and 'two' as password

(note that this two forms have different username and password field names, so things should be remembered correctly)

5. Go to password manager, view passwords, show passwords
6. Select the entry with 'one' as the password and press 'remove'
7. Close the view passwords dialog, reopen it, show passwords.
Actual Results:  
The entry with 'one' as the password is still there, and 'two' has been removed instead.

Expected Results:  
Should have deleted the entry with 'one' as the password


13 years ago
Ever confirmed: true

Comment 1

12 years ago
This is still valid in
Mozilla/5.0 (Windows; U; Windows NT 6.0; it; rv: Gecko/20070515 Firefox/

Instead on Trunk only the first login is saved. the second one is ignored
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007070614 Minefield/3.0a7pre
This feels like a slightly different bug, that maybe should be created
WFM on current trunk. Also tried deleting in the reverse order.

The old password manager was bad about mapping strings to logins, and could get confused in these kinds of cases. The new password manager keeps the actual logins around and modifies them based on the selected index, so it's more robust about not confusing similar logins.
Last Resolved: 11 years ago
Resolution: --- → WORKSFORME


10 years ago
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.