[PATCH 10/24] hostdev: Remove explicit NULL checks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



NULL checks are performed implicitly in the rest of the module,
including other allocations in the very same function.
---
 src/util/virhostdev.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/util/virhostdev.c b/src/util/virhostdev.c
index 48a44bc..098207e 100644
--- a/src/util/virhostdev.c
+++ b/src/util/virhostdev.c
@@ -142,16 +142,16 @@ virHostdevManagerNew(void)
     if (!(hostdevMgr = virObjectNew(virHostdevManagerClass)))
         return NULL;
 
-    if ((hostdevMgr->activePCIHostdevs = virPCIDeviceListNew()) == NULL)
+    if (!(hostdevMgr->activePCIHostdevs = virPCIDeviceListNew()))
         goto error;
 
-    if ((hostdevMgr->activeUSBHostdevs = virUSBDeviceListNew()) == NULL)
+    if (!(hostdevMgr->activeUSBHostdevs = virUSBDeviceListNew()))
         goto error;
 
-    if ((hostdevMgr->inactivePCIHostdevs = virPCIDeviceListNew()) == NULL)
+    if (!(hostdevMgr->inactivePCIHostdevs = virPCIDeviceListNew()))
         goto error;
 
-    if ((hostdevMgr->activeSCSIHostdevs = virSCSIDeviceListNew()) == NULL)
+    if (!(hostdevMgr->activeSCSIHostdevs = virSCSIDeviceListNew()))
         goto error;
 
     if (privileged) {
-- 
2.5.0

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



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]