A post push realization that the setting of the boolean needed to be inside the if condition. Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx> --- src/qemu/qemu_hotplug.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index 5e18d34..aee62bb 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -382,8 +382,8 @@ qemuDomainAttachVirtioDiskDevice(virConnectPtr conn, secobjProps = NULL; /* qemuMonitorAddObject consumes */ if (rv < 0) goto exit_monitor; + secobjAdded = true; } - secobjAdded = true; if (encobjProps) { rv = qemuMonitorAddObject(priv->mon, "secret", encinfo->s.aes.alias, @@ -391,8 +391,8 @@ qemuDomainAttachVirtioDiskDevice(virConnectPtr conn, encobjProps = NULL; /* qemuMonitorAddObject consumes */ if (rv < 0) goto exit_monitor; + encobjAdded = true; } - encobjAdded = true; if (qemuMonitorAddDrive(priv->mon, drivestr) < 0) goto exit_monitor; -- 2.5.5 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list