commit d0b896a7495b71313c43aff4329714ade56799d4
author Kostiantyn Kostiuk <konstantin@daynix.com> Tue Aug 03 09:38:39 2021 +0000
committer Kostiantyn Kostiuk <konstantin@daynix.com> Mon Jan 10 13:03:38 2022 +0000
tree e8063e1f9727748dd4fbf8863fa8e49f851b95de
parent cced3a72300c3b077be3bef7048688b9cf6f98c3

gqa-win: get_pci_info: Clean dev_info if handle is valid

Signed-off-by: Kostiantyn Kostiuk <konstantin@daynix.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Signed-off-by: Kostiantyn Kostiuk <kkostiuk@redhat.com>

qga/commands-win32.c

diff --git a/qga/commands-win32.c b/qga/commands-win32.c
index 4e84afd..3dd74fe 100644
--- a/qga/commands-win32.c
+++ b/qga/commands-win32.c
@@ -514,7 +514,7 @@
 
 static GuestPCIAddress *get_pci_info(int number, Error **errp)
 {
-    HDEVINFO dev_info;
+    HDEVINFO dev_info = INVALID_HANDLE_VALUE;
     SP_DEVINFO_DATA dev_info_data;
     SP_DEVICE_INTERFACE_DATA dev_iface_data;
     HANDLE dev_file;
@@ -749,7 +749,9 @@
     }
 
 free_dev_info:
-    SetupDiDestroyDeviceInfoList(dev_info);
+    if (dev_info != INVALID_HANDLE_VALUE) {
+        SetupDiDestroyDeviceInfoList(dev_info);
+    }
 out:
     return pci;
 }