On Thu, 16 Dec 2010 11:55:41 -0500
Stephen Gallagher <[email protected]> wrote:

> --- a/src/sss_client/pam_sss.c
> +++ b/src/sss_client/pam_sss.c
> @@ -413,7 +413,7 @@ static int do_pam_conversation(pam_handle_t
> *pamh, const int msg_style, }
>      } while (state != SSS_PAM_CONV_DONE);
>  
> -    *_answer = answer;
> +    if(answer) *_answer = answer;

FAIL-Style ^^^ there should be a space after the if :)

However it is not clear to me how this would prevent a segfault.

I think you meant it to be:
if (_answer) *_answer = answer;

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York
_______________________________________________
sssd-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to