Fix "make check" run requesting authentication

This patch fixes the 'make check' runs for me which, under certain
circumstances and login configurations, did invoke popups requesting
authentication. I removed the parameter conn from being passed into the
error reporting function.

* src/conf/nwfilter_conf.h src/conf/nwfilter_conf.c: remove conn from
  error reporting parameters.

diff --git a/src/conf/nwfilter_conf.c b/src/conf/nwfilter_conf.c
index e82aae69cc054526ee4a6e23b24aeb51a09fd541..668918d6cdc5278d4d2f83c46b99f45514eefd26 100644 (file)
--- a/src/conf/nwfilter_conf.c
+++ b/src/conf/nwfilter_conf.c
@@ -44,9 +44,6 @@
 
 #define VIR_FROM_THIS VIR_FROM_NWFILTER
 
-#define virNWFilterError(conn, code, fmt...)                             \
-            virReportErrorHelper(conn, VIR_FROM_NWFILTER, code, __FILE__,\
-                                  __FUNCTION__, __LINE__, fmt)
 
 VIR_ENUM_IMPL(virNWFilterRuleAction, VIR_NWFILTER_RULE_ACTION_LAST,
               "drop",
@@ -2248,9 +2245,9 @@ virNWFilterPoolObjLoad(virConnectPtr conn,
     }
 
     if (!virFileMatchesNameSuffix(file, def->name, ".xml")) {
-        virNWFilterError(conn, VIR_ERR_INVALID_NWFILTER,
-            "NWFilter pool config filename '%s' does not match pool name '%s'",
-                      path, def->name);
+        virNWFilterReportError(conn, VIR_ERR_INVALID_NWFILTER,
+            _("network filter pool config filename '%s' does not match pool name '%s'"),
+            path, def->name);
         virNWFilterDefFree(def);
         return NULL;
     }
@@ -2300,9 +2297,9 @@ virNWFilterPoolLoadAllConfigs(virConnectPtr conn,
 
         if (virFileBuildPath(configDir, entry->d_name,
                              NULL, path, PATH_MAX) < 0) {
-            virNWFilterError(conn, VIR_ERR_INTERNAL_ERROR,
-                            "Config filename '%s/%s' is too long",
-                            configDir, entry->d_name);
+            virNWFilterReportError(conn, VIR_ERR_INTERNAL_ERROR,
+                                   _("config filename '%s/%s' is too long"),
+                                   configDir, entry->d_name);
             continue;
         }
 

diff --git a/src/conf/nwfilter_conf.h b/src/conf/nwfilter_conf.h
index 9b6ca409734040830851c457138892dd899848ef..8fafc336943e7af1e29cc447cb442becb0af4737 100644 (file)
--- a/src/conf/nwfilter_conf.h
+++ b/src/conf/nwfilter_conf.h
@@ -523,7 +523,8 @@ int virNWFilterParamConfLayerInit(void);
 void virNWFilterParamConfLayerShutdown(void);
 
 # define virNWFilterReportError(conn, code, fmt...)                          \
-        virReportErrorHelper(conn, VIR_FROM_NWFILTER, code, __FILE__,       \
+        (void)conn;                                                          \
+        virReportErrorHelper(NULL, VIR_FROM_NWFILTER, code, __FILE__,  \
                                __FUNCTION__, __LINE__, fmt)