[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH] qemu-img: check bs_n when use old
From: |
Fam Zheng |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH] qemu-img: check bs_n when use old style option |
Date: |
Thu, 20 Apr 2017 15:51:28 +0800 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Thu, 04/20 12:04, address@hidden wrote:
> From: Lidong Chen <address@hidden>
>
> When use old style option like -o backing_file, img_convert
> continue run when bs_n > 1, this patch fix this bug.
>
> Signed-off-by: Lidong Chen <address@hidden>
> ---
> qemu-img.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/qemu-img.c b/qemu-img.c
> index b220cf7..c673aef 100644
> --- a/qemu-img.c
> +++ b/qemu-img.c
> @@ -2225,6 +2225,13 @@ static int img_convert(int argc, char **argv)
> out_baseimg = out_baseimg_param;
> }
>
> + if (bs_n > 1 && out_baseimg) {
> + error_report("-B makes no sense when concatenating multiple input "
> + "images");
> + ret = -1;
> + goto out;
> + }
> +
> /* Check if compression is supported */
> if (compress) {
> bool encryption =
> --
> 1.8.3.1
>
>
Is this essentially the same as the check a few lines above:
...
if (bs_n < 1) {
error_exit("Must specify image file name");
}
if (bs_n > 1 && out_baseimg) {
error_report("-B makes no sense when concatenating multiple input "
"images");
ret = -1;
goto out;
}
src_flags = 0;
ret = bdrv_parse_cache_mode(src_cache, &src_flags, &src_writethrough);
if (ret < 0) {
error_report("Invalid source cache option: %s", src_cache);
goto out;
}
...
How about moving that down?
Fam