[PATCH 3/6] virPCIDeviceBindToStub: Remove unused @oldDriverPath and @oldDriverName

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

 



These two chunks had to be part of df4283a55bf. But for some unclear
reason, the weren't. Anyway, these two variables are not used anywhere
within function. They're initialized to NULL and then VIR_FREE()-d. And
there's no reason do do two NOPs, right?

Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
---
 src/util/virpci.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/util/virpci.c b/src/util/virpci.c
index a41a22b..8a3a492 100644
--- a/src/util/virpci.c
+++ b/src/util/virpci.c
@@ -1085,8 +1085,6 @@ virPCIDeviceBindToStub(virPCIDevicePtr dev,
 
     char *stubDriverPath = NULL;
     char *driverLink = NULL;
-    char *oldDriverPath = NULL;
-    char *oldDriverName = NULL;
     char *path = NULL; /* reused for different purposes */
     const char *newDriverName = NULL;
 
@@ -1217,8 +1215,6 @@ remove_id:
 cleanup:
     VIR_FREE(stubDriverPath);
     VIR_FREE(driverLink);
-    VIR_FREE(oldDriverPath);
-    VIR_FREE(oldDriverName);
     VIR_FREE(path);
 
     if (newDriverName &&
-- 
1.8.3.2

--
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]