[libvirt] [PATCH] qemu: return -1 on error paths in qemuDomainSaveImageStartVM

Jiri Denemark jdenemar at redhat.com
Thu Jan 28 13:27:59 UTC 2016


On Thu, Jan 28, 2016 at 15:49:48 +0300, Nikolay Shirokovskiy wrote:
> The error paths after fix suppose ret == -1 as at the
> beginning of the function.
> 
> The better fix would be not to touch 'ret' at all until
> the end of the function where it is set to 0. Thus we'd
> better introduce some 'rc' variable to hold return
> values of called functions.

Exactly, it would be much better the way you describe, so why didn't you
do it that way? :-)

NACK to this patch.

Jirka




More information about the libvir-list mailing list