[libvirt] [PATCH] Fix pool define crash
Chris Lalancette
clalance at redhat.com
Wed Dec 17 20:26:51 UTC 2008
Cole Robinson wrote:
> Ahh, I didn't realize the AssignDef caveat. Thanks for pointing that
> out. Something like this should work then?
> diff --git a/src/storage_driver.c b/src/storage_driver.c
> index 2432a9a..330317c 100644
> --- a/src/storage_driver.c
> +++ b/src/storage_driver.c
> @@ -544,12 +544,13 @@ storagePoolDefine(virConnectPtr conn,
>
> if (!(pool = virStoragePoolObjAssignDef(conn, &driver->pools, def)))
> goto cleanup;
> - def = NULL;
>
> if (virStoragePoolObjSaveDef(conn, driver, pool, def) < 0) {
> virStoragePoolObjRemove(&driver->pools, pool);
> + def = NULL;
> goto cleanup;
> }
> + def = NULL;
>
> ret = virGetStoragePool(conn, pool->def->name, pool->def->uuid);
>
Yeah, that looks better. Seems to be good to me, so ACK.
--
Chris Lalancette
More information about the libvir-list
mailing list