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