[libvirt] [PATCH 2/8] virHostdevGetPCIHostDeviceList: Use VIR_AUTOPTR for virPCIDevice

Michal Privoznik posted 8 patches 6 years, 7 months ago
[libvirt] [PATCH 2/8] virHostdevGetPCIHostDeviceList: Use VIR_AUTOPTR for virPCIDevice
Posted by Michal Privoznik 6 years, 7 months ago
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
---
 src/util/virhostdev.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/util/virhostdev.c b/src/util/virhostdev.c
index 9c63d0aebd..5935d926aa 100644
--- a/src/util/virhostdev.c
+++ b/src/util/virhostdev.c
@@ -227,7 +227,7 @@ virHostdevGetPCIHostDeviceList(virDomainHostdevDefPtr *hostdevs, int nhostdevs)
     for (i = 0; i < nhostdevs; i++) {
         virDomainHostdevDefPtr hostdev = hostdevs[i];
         virDomainHostdevSubsysPCIPtr pcisrc = &hostdev->source.subsys.u.pci;
-        virPCIDevicePtr pci;
+        VIR_AUTOPTR(virPCIDevice) pci = NULL;
 
         if (hostdev->mode != VIR_DOMAIN_HOSTDEV_MODE_SUBSYS)
             continue;
@@ -251,10 +251,10 @@ virHostdevGetPCIHostDeviceList(virDomainHostdevDefPtr *hostdevs, int nhostdevs)
             virPCIDeviceSetStubDriver(pci, VIR_PCI_STUB_DRIVER_KVM);
 
         if (virPCIDeviceListAdd(pcidevs, pci) < 0) {
-            virPCIDeviceFree(pci);
             virObjectUnref(pcidevs);
             return NULL;
         }
+        pci = NULL;
     }
 
     return pcidevs;
-- 
2.21.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 2/8] virHostdevGetPCIHostDeviceList: Use VIR_AUTOPTR for virPCIDevice
Posted by Ján Tomko 6 years, 7 months ago
On Tue, Jun 18, 2019 at 03:04:21PM +0200, Michal Privoznik wrote:
>Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
>---
> src/util/virhostdev.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

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