[PATCH 03/19] qemu_firmware: only set nfeatures on success

[marcandre.lureau@xxxxxxxxxx]

From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>

The field is set just before returning on success.

Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
---
 src/qemu/qemu_firmware.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/qemu/qemu_firmware.c b/src/qemu/qemu_firmware.c
index f3d89d000d..bf29b10b9a 100644
--- a/src/qemu/qemu_firmware.c
+++ b/src/qemu/qemu_firmware.c
@@ -592,8 +592,6 @@ qemuFirmwareFeatureParse(const char *path,
     if (VIR_ALLOC_N(features, nfeatures) < 0)
         return -1;
 
-    fw->nfeatures = nfeatures;
-
     for (i = 0; i < nfeatures; i++) {
         virJSONValuePtr item = virJSONValueArrayGet(featuresJSON, i);
         const char *tmpStr = virJSONValueGetString(item);
-- 
2.22.0.rc2.384.g1a9a72ea1d

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list