[libvirt] [PATCH] qemu: Fix virFileMakePath error handling in snapshot creation

Osier Yang jyang at redhat.com
Mon Jul 4 09:01:52 UTC 2011


于 2011年07月04日 16:50, Matthias Bolte 写道:
> virFileMakePath returns an errno value on error, that will never be
> negative. So a virFileMakePath error whould have been ignored here.

s/whould/should/

> ---
>   src/qemu/qemu_driver.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 363a361..aab3ab9 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -7498,7 +7498,7 @@ static int qemuDomainSnapshotWriteMetadata(virDomainObjPtr vm,
>           goto cleanup;
>       }
>       err = virFileMakePath(snapDir);
> -    if (err<  0) {
> +    if (err != 0) {
>           virReportSystemError(err, _("cannot create snapshot directory '%s'"),
>                                snapDir);
>           goto cleanup;


ACK

Osier




More information about the libvir-list mailing list