Ping? Could you please review this? On Wed, 2020-02-19 at 15:44 +0100, Tomas Mraz wrote: > The attached patch adds a new pam_start_confdir() API call to > address > the issue #143. It allows setting a directory from which the > service > PAM configuration is to be loaded. Please review. > > _______________________________________________ > Pam-developers mailing list -- > pam-developers(a)lists.fedorahosted.org > To unsubscribe send an email to > pam-developers-leave(a)lists.fedorahosted.org > Fedora Code of Conduct: > https://docs.fedoraproject.org/en-US/project/code-of-conduct/ > List Guidelines: > https://fedoraproject.org/wiki/Mailing_list_guidelines > List Archives: > https://lists.fedorahosted.org/archives/list/pam-developers@lists.fedorah... -- Tomáš Mráz No matter how far down the wrong road you've gone, turn back. Turkish proverb [You'll know whether the road is wrong if you carefully listen to your conscience.] _______________________________________________ Pam-developers mailing list -- pam-developers(a)lists.fedorahosted.org To unsubscribe send an email to pam-developers-leave(a)lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/pam-developers@lists.fedorah...

On Wed, Feb 19, 2020 at 03:44:51PM +0100, Tomas Mraz wrote: > The attached patch adds a new pam_start_confdir() API call to > address > the issue #143. It allows setting a directory from which the > service > PAM configuration is to be loaded. Please review. Hmm. Do we want to allow arbitrary non-NULL confdir argument, or should we rather restrict it to a path starting with '/'?