On 07/10/2017 10:05 AM, Denis V. Lunev wrote: > We've had a shadowed 'ret' variable, which risks returning the wrong > value, introduced in commit b9c64947. > > Signed-off-by: Denis V. Lunev <[email protected]> > Reviewed-by: Fam Zheng <[email protected]> > CC: Stefan Hajnoczi <[email protected]> > CC: Kevin Wolf <[email protected]> > CC: Eric Blake <[email protected]> > --- > block/io.c | 1 - > 1 file changed, 1 deletion(-) >
Reviewed-by: Eric Blake <[email protected]> > diff --git a/block/io.c b/block/io.c > index 23170a57ee..87208990c4 100644 > --- a/block/io.c > +++ b/block/io.c > @@ -2327,7 +2327,6 @@ int coroutine_fn bdrv_co_pdiscard(BlockDriverState *bs, > int64_t offset, > assert(max_pdiscard >= bs->bl.request_alignment); > > while (bytes > 0) { > - int ret; > int num = bytes; > > if (head) { > -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
