summaryrefslogtreecommitdiff
path: root/block.c
diff options
context:
space:
mode:
authorKevin Wolf <kwolf@redhat.com>2013-04-09 13:19:18 +0200
committerKevin Wolf <kwolf@redhat.com>2013-04-22 10:27:59 +0200
commit2af5ef70af9fdc823407ddc4ae14766806d4837a (patch)
treefde91f9d44a54ea67cd13036d8cd2311dd2bd18f /block.c
parentbdda92324dad963e85589e9ea74b2f2620319304 (diff)
downloadqemu-2af5ef70af9fdc823407ddc4ae14766806d4837a.tar.gz
qemu-2af5ef70af9fdc823407ddc4ae14766806d4837a.tar.bz2
qemu-2af5ef70af9fdc823407ddc4ae14766806d4837a.zip
block: Fail gracefully when using a format driver on protocol level
Specifying the wrong driver could fail an assertion: $ qemu-system-x86_64 -drive file.driver=qcow2,file=x qemu-system-x86_64: block.c:721: bdrv_open_common: Assertion `file != ((void *)0)' failed. Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block.c')
-rw-r--r--block.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/block.c b/block.c
index 4ad663d3fa..bea47c4755 100644
--- a/block.c
+++ b/block.c
@@ -718,6 +718,13 @@ static int bdrv_open_common(BlockDriverState *bs, BlockDriverState *file,
assert(drv->bdrv_parse_filename || filename != NULL);
ret = drv->bdrv_file_open(bs, filename, options, open_flags);
} else {
+ if (file == NULL) {
+ qerror_report(ERROR_CLASS_GENERIC_ERROR, "Can't use '%s' as a "
+ "block driver for the protocol level",
+ drv->format_name);
+ ret = -EINVAL;
+ goto free_and_fail;
+ }
assert(file != NULL);
bs->file = file;
ret = drv->bdrv_open(bs, options, open_flags);