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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2010/4/7 Daniel Veillard <veillard@xxxxxxxxxx>:
> 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

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]