[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH] configure: fix pam test warning
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-trivial] [PATCH] configure: fix pam test warning |
Date: |
Thu, 11 Apr 2019 22:03:58 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
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" <address@hidden>
>>
>> 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 <address@hidden>
>> ---
>> configure | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Acked-by: Daniel P. Berrangé <address@hidden>
>
>>
>> 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