On 1/9/20 12:21 PM, Peter Krempa wrote:
If a disk is unplugged and then the user tries to delete a checkpoint
the code would try to use NULL node name as it was not checked.

Fix this by fetching the whole disk definition object and verifying it
was found.

Signed-off-by: Peter Krempa <pkre...@redhat.com>
---
  src/qemu/qemu_checkpoint.c | 23 +++++++++++++++++------
  1 file changed, 17 insertions(+), 6 deletions(-)


Reviewed-by: Eric Blake <ebl...@redhat.com>

But it also makes me wonder if the act of hot-unplug should update the definition of existing checkpoints. (Doesn't stop this patch from being useful as-is, but may point to further design work and future patches)

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

Reply via email to