Re: [Qemu-devel] [PATCH] configure: fix ALSA configure test

[Igor Mitsyanko <i.mitsyanko@samsung.com>]

I see now that this bug was already noticed, please ignore this mail

On 07/17/2012 09:34 PM, Igor Mitsyanko wrote:
> After commit 417c9d72d48275d19c60861896efd4962d21aca2 all configure tests are
> executed with -Werror flag. Current ALSA configure test program invokes a warning:
>
> warning: ‘handle’ is used uninitialized in this function [-Wuninitialized]
>
> which results in error with -Werror flag and, consequently, in alsa test failing.
> This means that QEMU won't configure with "--audio-drv-list=alsa".
>
> Initialize "handle" variable to fix compilation warning.
>
> Signed-off-by: Igor Mitsyanko <i.mitsyanko@samsung.com>
> ---
>   configure |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/configure b/configure
> index 0a3896e..cf4669f 100755
> --- a/configure
> +++ b/configure
> @@ -1888,7 +1888,7 @@ for drv in $audio_drv_list; do
>       case $drv in
>       alsa)
>       audio_drv_probe $drv alsa/asoundlib.h -lasound \
> -        "snd_pcm_t **handle; return snd_pcm_close(*handle);"
> +        "snd_pcm_t *handle = 0; return snd_pcm_close(handle);"
>       libs_softmmu="-lasound $libs_softmmu"
>       ;;
>
>