diff options
author | Steve Langasek <vorlon@debian.org> | 2022-04-25 16:12:04 -0700 |
---|---|---|
committer | Steve Langasek <vorlon@debian.org> | 2022-04-25 16:12:04 -0700 |
commit | 3cd1f867ea5668cb8de1f99a68618b7601a99f6a (patch) | |
tree | 9225d4c06c9a5fc9ac08b359509f4ea3b3a73404 /patches-applied/022_pam_unix_group_time_miscfixes | |
download | pam-3cd1f867ea5668cb8de1f99a68618b7601a99f6a.tar.gz pam-3cd1f867ea5668cb8de1f99a68618b7601a99f6a.tar.bz2 pam-3cd1f867ea5668cb8de1f99a68618b7601a99f6a.zip |
Import pam_1.4.0-13.debian.tar.xz
[dgit import tarball pam 1.4.0-13 pam_1.4.0-13.debian.tar.xz]
Diffstat (limited to 'patches-applied/022_pam_unix_group_time_miscfixes')
-rw-r--r-- | patches-applied/022_pam_unix_group_time_miscfixes | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/patches-applied/022_pam_unix_group_time_miscfixes b/patches-applied/022_pam_unix_group_time_miscfixes new file mode 100644 index 00000000..b940aa27 --- /dev/null +++ b/patches-applied/022_pam_unix_group_time_miscfixes @@ -0,0 +1,22 @@ +Description: handle the case of flags being empty or only PAM_SILENT, which is + documented in other PAM implementations as meaning PAM_ESTABLISH_CRED: + http://publib.boulder.ibm.com/infocenter/aix/v6r1/index.jsp?topic=%2Fcom.ibm.aix.basetechref%2Fdoc%2Fbasetrf1%2Fpam_setcred.htm + +Index: pam/modules/pam_group/pam_group.c +=================================================================== +--- pam.orig/modules/pam_group/pam_group.c ++++ pam/modules/pam_group/pam_group.c +@@ -761,9 +761,12 @@ + unsigned setting; + + /* only interested in establishing credentials */ ++ /* PAM docs say that an empty flag is to be treated as PAM_ESTABLISH_CRED. ++ Some people just pass PAM_SILENT, so cope with it, too. */ + + setting = flags; +- if (!(setting & (PAM_ESTABLISH_CRED | PAM_REINITIALIZE_CRED))) { ++ if (!(setting & (PAM_ESTABLISH_CRED | PAM_REINITIALIZE_CRED)) ++ && (setting != 0) && (setting != PAM_SILENT)) { + D(("ignoring call - not for establishing credentials")); + return PAM_SUCCESS; /* don't fail because of this */ + } |