[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [Qemu-devel][PATCH] qemu-img: fix failed qemu-img comma

From: Kevin Wolf
Subject: Re: [Qemu-block] [Qemu-devel][PATCH] qemu-img: fix failed qemu-img command return zero exit code defeat
Date: Fri, 14 Oct 2016 17:49:46 +0200
User-agent: Mutt/1.5.21 (2010-09-15)

Am 14.10.2016 um 17:39 hat Stefan Hajnoczi geschrieben:
> On Mon, Oct 10, 2016 at 11:07:09AM +0800, Xu Tian wrote:
> > If backing file can not open when do image rebase, flag 'ret' not
> > assign a non-zero value, then qemu-img process exit with code zero.
> > Assign value '-1' to flag 'ret' after report error message to fix
> > this defeat.
> I suggest rewording the commit description:
> img_rebase() returns 0 (success) when opening the backing file fails
> because 'ret' isn't set.

I already reworded it a bit when I applied the patch (it was sent twice
somehow and I replied to the other one):

    If the backing file cannot be opened when doing qemu-img rebase, the
    variable 'ret' was not assigned a non-zero value, and the qemu-img
    process terminated with exit code zero. Fix this.


Attachment: pgps2MLHxDfMA.pgp
Description: PGP signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]