summaryrefslogtreecommitdiffstats
path: root/block.c
diff options
context:
space:
mode:
authorKevin Wolf2011-06-01 14:03:31 +0200
committerKevin Wolf2011-06-08 11:56:40 +0200
commitd220894e025ea85d604a2cdc1eef541ed516728e (patch)
tree2e79df2e4e08ca540f9b8ebeffb77e1fbe632be0 /block.c
parentqcow2: Fix memory leaks in error cases (diff)
downloadqemu-d220894e025ea85d604a2cdc1eef541ed516728e.tar.gz
qemu-d220894e025ea85d604a2cdc1eef541ed516728e.tar.xz
qemu-d220894e025ea85d604a2cdc1eef541ed516728e.zip
bdrv_img_create: Fix segfault
Block drivers that don't support creating images don't have a size option. Fail gracefully instead of segfaulting when trying to access the option's value. Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block.c')
-rw-r--r--block.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/block.c b/block.c
index 3036a2deed..24a25d569b 100644
--- a/block.c
+++ b/block.c
@@ -2881,7 +2881,7 @@ int bdrv_img_create(const char *filename, const char *fmt,
char *options, uint64_t img_size, int flags)
{
QEMUOptionParameter *param = NULL, *create_options = NULL;
- QEMUOptionParameter *backing_fmt, *backing_file;
+ QEMUOptionParameter *backing_fmt, *backing_file, *size;
BlockDriverState *bs = NULL;
BlockDriver *drv, *proto_drv;
BlockDriver *backing_drv = NULL;
@@ -2964,7 +2964,8 @@ int bdrv_img_create(const char *filename, const char *fmt,
// The size for the image must always be specified, with one exception:
// If we are using a backing file, we can obtain the size from there
- if (get_option_parameter(param, BLOCK_OPT_SIZE)->value.n == -1) {
+ size = get_option_parameter(param, BLOCK_OPT_SIZE);
+ if (size && size->value.n == -1) {
if (backing_file && backing_file->value.s) {
uint64_t size;
char buf[32];