From a593675594f2f1a737b9ec9fc5e51bd7831da013 Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Wed, 13 Nov 2013 14:18:09 +0100 Subject: [PATCH] virPCIDeviceBindToStub: Remove unused @oldDriverPath and @oldDriverName 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 --- src/util/virpci.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/util/virpci.c b/src/util/virpci.c index a41a22b664..8a3a492136 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 && -- 2.47.2