Re: [libvirt] new NULL-dereference in qemu_driver.c

2010-04-28 Thread Jim Meyering
Jim Meyering wrote: Daniel P. Berrange wrote: On Tue, Apr 27, 2010 at 06:45:16PM +0200, Jim Meyering wrote: I ran clang on the very latest and it spotted this problem: From qemu_driver.c, around line 11100, else { /* qemu is a little funny with running guests and the

Re: [libvirt] new NULL-dereference in qemu_driver.c

2010-04-28 Thread Jim Meyering
Chris Lalancette wrote: On 04/27/2010 04:40 PM, Jim Meyering wrote: Daniel P. Berrange wrote: On Tue, Apr 27, 2010 at 06:45:16PM +0200, Jim Meyering wrote: I ran clang on the very latest and it spotted this problem: From qemu_driver.c, around line 11100, else { /* qemu is a

Re: [libvirt] new NULL-dereference in qemu_driver.c

2010-04-28 Thread Chris Lalancette
On 04/28/2010 11:43 AM, Jim Meyering wrote: Chris Lalancette wrote: On 04/27/2010 04:40 PM, Jim Meyering wrote: Daniel P. Berrange wrote: On Tue, Apr 27, 2010 at 06:45:16PM +0200, Jim Meyering wrote: I ran clang on the very latest and it spotted this problem: From qemu_driver.c, around line

Re: [libvirt] new NULL-dereference in qemu_driver.c

2010-04-28 Thread Jim Meyering
Chris Lalancette wrote: On 04/28/2010 11:43 AM, Jim Meyering wrote: Chris Lalancette wrote: On 04/27/2010 04:40 PM, Jim Meyering wrote: Daniel P. Berrange wrote: On Tue, Apr 27, 2010 at 06:45:16PM +0200, Jim Meyering wrote: I ran clang on the very latest and it spotted this problem: From

[libvirt] new NULL-dereference in qemu_driver.c

2010-04-27 Thread Jim Meyering
I ran clang on the very latest and it spotted this problem: From qemu_driver.c, around line 11100, else { /* qemu is a little funny with running guests and the restoration * of snapshots. If the snapshot was taken online, * then after a loadvm monitor command, the

Re: [libvirt] new NULL-dereference in qemu_driver.c

2010-04-27 Thread Daniel P. Berrange
On Tue, Apr 27, 2010 at 06:45:16PM +0200, Jim Meyering wrote: I ran clang on the very latest and it spotted this problem: From qemu_driver.c, around line 11100, else { /* qemu is a little funny with running guests and the restoration * of snapshots. If the snapshot was

Re: [libvirt] new NULL-dereference in qemu_driver.c

2010-04-27 Thread Jim Meyering
Daniel P. Berrange wrote: On Tue, Apr 27, 2010 at 06:45:16PM +0200, Jim Meyering wrote: I ran clang on the very latest and it spotted this problem: From qemu_driver.c, around line 11100, else { /* qemu is a little funny with running guests and the restoration * of

Re: [libvirt] new NULL-dereference in qemu_driver.c

2010-04-27 Thread Chris Lalancette
On 04/27/2010 04:40 PM, Jim Meyering wrote: Daniel P. Berrange wrote: On Tue, Apr 27, 2010 at 06:45:16PM +0200, Jim Meyering wrote: I ran clang on the very latest and it spotted this problem: From qemu_driver.c, around line 11100, else { /* qemu is a little funny with running