[Qemu-devel] [PULL v2 06/12] configure: fix pam test warning

2019-05-03 Thread Laurent Vivier
From: "Dr. David Alan Gilbert" 

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 
Acked-by: Daniel P. Berrangé 
Reviewed-by: Philippe Mathieu-Daudé 
Message-Id: <20190404091725.20595-1-dgilb...@redhat.com>
Signed-off-by: Laurent Vivier 
---
 configure | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index 60719ddcc5b0..a2607afb3408 100755
--- a/configure
+++ b/configure
@@ -2940,9 +2940,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, );
+   pam_start(service_name, user, _conv, );
return 0;
 }
 EOF
-- 
2.20.1




[Qemu-devel] [PULL v2 06/12] configure: fix pam test warning

2019-05-03 Thread Laurent Vivier
From: "Dr. David Alan Gilbert" 

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 
Acked-by: Daniel P. Berrangé 
Reviewed-by: Philippe Mathieu-Daudé 
Message-Id: <20190404091725.20595-1-dgilb...@redhat.com>
Signed-off-by: Laurent Vivier 
---
 configure | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index 60719ddcc5b0..a2607afb3408 100755
--- a/configure
+++ b/configure
@@ -2940,9 +2940,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, );
+   pam_start(service_name, user, _conv, );
return 0;
 }
 EOF
-- 
2.20.1




[Qemu-devel] [PULL v2 06/12] configure: fix pam test warning

2019-05-03 Thread Laurent Vivier
From: "Dr. David Alan Gilbert" 

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 
Acked-by: Daniel P. Berrangé 
Reviewed-by: Philippe Mathieu-Daudé 
Message-Id: <20190404091725.20595-1-dgilb...@redhat.com>
Signed-off-by: Laurent Vivier 
---
 configure | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index 60719ddcc5b0..a2607afb3408 100755
--- a/configure
+++ b/configure
@@ -2940,9 +2940,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, );
+   pam_start(service_name, user, _conv, );
return 0;
 }
 EOF
-- 
2.20.1