aboutsummaryrefslogtreecommitdiff
path: root/block/qcow2-refcount.c
diff options
context:
space:
mode:
authorStefan Hajnoczi <stefanha@linux.vnet.ibm.com>2010-04-17 10:49:07 +0100
committerKevin Wolf <kwolf@redhat.com>2010-05-03 10:07:31 +0200
commit508e0893686794be55cfaa336fea584b16a471d9 (patch)
treef0bfec250bdf626d18da8f4d71c933b43309427f /block/qcow2-refcount.c
parent557df6aca2e389ef631b283ca1522b1fdf121eff (diff)
downloadqemu-508e0893686794be55cfaa336fea584b16a471d9.zip
qemu-508e0893686794be55cfaa336fea584b16a471d9.tar.gz
qemu-508e0893686794be55cfaa336fea584b16a471d9.tar.bz2
qcow2: Avoid shadowing variable in alloc_clusters_noref()
The i loop iterator is shadowed by the next free cluster index. Both using the variable name 'i' makes the code harder to read. Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block/qcow2-refcount.c')
-rw-r--r--block/qcow2-refcount.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
index 2661493..95491d3 100644
--- a/block/qcow2-refcount.c
+++ b/block/qcow2-refcount.c
@@ -556,8 +556,8 @@ static int64_t alloc_clusters_noref(BlockDriverState *bs, int64_t size)
nb_clusters = size_to_clusters(s, size);
retry:
for(i = 0; i < nb_clusters; i++) {
- int64_t i = s->free_cluster_index++;
- if (get_refcount(bs, i) != 0)
+ int64_t next_cluster_index = s->free_cluster_index++;
+ if (get_refcount(bs, next_cluster_index) != 0)
goto retry;
}
#ifdef DEBUG_ALLOC2