diff options
| author | Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> | 2010-04-17 10:49:07 +0100 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2010-05-03 10:07:31 +0200 |
| commit | 508e0893686794be55cfaa336fea584b16a471d9 (patch) | |
| tree | f0bfec250bdf626d18da8f4d71c933b43309427f /block/qcow2-refcount.c | |
| parent | 557df6aca2e389ef631b283ca1522b1fdf121eff (diff) | |
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.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c index 2661493a7d..95491d3388 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 |