Re: [Qemu-devel] [PATCH 1/3] qcow2: Check bs->drv in copy_sectors()

2014-03-11 Thread Max Reitz
On 11.03.2014 11:16, Kevin Wolf wrote: Am 11.03.2014 um 00:16 hat Laszlo Ersek geschrieben: On 03/10/14 23:44, Max Reitz wrote: Before dereferencing bs->drv for a call to its member bdrv_co_readv(), copy_sectors() should check whether that pointer is indeed valid, since it may have been set to

Re: [Qemu-devel] [PATCH 1/3] qcow2: Check bs->drv in copy_sectors()

2014-03-11 Thread Max Reitz
On 11.03.2014 15:04, Paolo Bonzini wrote: Il 11/03/2014 11:16, Kevin Wolf ha scritto: Am 11.03.2014 um 00:16 hat Laszlo Ersek geschrieben: On 03/10/14 23:44, Max Reitz wrote: Before dereferencing bs->drv for a call to its member bdrv_co_readv(), copy_sectors() should check whether that pointer

Re: [Qemu-devel] [PATCH 1/3] qcow2: Check bs->drv in copy_sectors()

2014-03-11 Thread Paolo Bonzini
Il 11/03/2014 11:16, Kevin Wolf ha scritto: Am 11.03.2014 um 00:16 hat Laszlo Ersek geschrieben: On 03/10/14 23:44, Max Reitz wrote: Before dereferencing bs->drv for a call to its member bdrv_co_readv(), copy_sectors() should check whether that pointer is indeed valid, since it may have been se

Re: [Qemu-devel] [PATCH 1/3] qcow2: Check bs->drv in copy_sectors()

2014-03-11 Thread Kevin Wolf
Am 11.03.2014 um 00:16 hat Laszlo Ersek geschrieben: > On 03/10/14 23:44, Max Reitz wrote: > > Before dereferencing bs->drv for a call to its member bdrv_co_readv(), > > copy_sectors() should check whether that pointer is indeed valid, since > > it may have been set to NULL by e.g. a concurrent wri

Re: [Qemu-devel] [PATCH 1/3] qcow2: Check bs->drv in copy_sectors()

2014-03-10 Thread Laszlo Ersek
On 03/10/14 23:44, Max Reitz wrote: > Before dereferencing bs->drv for a call to its member bdrv_co_readv(), > copy_sectors() should check whether that pointer is indeed valid, since > it may have been set to NULL by e.g. a concurrent write triggering the > corruption prevention mechanism. > > Sig

[Qemu-devel] [PATCH 1/3] qcow2: Check bs->drv in copy_sectors()

2014-03-10 Thread Max Reitz
Before dereferencing bs->drv for a call to its member bdrv_co_readv(), copy_sectors() should check whether that pointer is indeed valid, since it may have been set to NULL by e.g. a concurrent write triggering the corruption prevention mechanism. Signed-off-by: Max Reitz --- To be precise, this s