Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 22942 - pam version is outdated
Summary: pam version is outdated
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-06-16 12:44 UTC by Daniel Ahlberg (RETIRED)
Modified: 2004-05-02 12:19 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
pam-0.77-sigchld.patch (pam-0.77-sigchld.patch,950 bytes, patch)
2003-07-21 21:10 UTC, Andy Dustman
Details | Diff
Corrected sigchld patch from current gentoo patches (pam-0.75-sigchld.patch,2.06 KB, patch)
2003-07-24 17:06 UTC, Andy Dustman
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Ahlberg (RETIRED) gentoo-dev 2003-06-16 12:44:23 UTC
Available PAM version in portage is 0.75. PAM 0.77 was released September 23rd 2002.
Comment 1 Donny Davies (RETIRED) gentoo-dev 2003-06-17 09:29:27 UTC
I cannot handle this, its to big/complex with many patches.

The only person I feel comfortable handling this is Azarah --
and Im not certain that we're really 'outdated' because
the pam is basically a totally patched up affair.  Im
afraid Azarah is the authority on this one.
Comment 2 Andy Dustman 2003-07-21 21:10:05 UTC
Created attachment 14851 [details, diff]
pam-0.77-sigchld.patch

Found this on LKML:
http://marc.theaimsgroup.com/?l=linux-kernel&m=105838976602659&w=2

Starting with late 2.5.7x kernel, and continuing with 2.6.0-test1, if you lock
your screen with xscreensaver, after you try to unlock, xscreensaver hangs. I
found that by attaching strace to the process and then quitting, it would
continue on normally. It turns out to have something to do with signal
handling.

In any case, it would be good to get this patch into at least 0.75, because
switching VTs to unlock your screen is tiresome.

It's a pretty trivial patch, so I can't imagine it would be too hard to apply
to 0.75, but I haven't tried.
Comment 3 Andy Dustman 2003-07-24 17:06:24 UTC
Created attachment 14973 [details, diff]
Corrected sigchld patch from current gentoo patches

Upon further inspection, it appears the ebuild is already applying a sigchld
patch. The patch saves the SIGCHLD signal handler. However, it sets the handler
to  SIG_IGN instead of SIG_DFL, which is apparently the correct way. I modified
pam-0.75-sigchld.patch from pam-0.75-r11.gentoo.tbz2 to do this, which is
attached. It fixes the xscreensaver problem and does not appear to break
anything else.
Comment 4 Andy Dustman 2003-08-04 10:07:43 UTC
I've been running the above patch for about a week and it works fine.
Comment 5 Andy Dustman 2003-10-06 10:12:26 UTC
I've been running this patch for 2 1/2 mo now and it works, plus it is "obviously"
correct, i.e. the current signal handling in pam-0.75-r11 is buggy. Azarah,
can you give these patches (and their comments) another look?
Comment 6 Dizzy 2003-10-21 10:35:04 UTC
I just applied the modified sigchld patch and I can confirm now that my xlock
works :) thanks!
Comment 7 Martin Holzer (RETIRED) gentoo-dev 2003-10-29 13:53:34 UTC
any work done on 0.77 ?
Comment 8 Martin Schlemmer (RETIRED) gentoo-dev 2003-10-29 14:20:25 UTC
Yes, http://dev.gentoo.org/~woodchip/pam/.
Comment 9 Haroon Rafique 2003-12-09 11:31:09 UTC
As described in comment 3, I just tried the following:

1. re-packaged pam-0.75-r11.gentoo.tbz2 by first extracting it and then modifying
pam/patchdir/pam-0.75-sigchld.patch.bz2 to replace SIG_IGN with SIG_DFL in 2 places.

2. Put the repackaged pam-0.75-r11.gentoo.tbz2 in /usr/portage/distfiles

3. Updated /usr/portage/sys-libs/pam/files/digest-pam-0.75-r11 to reflect the
new size and md5sum for pam-0.75-r11.gentoo.tbz2

4. emerge pam

Happy to report that pam-0.75-r11 now let's xlock unlock the screen properly.

Running 2.6.0-test10-mm1.

Comment 10 Andy Dustman 2003-12-15 08:47:39 UTC
pam-0.77 hit testing recently; I'm merging it today.
Comment 11 Andy Dustman 2003-12-15 11:34:02 UTC
In 0.77, as in earlier versions, /etc/security/console.perms contains this line:

<dri>=/dev/nvidia* /dev/3dfx*

It really ought to be:

<dri>=/dev/nvidia* /dev/3dfx* /dev/dri/*
Comment 12 Andy Dustman 2004-02-23 13:42:10 UTC
pam-0.77 is now the stable version in Gentoo; maybe this bug can be closed.
Comment 13 Daniel Ahlberg (RETIRED) gentoo-dev 2004-04-27 06:33:50 UTC
Reassigning to base-system for decision on wethever this should be closed or not.
Comment 14 SpanKY gentoo-dev 2004-05-02 12:19:24 UTC
as noted, 0.77 is in portage