[libvirt] PATCH: Fix non-raw storage volume cloning
Cole Robinson
crobinso at redhat.com
Fri Jul 17 13:45:12 UTC 2009
Daniel P. Berrange wrote:
> The recent storage patches broke cloning of non-raw volumes, due to a
> small typo. THis was identified by the libvirt TCK storage tests!
>
Go automated testing! ACK from me.
- Cole
> Daniel
>
> diff --git a/src/storage_backend.c b/src/storage_backend.c
> index 1664804..67815d7 100644
> --- a/src/storage_backend.c
> +++ b/src/storage_backend.c
> @@ -567,7 +567,7 @@ virStorageBackendGetBuildVolFromFunction(virConnectPtr conn,
> (inputvol->type == VIR_STORAGE_VOL_FILE &&
> inputvol->target.format != VIR_STORAGE_VOL_FILE_RAW)) {
>
> - if ((tool_type = virStorageBackendFindFSImageTool(NULL)) != -1) {
> + if ((tool_type = virStorageBackendFindFSImageTool(NULL)) < 0) {
> virStorageReportError(conn, VIR_ERR_INTERNAL_ERROR,
> "%s", _("creation of non-raw file images is "
> "not supported without qemu-img."));
>
More information about the libvir-list
mailing list