search

SUSE / pam-config

Tool to modify common PAM configuration files
GNU General Public License v2.0
9 stars 16 forks source link

Remove bad "access" call and handle failing "stat" call properly. #9

Closed jmoellers closed 3 years ago

jmoellers commented 3 years ago

Prior to writing an service-specific config file, the main function calls access() on the destination file in /etc/pam.d. This will fail and no config file will be written when the original config file was installed in /usr/etc/pam.d. A similar problem exists when creating the new service file: create_service_file() wants to give the new service file the same user, group and mode as the old one, but the old one may not exist. In that case, set these to 0(root), 0(root), and 0644.

Also added "revoke" to the option list for pam_keyinit

laenion commented 3 years ago

I guess the pam-config.c.orig file shouldn't be there?

jmoellers commented 3 years ago

I do not have the slightest idea where that came from, but I've removed it.