[libvirt] [PATCH] esx: Report an error for invalid arguments in esxList(Defined)Domains

Matthias Bolte matthias.bolte at googlemail.com
Thu Apr 8 10:04:16 UTC 2010


2010/4/7 Daniel Veillard <veillard at redhat.com>:
> On Wed, Apr 07, 2010 at 11:55:41AM +0200, 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 888ac10..eb06555 100644
>> --- a/src/esx/esx_driver.c
>> +++ b/src/esx/esx_driver.c
>> @@ -964,7 +964,8 @@ esxListDomains(virConnectPtr conn, int *ids, int maxids)
>>      int count = 0;
>>
>>      if (ids == NULL || maxids < 0) {
>> -        goto failure;
>> +        ESX_ERROR(VIR_ERR_INVALID_ARG, "%s", _("Invalid arguemnt"));
>> +        return -1;
>>      }
>>
>>      if (maxids == 0) {
>> @@ -2255,7 +2256,8 @@ esxListDefinedDomains(virConnectPtr conn, char **const names, int maxnames)
>>      int i;
>>
>>      if (names == NULL || maxnames < 0) {
>> -        goto failure;
>> +        ESX_ERROR(VIR_ERR_INVALID_ARG, "%s", _("Invalid arguemnt"));
>> +        return -1;
>>      }
>>
>>      if (maxnames == 0) {
>
>  Looks fine, ACK
>
> Daniel
>

There is a arguemnt vs argument typo. I've fixed it and pushed, thanks.

Matthias




More information about the libvir-list mailing list