On Tue, Jan 21, 2014 at 11:30:47AM +0100, Lukas Slebodnik wrote: > ehlo, > > I found few unused macros: PAM_SM_AUTH, PAM_SM_ACCOUNT, PAM_SM_SESSION, > PAM_SM_PASSWORD. > > In my opinion, they were proposed for the same purpose as values from > enum sss_cli_command (SSS_PAM_AUTHENTICATE, SSS_PAM_ACCT_MGMT, > SSS_APM_*). > > LS
Those macros are use as switches in /usr/include/security/pam_modules.h but only if used together with PAM_STATIC. Since we do not use PAM_STATIC I think it is safe to remove them. Additionally I checked the intermediate code after the cpp run and didn't found any difference. ACK bye, Sumit _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
