hyperv: use g_autoptr for Msvm_ProcessorSettingData in hypervConnectGetMaxVcpus

Signed-off-by: Matt Coleman <matt@datto.com>
Reviewed-by: Laine Stump <laine@redhat.com>

diff --git a/src/hyperv/hyperv_driver.c b/src/hyperv/hyperv_driver.c
index 8b59dd05f7fb8ae9ec8e9a144f157d707c16cd87..6375f6b011772157eb49c4d2bc4ae34cd9d351b0 100644 (file)
--- a/src/hyperv/hyperv_driver.c
+++ b/src/hyperv/hyperv_driver.c
@@ -1514,10 +1514,9 @@ hypervConnectGetCapabilities(virConnectPtr conn)
 static int
 hypervConnectGetMaxVcpus(virConnectPtr conn, const char *type G_GNUC_UNUSED)
 {
-    int result = -1;
     hypervPrivate *priv = conn->privateData;
     g_auto(virBuffer) query = VIR_BUFFER_INITIALIZER;
-    Msvm_ProcessorSettingData *processorSettingData = NULL;
+    g_autoptr(Msvm_ProcessorSettingData) processorSettingData = NULL;
 
     /* Get max processors definition */
     virBufferAddLit(&query,
@@ -1525,21 +1524,16 @@ hypervConnectGetMaxVcpus(virConnectPtr conn, const char *type G_GNUC_UNUSED)
                     "WHERE InstanceID LIKE 'Microsoft:Definition%Maximum'");
 
     if (hypervGetWmiClass(Msvm_ProcessorSettingData, &processorSettingData) < 0)
-        goto cleanup;
+        return -1;
 
     if (!processorSettingData) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
                        _("Could not get maximum definition of Msvm_ProcessorSettingData for host %s"),
                        conn->uri->server);
-        goto cleanup;
+        return -1;
     }
 
-    result = processorSettingData->data->VirtualQuantity;
-
- cleanup:
-    hypervFreeObject((hypervObject *)processorSettingData);
-
-    return result;
+    return processorSettingData->data->VirtualQuantity;
 }