[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] block/stream.c -Werror build failure
From: |
Stefan Weil |
Subject: |
Re: [Qemu-devel] block/stream.c -Werror build failure |
Date: |
Thu, 19 Sep 2013 18:59:01 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130803 Thunderbird/17.0.8 |
Am 19.09.2013 18:52, schrieb Andreas Färber:
> Hello,
>
> As reported yesterday on IRC, since the last round of pulls I am getting
> the following warning-treated-as-error:
>
> CC block/stream.o
> /home/andreas/QEMU/qemu/block/stream.c: In function ‘stream_run’:
> /home/andreas/QEMU/qemu/block/stream.c:111:14: error: ‘copy’ may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> cc1: all warnings being treated as errors
> make: *** [block/stream.o] Fehler 1
>
> $ gcc --version
> gcc (SUSE Linux) 4.7.2 20130108 [gcc-4_7-branch revision 195012]
>
> However I only got this in 1 of 2 repositories I tried. It turns out
> that --enable-debug avoids this build issue.
>
> The stream.c code itself looks okay to me. In the case of ret >= 0, copy
> is always assigned, including the goto code path.
> Still, could we initialize the copy variable to false or revert whatever
> change introduced this build regression? (-Wextra?)
>
> Regards,
> Andreas
>
This will be fixed by the next pull request for the block tree.
See also http://patchwork.ozlabs.org/patch/275503/
Regards,
Stefan