empty autologin-user should not be passed to PAM

[Guido Berhoerster]

If autologin-user in lightdm.conf is set to an empty string this should not be passed on to PAM. At least with the pam_permit module from Linux PAM which is usually employed for password-less login on Linux setting the username to an empty string will result in a successful login as the "nobody" user. This is probably never desirable.
See the attached patch for a proposed fix.

[Sebastien Bacher (seb128)]

Thank you for your work there, Robert, Michael could you review the patch? Not sure if that would solve the liveCD case but seems similar

[Michael Terry (mterry)]

Looks fine to me. I put the patch into a bzr merge proposal for Robert's review ease. Thanks Guido!

[Launchpad Janitor (janitor)]

This bug was fixed in the package lightdm - 0.9.3-0ubuntu4

---------------
lightdm (0.9.3-0ubuntu4) oneiric; urgency=low

  * Updated to current trunk, that's a candidate version version for the next
    update, it fixes those issues:
    - login doesn't work for ecryptfs users (lp: #823775, #824594)
    - "lightdm-gtk-greeter segfaults in get_user_iter when adding a new user"
    (lp: #822470)
    - fix fallback from org.freedesktop.Accounts to passwd format (lp: #817835)
    - empty autologin-user should not be passed to pam (lp: #817581)
  * debian/control.in:
    - build-depends on quilt, it's needed with source v1
    - don't build-depends on valac, vala is not used in the current version
  * debian/lightdm.install:
    - install the manpages as well
  * debian/lightdm.manpages:
    - dropped, it's installed by the upstream make install
  * debian/rules:
    - use the quilt rule
  * debian/source/format:
    - use source v1, it works better with vcs workflows
 -- Sebastien Bacher <email address hidden> Thu, 18 Aug 2011 15:29:42 +0200