[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH V8 2/4] block: Add check infinite loop in bdrv_img_c
From: |
Xu Wang |
Subject: |
[Qemu-devel] [PATCH V8 2/4] block: Add check infinite loop in bdrv_img_create() |
Date: |
Fri, 15 Nov 2013 01:37:21 -0500 |
Backing file loop should be checked before qemu-img create command
execution. If loop is found, qemu-img create should be stopped and
an error printed.
Signed-off-by: Xu Wang <address@hidden>
---
block.c | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/block.c b/block.c
index 24202c0..6fbc702 100644
--- a/block.c
+++ b/block.c
@@ -4627,15 +4627,17 @@ void bdrv_img_create(const char *filename, const char
*fmt,
}
backing_file = get_option_parameter(param, BLOCK_OPT_BACKING_FILE);
+ backing_fmt = get_option_parameter(param, BLOCK_OPT_BACKING_FMT);
if (backing_file && backing_file->value.s) {
- if (!strcmp(filename, backing_file->value.s)) {
- error_setg(errp, "Error: Trying to create an image with the "
- "same filename as the backing file");
+ if (!bdrv_backing_chain_okay(backing_file->value.s,
+ backing_fmt->value.s, filename,
+ &local_err)) {
+ error_setg(errp, "bdrv_img_create: Image %s create failed. %s",
+ filename, error_get_pretty(local_err));
goto out;
}
}
- backing_fmt = get_option_parameter(param, BLOCK_OPT_BACKING_FMT);
if (backing_fmt && backing_fmt->value.s) {
backing_drv = bdrv_find_format(backing_fmt->value.s);
if (!backing_drv) {
--
1.8.1.4