Thank you for your bug report. I think the line is correct, "user->password_mode" is the old mode and "mode" the new mode, that check does "if changing from mode_none to something else, then"
** Changed in: accountsservice (Ubuntu) Importance: Undecided => Low -- You received this bug notification because you are a member of DX Packages, which is subscribed to accountsservice in Ubuntu. Matching subscriptions: dx-packages https://bugs.launchpad.net/bugs/1302230 Title: error in accountsservice/user.c fails to remove user from nopasswdlogin when mode is set to none Status in “accountsservice” package in Ubuntu: Confirmed Bug description: /* Remember to remove user from nopasswdlogin group if we're switching away from no-password mode */ 2683 if (user->password_mode == PASSWORD_MODE_NONE) { argv[0] = "/usr/bin/gpasswd"; argv[1] = "-d"; argv[2] = user->user_name; argv[3] = "nopasswdlogin"; argv[4] = NULL; Line 2683 should be if (user->password_mode != PASSWORD_MODE_NONE) { See related: https://bugs.launchpad.net/ubuntu/+source/accountsservice/+bug/1169054 To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/accountsservice/+bug/1302230/+subscriptions -- Mailing list: https://launchpad.net/~dx-packages Post to : dx-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~dx-packages More help : https://help.launchpad.net/ListHelp
