Re: [Qemu-devel] [PATCH 3/3] block: Catch !bs->drv in bdrv_check()

2014-08-08 Thread Max Reitz
On 08.08.2014 23:11, Max Reitz wrote: On 08.08.2014 11:15, Kevin Wolf wrote: Am 07.08.2014 um 22:47 hat Max Reitz geschrieben: qemu-img check calls bdrv_check() twice if the first run repaired some inconsistencies. If the first run however again triggered corruption prevention (on qcow2) due to

Re: [Qemu-devel] [PATCH 3/3] block: Catch !bs->drv in bdrv_check()

2014-08-08 Thread Max Reitz
On 08.08.2014 11:15, Kevin Wolf wrote: Am 07.08.2014 um 22:47 hat Max Reitz geschrieben: qemu-img check calls bdrv_check() twice if the first run repaired some inconsistencies. If the first run however again triggered corruption prevention (on qcow2) due to very bad inconsistencies, bs->drv may

Re: [Qemu-devel] [PATCH 3/3] block: Catch !bs->drv in bdrv_check()

2014-08-08 Thread Kevin Wolf
Am 07.08.2014 um 22:47 hat Max Reitz geschrieben: > qemu-img check calls bdrv_check() twice if the first run repaired some > inconsistencies. If the first run however again triggered corruption > prevention (on qcow2) due to very bad inconsistencies, bs->drv may be > NULL afterwards. Thus, bdrv_che

[Qemu-devel] [PATCH 3/3] block: Catch !bs->drv in bdrv_check()

2014-08-07 Thread Max Reitz
qemu-img check calls bdrv_check() twice if the first run repaired some inconsistencies. If the first run however again triggered corruption prevention (on qcow2) due to very bad inconsistencies, bs->drv may be NULL afterwards. Thus, bdrv_check() should check whether bs->drv is set. Signed-off-by: