mirrors/qemu
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

nbd/server: Silence gcc false positive

Browse Source 
The code has a while() loop that always initialized 'end', and
the loop always executes at least once (as evidenced by the assert()
just prior to the loop).  But some versions of gcc still complain
that 'end' is used uninitialized, so silence them.

Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-id: 20180622125814.345274-1-eblake@redhat.com
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
Eric Blake 2018-06-22 07:58:14 -05:00 committed by Peter Maydell
parent c74b91ac1c
commit 45eb6fb6ce
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
nbd/server.c
View File

@ -1937,7 +1937,7 @@ static unsigned int bitmap_to_extents(BdrvDirtyBitmap *bitmap, uint64_t offset,
unsigned int nb_extents, unsigned int nb_extents,
bool dont_fragment) bool dont_fragment)
{ {
uint64_t begin = offset, end; uint64_t begin = offset, end = offset;
uint64_t overall_end = offset + *length; uint64_t overall_end = offset + *length;
unsigned int i = 0; unsigned int i = 0;
BdrvDirtyBitmapIter *it; BdrvDirtyBitmapIter *it;
@ -1977,6 +1977,7 @@ static unsigned int bitmap_to_extents(BdrvDirtyBitmap *bitmap, uint64_t offset,
bdrv_dirty_bitmap_unlock(bitmap); bdrv_dirty_bitmap_unlock(bitmap);
assert(offset > end);
*length = end - offset; *length = end - offset;
return i; return i;
} }