On 04/04/2019 11:19, Daniel P. Berrangé wrote: > CC trivial > > On Thu, Apr 04, 2019 at 10:17:25AM +0100, Dr. David Alan Gilbert (git) wrote: >> From: "Dr. David Alan Gilbert" <dgilb...@redhat.com> >> >> The pam test generates a warning on Fedora 29 with -O3 compilation >> because the headers declare that the pam_conversation pointer to >> pam_start must be non-NULL. Change it to use the same 0 initialised >> structure as we actually use in qauthz. >> >> Signed-off-by: Dr. David Alan Gilbert <dgilb...@redhat.com> >> --- >> configure | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > Acked-by: Daniel P. Berrangé <berra...@redhat.com> > >> >> diff --git a/configure b/configure >> index 1c563a7027..73f7ad2be0 100755 >> --- a/configure >> +++ b/configure >> @@ -2946,9 +2946,9 @@ if test "$auth_pam" != "no"; then >> int main(void) { >> const char *service_name = "qemu"; >> const char *user = "frank"; >> - const struct pam_conv *pam_conv = NULL; >> + const struct pam_conv pam_conv = { 0 }; >> pam_handle_t *pamh = NULL; >> - pam_start(service_name, user, pam_conv, &pamh); >> + pam_start(service_name, user, &pam_conv, &pamh); >> return 0; >> } >> EOF > > Regards, > Daniel >
Applied to my trivial-patches branch. Thanks, Laurent