From: Michal Privoznik <mprivozn@redhat.com>
Date: Mon, 7 Feb 2022 14:32:48 +0000 (+0100)
Subject: vbox_common: Drop needless set to NULL
X-Git-Tag: v8.1.0-rc1~158
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=1688d2527f1ae34d57d0b5c01fbea79e40dac7b7;p=thirdparty%2Flibvirt.git

vbox_common: Drop needless set to NULL

There are two places where a variable passed to VBOX_RELEASE()
macro is set to NULL explicitly. There is no need for that
because the macro sets the variable to NULL already.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
---

diff --git a/src/vbox/vbox_common.c b/src/vbox/vbox_common.c
index eec47a02fc..db54eacb3b 100644
--- a/src/vbox/vbox_common.c
+++ b/src/vbox/vbox_common.c
@@ -2808,7 +2808,6 @@ static int vboxDomainGetInfo(virDomainPtr dom, virDomainInfoPtr info)
             if (systemProperties) {
                 gVBoxAPI.UISystemProperties.GetMaxGuestRAM(systemProperties, &maxMemorySize);
                 VBOX_RELEASE(systemProperties);
-                systemProperties = NULL;
             }
 
             gVBoxAPI.UIMachine.GetCPUCount(machine, &CPUCount);
@@ -4015,7 +4014,6 @@ static char *vboxDomainGetXMLDesc(virDomainPtr dom, unsigned int flags)
         gVBoxAPI.UISystemProperties.GetSerialPortCount(systemProperties, &serialPortCount);
         gVBoxAPI.UISystemProperties.GetParallelPortCount(systemProperties, &parallelPortCount);
         VBOX_RELEASE(systemProperties);
-        systemProperties = NULL;
     }
     /* Currently setting memory and maxMemory as same, cause
      * the notation here seems to be inconsistent while