[Bug 1160288] Re: uninitialized value $3 / $curmod

[Launchpad Bug Tracker]

This bug was fixed in the package pam - 1.1.3-10ubuntu1

---------------
pam (1.1.3-10ubuntu1) trusty; urgency=low

  * Merge from Debian unstable, remaining changes:
    - debian/libpam-modules.postinst: Add PATH to /etc/environment if it's
      not present there or in /etc/security/pam_env.conf. (should send to
      Debian).
    - debian/libpam0g.postinst: only ask questions during update-manager when
      there are non-default services running.
    - debian/libpam0g.postinst: check if gdm is actually running before
      trying to reload it.
    - debian/libpam0g.postinst: the init script for 'samba' is now named
      'smbd' in Ubuntu, so fix the restart handling.
    - Change Vcs-Bzr to point at the Ubuntu branch.
    - debian/patches-applied/series: Ubuntu patches are as below ...
    - debian/patches-applied/ubuntu-rlimit_nice_correction: Explicitly
      initialise RLIMIT_NICE rather than relying on the kernel limits.
    - debian/patches-applied/pam_umask_usergroups_from_login.defs.patch:
      Deprecate pam_unix's explicit "usergroups" option and instead read it
      from /etc/login.def's "USERGROUP_ENAB" option if umask is only defined
      there. This restores compatibility with the pre-PAM behaviour of login.
    - debian/patches-applied/pam_motd-legal-notice: display the contents of
      /etc/legal once, then set a flag in the user's homedir to prevent
      showing it again.
    - debian/update-motd.5, debian/libpam-modules.manpages: add a manpage
      for update-motd, with some best practices and notes of explanation.
    - debian/patches/update-motd-manpage-ref: add a reference in pam_motd(8)
      to update-motd(5)
    - debian/local/common-session{,-noninteractive}: Enable pam_umask by
      default, now that the umask setting is gone from /etc/profile.
    - debian/local/pam-auth-update: Add the new md5sums for pam_umask addition.
    - Build-depend on libfl-dev in addition to flex, for cross-building
      support.
    - Add /usr/local/games to PATH.
    - Disable libaudit for stage1 bootstrap.
    - Adjust debian/patches-applied/update-motd to write to
      /run/motd.dynamic, as sysvinit/ssh/login in Debian have been changed
      to use this file and no longer links /etc/motd to /var/run/motd.

pam (1.1.3-10) unstable; urgency=low

  * Fix pam-auth-update handling of trailing blank lines in the fields of
    profiles.  LP: #1160288.
  * Reintroduce libaudit support now that libaudit has been multiarched.
    Closes: #699159.

pam (1.1.3-9) unstable; urgency=low

  * Revert libaudit support for now, because libaudit isn't multiarched yet
    in unstable so this regresses cross-installability.  Reopens bug
    #699159.
  * Add an or'ed dependency on cdebconf, which also implements the
    xloadtemplatefile extension that prevents us from depending on just
    'debconf-2.0'.  Thanks to Régis Boudin <regis at boudin.name> for the info.
    Closes: #677278.
 -- Steve Langasek <steve.langasek at ubuntu.com>   Sun, 20 Oct 2013 18:21:34 -0700

** Changed in: pam (Ubuntu)
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to pam in Ubuntu.
https://bugs.launchpad.net/bugs/1160288

Title:
  uninitialized value $3 / $curmod

Status in “pam” package in Ubuntu:
  Fix Released

Bug description:
  [impact]
  Call of pam-auth-update raises some warnings of uninitalized variables

  [test case]
  install sssd libpam-sss libnss-sss and run pam-auth-update

  [versions]
  Distributor ID: Ubuntu
  Description:    Ubuntu 12.04.2 LTS
  Release:        12.04
  Codename:       precise

  libpam-runtime
  1.1.3-7ubuntu2

  
  [possible solution]
  --- a/pam-auth-update   Tue Mar 26 11:05:37 2013 +0100
  +++ b/pam-auth-update   Tue Mar 26 11:05:56 2013 +0100
  @@ -620,8 +620,8 @@
                                  ($modname,$line) = @{$saved[0]};
                                  shift(@saved);
                                  $line =~ /^((\[[^]]+\]|\w+)\s+\S+)\s*(.*)/;
  -                               @prev_opts = split(/\s+/,$3);
  -                               $curmod = $1;
  +                               @prev_opts = split(/\s+/, (defined $3?$3:"") );
  +                               $curmod = (defined $1?$1:"");
                                  # FIXME: the key isn't derived from the config
                                  # name, so collisions are possible if more
                                  # than one config references the same module

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/pam/+bug/1160288/+subscriptions