Ubuntu
pam package

pam-auth-update: errors on package removal when calling

Bug #260122 reported by Dustin Kirkland 
2
Affects Status Importance Assigned to Milestone
pam (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

I have a call to pam-auth-update --package in the ecryptfs-utils postrm script.

It does do the right thing (removing the ecryptfs pam configuration).

However, a bunch of noisy error messages get dumped...

(Reading database ... 24977 files and directories currently installed.)
Removing ecryptfs-utils ...
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Use of uninitialized value in numeric comparison (<=>) at /usr/sbin/pam-auth-update line 104.
Processing triggers for man-db ...

:-Dustin

Related branches

lp:ubuntu/karmic/pam
Revision history for this message
Steve Langasek (vorlon) wrote :

        # FIXME: we need an option that will permit us to remove configs in
        # prerm remove, to avoid having a broken config at any point.

USTL ;)

I'll fix this to make the code less whiny in the case of a reference to a non-existent option. In the meantime, please don't call pam-auth-update in the postrm - it should be called with this not-yet-implemented --remove option in the prerm remove case, so that the config is fixed up on disk /before/ the module is pulled out from under us.

Thanks for continuing to be a guinea pig for this :)

Changed in pam:
status: New → Triaged
Revision history for this message
Steve Langasek (vorlon) wrote :

Fixed in pam 1.0.1-3ubuntu3. Changelog follows:

pam (1.0.1-3ubuntu3) intrepid; urgency=low

  * s/pam-auth-config/pam-auth-update/ in the source, I can't seem to get
    this name consistent to save my life - I'm starting to think I named it
    wrong...
  * Fix the regex used when suppressing jump counts when reading the saved
    config, so that we don't clobber module options with numbers in them.
  * If the target doesn't already exist, don't try to copy it.
  * Filter the config list to exclude configs that no longer exist.
    LP: #260122.
  * Avoid unnecessary sort/grep in the case where we already have a sorted
    list.
  * Implement pam-auth-update --remove, for use in package prerms when called
    with "remove".

Changed in pam:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.