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 spott

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 qem

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 {

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 guest

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

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 >>

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 snapsh

[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