[SSSD] [PATCH] PAM: close socket fd with pam_set_data
Sumit Bose
sbose at redhat.com
Thu Oct 11 11:28:03 UTC 2012
On Thu, Oct 11, 2012 at 10:00:15AM +0200, Jakub Hrozek wrote:
> When a PAM-aware application calls pam_end(), our module would call its
> close_fd "destructor" and close the pipe from its end.
>
> https://fedorahosted.org/sssd/ticket/1569
Patch looks good, I just wonder if pam_set_data() better should be
called directly after sss_pam_make_request() to make sure the socket is
closed even in the case of errors in sss_pam_make_request() or
eval_response() ?
bye,
Sumit
More information about the sssd-devel
mailing list