dx-packages team mailing list archive

[Sebastien Bacher <seb128@xxxxxxxxxx>]

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