After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 691265 - Hang when changing password if new password is old password
Hang when changing password if new password is old password
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: User Accounts
3.6.x
Other Linux
: Normal major
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
3.10
Depends on:
Blocks:
 
 
Reported: 2013-01-07 02:19 UTC by Michael Catanzaro
Modified: 2013-07-08 03:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
don't hang when calling passwd (1.06 KB, patch)
2013-01-16 17:57 UTC, Michael Catanzaro
none Details | Review

Description Michael Catanzaro 2013-01-07 02:19:53 UTC
GNOME Control Center hangs when changing a user's password if the new password is the same as the old password.

See http://git.gnome.org/browse/gnome-control-center/tree/panels/user-accounts/um-password-dialog.c?h=gnome-3-6#n243

We need a new conditional there:  else if (strcmp (password, old_password) == 0)
Comment 1 Michael Catanzaro 2013-01-16 17:57:29 UTC
Created attachment 233616 [details] [review]
don't hang when calling passwd

A longer tooltip might be more clear. This one was selected because it's already localized.
Comment 2 Michael Catanzaro 2013-02-16 16:37:25 UTC
Does the patch look OK?

The underlying problem is that passwd gives another prompt when new password equals old password (at least on openSUSE) which gnome-control-center does not expect:

michael@linux-14l9:~> passwd
Changing password for michael.
Old Password: 
New Password: 
You must choose a new password.
New Password: 
You must choose a new password.
New Password: 

Seems simplest to avoid this in gnome-control-center by making sure the password is actually changed.
Comment 3 Michael Catanzaro 2013-02-20 05:17:09 UTC
Actually, this is now kind of redundant with the recently-submitted pwquality changes as it should now reject passwords if they're the same as they were before.  (Still would be more robust to include the check, though.)
Comment 4 Michael Catanzaro 2013-07-08 03:14:42 UTC
(In reply to comment #3)
> Actually, this is now kind of redundant with the recently-submitted pwquality
> changes as it should now reject passwords if they're the same as they were
> before.  (Still would be more robust to include the check, though.)

It's just not possible to trigger this anymore for this reason.