Red Hat Bugzilla – Bug 229755
pam_succeed_if fails when it should succeed
Last modified: 2015-01-07 19:15:48 EST
Description of problem:
I'm trying to get vsftpd only to let people who DON'T have valid shells log in.
To do this, in /etc/pam.d/vsftpd I've changed the line
auth required pam_shells.so
auth required pam_succeed_if.so debug shell notin /bin/bash:/bin/sh
And in my logs I get:
Feb 23 06:41:48 cat vsftpd: pam_succeed_if: 'shell' resolves to
Feb 23 06:41:48 cat vsftpd: pam_succeed_if: requirement "shell notin
/bin/bash:/bin/sh" not met by user "john"
Obviously /usr/libexec/openssh/sftp-server is neither /bin/bash nor /bin/sh, so
I think it ought to have succeeded.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1.Make change as described above
2.Change a user's shell so it's not /bin/bash e.g. to /bin/false or whatever
3.Attempt FTP login (which will fail)
Login fails, bizarre-looking log message
Login succeeds, correct log message
I've followed this up and discovered that the "in" and "notin" feature was only
added in August last year, and isn't present in EL4's PAM, which will be why it
doesn't work. However, I really think the error message ought to say "no such
operator" or something like that, instead of just "requirement not met". Just in
case you're interested, I'll attach a patch to provide the feature (originally
but modified to apply cleanly to your pam-0.77-66.17).
Created attachment 148651 [details]
Add upstream "in" and "notin" list matching feature
Created attachment 148652 [details]
Patch to pam spec to (i) build cleanly and (ii) add list match feature
Build fails with unpackaged files error, so remove them (they're unnecessary
*** This bug has been marked as a duplicate of 173926 ***