On Thu, Apr 20, 2017 at 3:51 PM, Fam Zheng <f...@redhat.com> wrote: > On Thu, 04/20 12:04, jemmy858...@gmail.com wrote: >> From: Lidong Chen <lidongc...@tencent.com> >> >> 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 <lidongc...@tencent.com> >> --- >> 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? moving that down is ok. but will exit later if use -B option. which way do you think better?
> > Fam