[libvirt] [PATCH 2/3] nwfilter: Remove unprivileged code path to set base

John Ferlan jferlan at redhat.com
Tue Jan 8 21:21:50 UTC 2013


Commit id f8ab364c removed ability to run this driver unprivileged. Coverity
detected the check and flagged it.
---
 src/nwfilter/nwfilter_driver.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/src/nwfilter/nwfilter_driver.c b/src/nwfilter/nwfilter_driver.c
index 1ac91cf..815f588 100644
--- a/src/nwfilter/nwfilter_driver.c
+++ b/src/nwfilter/nwfilter_driver.c
@@ -219,14 +219,8 @@ nwfilterDriverStartup(bool privileged ATTRIBUTE_UNUSED,
         goto error;
     }
 
-    if (privileged) {
-        if ((base = strdup(SYSCONFDIR "/libvirt")) == NULL)
-            goto out_of_memory;
-    } else {
-        base = virGetUserConfigDirectory();
-        if (!base)
-            goto error;
-    }
+    if ((base = strdup(SYSCONFDIR "/libvirt")) == NULL)
+        goto out_of_memory;
 
     if (virAsprintf(&driverState->configDir,
                     "%s/nwfilter", base) == -1)
-- 
1.7.11.7




More information about the libvir-list mailing list