Re: [libvirt] [PATCH] esx: avoid null dereference on error

2011-05-04 Thread Matthias Bolte
2011/5/4 Daniel Veillard : > On Wed, May 04, 2011 at 08:38:43AM +0200, Matthias Bolte wrote: >> 2011/5/4 Laine Stump : >> > On 05/03/2011 03:10 PM, Eric Blake wrote: >> >> >> >> Detected by clang. >> >> >> >> * src/esx/esx_driver.c (esxDomainGetInfo): Fail early on error. >> >> --- >> >>  src/esx/e

Re: [libvirt] [PATCH] esx: avoid null dereference on error

2011-05-04 Thread Daniel Veillard
On Wed, May 04, 2011 at 08:38:43AM +0200, Matthias Bolte wrote: > 2011/5/4 Laine Stump : > > On 05/03/2011 03:10 PM, Eric Blake wrote: > >> > >> Detected by clang. > >> > >> * src/esx/esx_driver.c (esxDomainGetInfo): Fail early on error. > >> --- > >>  src/esx/esx_driver.c |    1 + > >>  1 files ch

Re: [libvirt] [PATCH] esx: avoid null dereference on error

2011-05-03 Thread Matthias Bolte
2011/5/4 Laine Stump : > On 05/03/2011 03:10 PM, Eric Blake wrote: >> >> Detected by clang. >> >> * src/esx/esx_driver.c (esxDomainGetInfo): Fail early on error. >> --- >>  src/esx/esx_driver.c |    1 + >>  1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/src/esx/esx_driver.c b/s

Re: [libvirt] [PATCH] esx: avoid null dereference on error

2011-05-03 Thread Laine Stump
On 05/03/2011 03:10 PM, Eric Blake wrote: Detected by clang. * src/esx/esx_driver.c (esxDomainGetInfo): Fail early on error. --- src/esx/esx_driver.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c index 1f8f90b..e929208 100

[libvirt] [PATCH] esx: avoid null dereference on error

2011-05-03 Thread Eric Blake
Detected by clang. * src/esx/esx_driver.c (esxDomainGetInfo): Fail early on error. --- src/esx/esx_driver.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c index 1f8f90b..e929208 100644 --- a/src/esx/esx_driver.c +++ b/src/esx/e