OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

qcow2: Increase maximum cluster size to 2 MB

Browse Source 
This patch increases the maximum qcow2 cluster size to 2 MB. Starting with 128k
clusters, L2 tables span 2 GB or more of virtual disk space, causing 32 bit
truncation and wraparound of signed integers. Therefore some variables need to
use a larger data type.

While being at reviewing data types, change some integers that are used for
array indices to unsigned. In some places they were checked against some upper
limit but not for negative values. This could avoid potential segfaults with
corrupted qcow2 images.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
Kevin Wolf 2009-09-15 12:30:43 +02:00 committed by Anthony Liguori
parent 85352471ce
commit 80ee15a6b2
4 changed files with 12 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
block/qcow2-cluster.c
View File

@ -264,7 +264,7 @@ static int count_contiguous_clusters(uint64_t nb_clusters, int cluster_size,
return 0; return 0;
for (i = start; i < start + nb_clusters; i++) for (i = start; i < start + nb_clusters; i++)
if (offset + i * cluster_size != (be64_to_cpu(l2_table[i]) & ~mask)) if (offset + (uint64_t) i * cluster_size != (be64_to_cpu(l2_table[i]) & ~mask))
break; break;
return (i - start); return (i - start);
@ -395,10 +395,11 @@ uint64_t qcow2_get_cluster_offset(BlockDriverState *bs, uint64_t offset,
int *num) int *num)
{ {
BDRVQcowState *s = bs->opaque; BDRVQcowState *s = bs->opaque;
int l1_index, l2_index; unsigned int l1_index, l2_index;
uint64_t l2_offset, *l2_table, cluster_offset; uint64_t l2_offset, *l2_table, cluster_offset;
int l1_bits, c; int l1_bits, c;
int index_in_cluster, nb_available, nb_needed, nb_clusters; unsigned int index_in_cluster, nb_clusters;
uint64_t nb_available, nb_needed;
index_in_cluster = (offset >> 9) & (s->cluster_sectors - 1); index_in_cluster = (offset >> 9) & (s->cluster_sectors - 1);
nb_needed = *num + index_in_cluster; nb_needed = *num + index_in_cluster;
@ -409,7 +410,7 @@ uint64_t qcow2_get_cluster_offset(BlockDriverState *bs, uint64_t offset,
* the end of the l1 entry * the end of the l1 entry
*/ */
nb_available = (1 << l1_bits) - (offset & ((1 << l1_bits) - 1)); nb_available = (1ULL << l1_bits) - (offset & ((1ULL << l1_bits) - 1));
/* compute the number of available sectors */ /* compute the number of available sectors */
@ -483,8 +484,9 @@ static int get_cluster_table(BlockDriverState *bs, uint64_t offset,
int *new_l2_index) int *new_l2_index)
{ {
BDRVQcowState *s = bs->opaque; BDRVQcowState *s = bs->opaque;
int l1_index, l2_index, ret; unsigned int l1_index, l2_index;
uint64_t l2_offset, *l2_table; uint64_t l2_offset, *l2_table;
int ret;
/* seek the the l2 offset in the l1 table */ /* seek the the l2 offset in the l1 table */
@ -683,7 +685,7 @@ uint64_t qcow2_alloc_cluster_offset(BlockDriverState *bs,
BDRVQcowState *s = bs->opaque; BDRVQcowState *s = bs->opaque;
int l2_index, ret; int l2_index, ret;
uint64_t l2_offset, *l2_table, cluster_offset; uint64_t l2_offset, *l2_table, cluster_offset;
int nb_clusters, i = 0; unsigned int nb_clusters, i = 0;
QCowL2Meta *old_alloc; QCowL2Meta *old_alloc;
ret = get_cluster_table(bs, offset, &l2_table, &l2_offset, &l2_index); ret = get_cluster_table(bs, offset, &l2_table, &l2_offset, &l2_index);

3
block/qcow2-refcount.c
View File

@ -192,7 +192,8 @@ static int64_t alloc_refcount_block(BlockDriverState *bs, int64_t cluster_index)
{ {
BDRVQcowState *s = bs->opaque; BDRVQcowState *s = bs->opaque;
int64_t offset, refcount_block_offset; int64_t offset, refcount_block_offset;
int ret, refcount_table_index; unsigned int refcount_table_index;
int ret;
uint64_t data64; uint64_t data64;
int cache = cache_refcount_updates; int cache = cache_refcount_updates;

2
block/qcow2.h
View File

@ -47,7 +47,7 @@
#define REFCOUNT_SHIFT 1 /* refcount size is 2 bytes */ #define REFCOUNT_SHIFT 1 /* refcount size is 2 bytes */
#define MIN_CLUSTER_BITS 9 #define MIN_CLUSTER_BITS 9
#define MAX_CLUSTER_BITS 16 #define MAX_CLUSTER_BITS 21
#define L2_CACHE_SIZE 16 #define L2_CACHE_SIZE 16

2
qemu-img.c
View File

@ -530,7 +530,7 @@ static int is_allocated_sectors(const uint8_t *buf, int n, int *pnum)
return v; return v;
} }
#define IO_BUF_SIZE 65536 #define IO_BUF_SIZE (2 * 1024 * 1024)
static int img_convert(int argc, char **argv) static int img_convert(int argc, char **argv)
{ {