Re: [Qemu-devel] [PATCH 1/2] block: Set backing_hd to NULL after deleting it

2010-04-17 Thread Stefan Hajnoczi
On Sat, Apr 17, 2010 at 10:49 AM, Stefan Hajnoczi wrote: > This patch should be applied after Kevin Wolf's "vmdk: Convert to > bdrv_open" so that vmdk does not segfault on close. This is a mistake, vmdk will check backing_hd != NULL so it is safe. Stefan

[Qemu-devel] [PATCH 1/2] block: Set backing_hd to NULL after deleting it

2010-04-17 Thread Stefan Hajnoczi
It is safer to set backing_hd to NULL after deleting it so that any use after deletion is obvious during development. Happy segfaulting! This patch should be applied after Kevin Wolf's "vmdk: Convert to bdrv_open" so that vmdk does not segfault on close. Signed-off-by: Stefan Hajnoczi --- bloc