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 786530 - cannot cope with incorrect current password under LDAP backend
cannot cope with incorrect current password under LDAP backend
Status: RESOLVED OBSOLETE
Product: gnome-control-center
Classification: Core
Component: User Accounts
3.14.x
Other Linux
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-08-19 23:44 UTC by Laurent Bigonville
Modified: 2019-06-05 11:17 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
handle_passwd_with_ldap.patch (1.09 KB, patch)
2017-08-19 23:44 UTC, Laurent Bigonville 		accepted-commit_now Details | Review

Description Laurent Bigonville 2017-08-19 23:44:58 UTC 
Created attachment 358001 [details] [review]
handle_passwd_with_ldap.patch

Hi,

In debian/ubuntu we have the attached patch for quite sometimes

Original bug report can be found at: https://bugs.launchpad.net/ubuntu/+source/gnome-control-center/+bug/607357

===

When changing a UNIX password, if you enter an incorrect current password passwd will print an error and exit. However when using LDAP, unless the password has length 0 passwd offers another chance.

Enter login(LDAP) password:
LDAP Password incorrect: try again
Enter login(LDAP) password:

gnome-about-me mistakes the second prompt for the prompt to enter a new password and assumes the initial password was correct. Proceeding through the dialog eventually results in gnome-about-me hanging since passwd exits either at an unexpected time or not at all.

The interaction completes as expected when the correct initial password is given.



===

I guess this patch could be merged upstream?
Comment 1 Ondrej Holy 2018-01-31 12:07:23 UTC 
Review of attachment 358001 [details] [review]:

Thanks. I do not have an environment to test, but it should not break anything, so let's go with it.
Comment 2 Ondrej Holy 2019-06-05 11:17:25 UTC 
Let's close this in favor of https://gitlab.gnome.org/GNOME/gnome-control-center/merge_requests/483.