On 07/30/2010 09:23 AM, Matthias Bolte wrote:
> ---
> src/esx/esx_driver.c | 6 ++++--
> 1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c
> index eb64556..964a3a5 100644
> --- a/src/esx/esx_driver.c
> +++ b/src/esx/esx_driver.c
> @@ -2483,6 +2483,7 @@ esxDomainCreateWithFlags(virDomainPtr domain, unsigned int flags)
> esxVI_ObjectContent *virtualMachine = NULL;
> esxVI_String *propertyNameList = NULL;
> esxVI_VirtualMachinePowerState powerState;
> + int id = -1;
> esxVI_ManagedObjectReference *task = NULL;
> esxVI_TaskInfoState taskInfoState;
>
> @@ -2497,8 +2498,8 @@ esxDomainCreateWithFlags(virDomainPtr domain, unsigned int flags)
> esxVI_LookupVirtualMachineByUuidAndPrepareForTask
> (priv->primary, domain->uuid, propertyNameList, &virtualMachine,
> priv->autoAnswer) < 0 ||
> - esxVI_GetVirtualMachinePowerState(virtualMachine,
> - &powerState) < 0) {
> + esxVI_GetVirtualMachinePowerState(virtualMachine, &powerState) < 0 ||
> + esxVI_GetVirtualMachineIdentity(virtualMachine, &id, NULL, NULL) < 0) {
ACK. This leaves domain->id unchanged on failure, but I think that's
okay, because the caller shouldn't be relying on the contents of domain
on failure.
--
Eric Blake eblake@xxxxxxxxxx +1-801-349-2682
Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list