[libvirt] [PATCH] qemuProcessStop: Don't overwrite any errors

[Eric Blake]

On 01/07/2014 06:41 AM, Michal Privoznik wrote:
> Currently, the qemuProcessStop tries to open the domain log file
> and saves the original error afterwards. Then all the cleanup is
> done after which the error is restored back. This has however one
> flaw: if opening of the log file fails an error is reported,
> which results in previous error being overwritten (the useful
> one, e.g. "PCI device XXXX:XXXX could not be found"). Hence, user
> sees something like:
> 
>   error: failed to create logfile /var/log/libvirt/qemu/ovirt_usb.log: No such file or directory
> 
> instead of:
> 
>   error: internal error: Did not find USB device 8644:8003
> 
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> Reported-by: Zhou Yimin <zhouyimin at huawei.com>
> ---
>  src/qemu/qemu_process.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)

ACK.

[It may _still_ be worth a patch that pre-creates the containing
directory at the right point in time, but this patch should go in first]

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140107/fb82536e/attachment-0001.sig>