diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2011-10-05 09:12:04 +0200 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2011-11-04 15:55:45 +0100 |
| commit | e654bfe4c1211754d52b324a505ac46c2bfe2069 (patch) | |
| tree | 4deaa34805b58ab7b4515ba0e3ee3b7bb62335e4 /block/vvfat.c | |
| parent | 2b6a43a835e5082dedc6a5eea39a59463f97c81c (diff) | |
| download | qemu-e654bfe4c1211754d52b324a505ac46c2bfe2069.tar.gz | |
vvfat: do not fail if the disk has spare sectors
If the number of "faked sectors" + the number of sectors that are
part of a cluster does not sum up to the total number of sectors,
qemu-img convert fails. Read these spare sectors as all zeros.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block/vvfat.c')
| -rw-r--r-- | block/vvfat.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/block/vvfat.c b/block/vvfat.c index 75d0dc07e0..9f851b09de 100644 --- a/block/vvfat.c +++ b/block/vvfat.c @@ -1245,7 +1245,7 @@ static int vvfat_read(BlockDriverState *bs, int64_t sector_num, int i; for(i=0;i<nb_sectors;i++,sector_num++) { - if (sector_num >= s->sector_count) + if (sector_num >= bs->total_sectors) return -1; if (s->qcow) { int n; @@ -1271,7 +1271,7 @@ DLOG(fprintf(stderr, "sector %d not allocated\n", (int)sector_num)); uint32_t sector=sector_num-s->faked_sectors, sector_offset_in_cluster=(sector%s->sectors_per_cluster), cluster_num=sector/s->sectors_per_cluster; - if(read_cluster(s, cluster_num) != 0) { + if(cluster_num > s->cluster_count || read_cluster(s, cluster_num) != 0) { /* LATER TODO: strict: return -1; */ memset(buf+i*0x200,0,0x200); continue; |