[libvirt] [PATCH] qemuDomainSnapshotPrepare: Don't always assume vm->def->os.loader

Peter Krempa pkrempa at redhat.com
Thu Mar 30 12:11:17 UTC 2017


On Thu, Mar 30, 2017 at 14:06:22 +0200, Michal Privoznik wrote:
> In 9e2465834 a check that denies internal snapshots when pflash
> based loader is configured for the domain. However, if there's
> none and an user tries to do an internal snapshot they will
> witness daemon crash as in that case vm->def->os.loader is NULL
> and we dereference it unconditionally.
> 
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
>  src/qemu/qemu_driver.c | 1 +
>  1 file changed, 1 insertion(+)

OOPS, ACK, safe for freeze.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170330/d537d4cf/attachment-0001.sig>


More information about the libvir-list mailing list