[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] block/stream.c -Werror build failure

From: Andreas Färber
Subject: Re: [Qemu-devel] block/stream.c -Werror build failure
Date: Thu, 19 Sep 2013 19:00:18 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130801 Thunderbird/17.0.8

Am 19.09.2013 18:59, schrieb Stefan Weil:
> 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/

Just spotted it myself and added a review comment.


SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

reply via email to

[Prev in Thread] Current Thread [Next in Thread]