Remove virConnectPtr from virRaiseErrorFull

And from all related macros and functions.

diff --git a/src/conf/cpu_conf.c b/src/conf/cpu_conf.c
index 7f03eaa8a58680ce4effb5a790b307d95bfbf20a..ad49916a9d8ab0e9531e8e525975d64acf6501c1 100644 (file)
--- a/src/conf/cpu_conf.c
+++ b/src/conf/cpu_conf.c
@@ -32,7 +32,7 @@
 #define VIR_FROM_THIS VIR_FROM_CPU
 
 #define virCPUReportError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_CPU, code, __FILE__,      \
+    virReportErrorHelper(VIR_FROM_CPU, code, __FILE__,            \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 VIR_ENUM_IMPL(virCPUMatch, VIR_CPU_MATCH_LAST,

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index c827ef564a458594ebd5226f84159ffa34975f18..6b733d474b9ded8c1fe482df936a77e199bdaf0b 100644 (file)
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -415,7 +415,7 @@ VIR_ENUM_IMPL(virDomainTimerMode, VIR_DOMAIN_TIMER_MODE_LAST,
               "smpsafe");
 
 #define virDomainReportError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_DOMAIN, code, __FILE__,      \
+    virReportErrorHelper(VIR_FROM_DOMAIN, code, __FILE__,            \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #define VIR_DOMAIN_XML_WRITE_FLAGS  VIR_DOMAIN_XML_SECURE

diff --git a/src/conf/domain_event.c b/src/conf/domain_event.c
index 5f086bd224a6263147da47b3ebca40b6f38f71ad..f0380d3bb6c6739a4486235f5b0fab2779510cee 100644 (file)
--- a/src/conf/domain_event.c
+++ b/src/conf/domain_event.c
@@ -32,7 +32,7 @@
 #define VIR_FROM_THIS VIR_FROM_NONE
 
 #define eventReportError(code, ...)                                 \
-    virReportErrorHelper(NULL, VIR_FROM_THIS, code, __FILE__,       \
+    virReportErrorHelper(VIR_FROM_THIS, code, __FILE__,             \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 struct _virDomainMeta {

diff --git a/src/conf/interface_conf.c b/src/conf/interface_conf.c
index b24526fc2126b987920b46c613f0c9a52ec74fc7..2fa2fa06045062a7f52f1396d2ced4afd3341fed 100644 (file)
--- a/src/conf/interface_conf.c
+++ b/src/conf/interface_conf.c
@@ -46,7 +46,7 @@ virInterfaceDefDevFormat(virBufferPtr buf,
                          const virInterfaceDefPtr def, int level);
 
 #define virInterfaceReportError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_INTERFACE, code, __FILE__,      \
+    virReportErrorHelper(VIR_FROM_INTERFACE, code, __FILE__,            \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 static

diff --git a/src/conf/network_conf.c b/src/conf/network_conf.c
index dcab9de21dbf477d4b5de65dbfa001b0fe4ac516..5738757cbc9edd8d2007604b72293d8e9c853a9a 100644 (file)
--- a/src/conf/network_conf.c
+++ b/src/conf/network_conf.c
@@ -53,7 +53,7 @@ VIR_ENUM_IMPL(virNetworkForward,
               "none", "nat", "route" )
 
 #define virNetworkReportError(code, ...)                                \
-    virReportErrorHelper(NULL, VIR_FROM_NETWORK, code, __FILE__,        \
+    virReportErrorHelper(VIR_FROM_NETWORK, code, __FILE__,              \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 virNetworkObjPtr virNetworkFindByUUID(const virNetworkObjListPtr nets,

diff --git a/src/conf/node_device_conf.h b/src/conf/node_device_conf.h
index fe0f2bffb030dd32b868f78c3c4d200315bf5e34..975abb3345c6442a08b91f3fcb1f97e3f1140365 100644 (file)
--- a/src/conf/node_device_conf.h
+++ b/src/conf/node_device_conf.h
@@ -219,7 +219,7 @@ struct _virDeviceMonitorState {
 };
 
 # define virNodeDeviceReportError(code, ...)                             \
-    virReportErrorHelper(NULL, VIR_FROM_NODEDEV, code, __FILE__,       \
+    virReportErrorHelper(VIR_FROM_NODEDEV, code, __FILE__,               \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 int virNodeDeviceHasCap(const virNodeDeviceObjPtr dev, const char *cap);

diff --git a/src/conf/nwfilter_conf.h b/src/conf/nwfilter_conf.h
index 9281f567be3e055636fed1d464963b718d8c4bc5..12e2a5c3881d3f6c18a2938802b92300b04c834c 100644 (file)
--- a/src/conf/nwfilter_conf.h
+++ b/src/conf/nwfilter_conf.h
@@ -647,9 +647,9 @@ void virNWFilterUnlockFilterUpdates(void);
 int virNWFilterConfLayerInit(virHashIterator domUpdateCB);
 void virNWFilterConfLayerShutdown(void);
 
-# define virNWFilterReportError(code, fmt...)                          \
-        virReportErrorHelper(NULL, VIR_FROM_NWFILTER, code, __FILE__,  \
-                               __FUNCTION__, __LINE__, fmt)
+# define virNWFilterReportError(code, fmt...)                      \
+        virReportErrorHelper(VIR_FROM_NWFILTER, code, __FILE__,    \
+                             __FUNCTION__, __LINE__, fmt)
 
 
 typedef int (*virNWFilterRebuild)(virConnectPtr conn,

diff --git a/src/conf/secret_conf.h b/src/conf/secret_conf.h
index 66294cee32eb6ecbfb479412926f3007f06bdb2e..4b47c527dfdc002eb936e70eb1bb636e3ecdd7a4 100644 (file)
--- a/src/conf/secret_conf.h
+++ b/src/conf/secret_conf.h
@@ -27,7 +27,7 @@
 # include "util.h"
 
 # define virSecretReportError(code, ...)                         \
-    virReportErrorHelper(NULL, VIR_FROM_SECRET, code, __FILE__, \
+    virReportErrorHelper(VIR_FROM_SECRET, code, __FILE__,        \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 VIR_ENUM_DECL(virSecretUsageType)

diff --git a/src/conf/storage_conf.h b/src/conf/storage_conf.h
index 9e1f534f8d3d5371aafdb0f39dd75c4593a0ff5b..271441a9559ace9e16f583ceb65128ca46a04108 100644 (file)
--- a/src/conf/storage_conf.h
+++ b/src/conf/storage_conf.h
@@ -324,7 +324,7 @@ static inline int virStoragePoolObjIsActive(virStoragePoolObjPtr pool) {
 }
 
 # define virStorageReportError(code, ...)                                \
-    virReportErrorHelper(NULL, VIR_FROM_STORAGE, code, __FILE__,        \
+    virReportErrorHelper(VIR_FROM_STORAGE, code, __FILE__,               \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 int virStoragePoolLoadAllConfigs(virStoragePoolObjListPtr pools,

diff --git a/src/cpu/cpu.h b/src/cpu/cpu.h
index 76d0e8e2e6ae43705d5f45cb60e97f418eb9cb77..9f01f17cde067b637d5551f9ba38ebcbdd9dd1e6 100644 (file)
--- a/src/cpu/cpu.h
+++ b/src/cpu/cpu.h
@@ -31,7 +31,7 @@
 
 
 # define virCPUReportError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_CPU, code, __FILE__,      \
+    virReportErrorHelper(VIR_FROM_CPU, code, __FILE__,             \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 

diff --git a/src/datatypes.c b/src/datatypes.c
index 3edd1ff45739682844f39811ba29f4d8ad8296bc..4c4cbd02393d90c6d6ffb10b0189efa9e6679179 100644 (file)
--- a/src/datatypes.c
+++ b/src/datatypes.c
@@ -32,7 +32,7 @@
 #define VIR_FROM_THIS VIR_FROM_NONE
 
 #define virLibConnError(code, ...)                                \
-    virReportErrorHelper(NULL, VIR_FROM_THIS, code, __FILE__,     \
+    virReportErrorHelper(VIR_FROM_THIS, code, __FILE__,           \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /************************************************************************

diff --git a/src/esx/esx_private.h b/src/esx/esx_private.h
index 1da25528727ae43e5854643d9e1c8b512aac7731..7ce237e533031cda81b99474c5eec23c76c4e697 100644 (file)
--- a/src/esx/esx_private.h
+++ b/src/esx/esx_private.h
@@ -29,7 +29,7 @@
 # include "esx_vi.h"
 
 # define ESX_ERROR(code, ...)                                                 \
-    virReportErrorHelper(NULL, VIR_FROM_ESX, code, __FILE__, __FUNCTION__,    \
+    virReportErrorHelper(VIR_FROM_ESX, code, __FILE__, __FUNCTION__,          \
                          __LINE__, __VA_ARGS__)
 
 typedef struct _esxPrivate {

diff --git a/src/esx/esx_vi.h b/src/esx/esx_vi.h
index d046bf98a28a0035c3dae7f31b262823530a2bfd..7d420da720d52c29c818ddc7e4c8add10a94582c 100644 (file)
--- a/src/esx/esx_vi.h
+++ b/src/esx/esx_vi.h
@@ -36,7 +36,7 @@
 
 
 # define ESX_VI_ERROR(code, ...)                                              \
-    virReportErrorHelper(NULL, VIR_FROM_ESX, code, __FILE__, __FUNCTION__,    \
+    virReportErrorHelper(VIR_FROM_ESX, code, __FILE__, __FUNCTION__,          \
                          __LINE__, __VA_ARGS__)
 
 

diff --git a/src/fdstream.c b/src/fdstream.c
index 3475bfdcb6df0d46cc256159d961e4c0825d9798..d2325aea55642bd7928a9ab13174380828292da9 100644 (file)
--- a/src/fdstream.c
+++ b/src/fdstream.c
@@ -45,7 +45,7 @@
 
 #define VIR_FROM_THIS VIR_FROM_STREAMS
 #define streamsReportError(code, ...)                                \
-    virReportErrorHelper(NULL, VIR_FROM_THIS, code, __FILE__,        \
+    virReportErrorHelper(VIR_FROM_THIS, code, __FILE__,              \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /* Tunnelled migration stream support */

diff --git a/src/interface/netcf_driver.c b/src/interface/netcf_driver.c
index 0217b90b3112566dffc691913039198db8e8ba3c..0190bf4e12d78da28af96e08b8c6a0cb9117e115 100644 (file)
--- a/src/interface/netcf_driver.c
+++ b/src/interface/netcf_driver.c
@@ -34,7 +34,7 @@
 #define VIR_FROM_THIS VIR_FROM_INTERFACE
 
 #define interfaceReportError(code, ...)                               \
-    virReportErrorHelper(NULL, VIR_FROM_THIS, code, __FILE__,         \
+    virReportErrorHelper(VIR_FROM_THIS, code, __FILE__,               \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /* Main driver state */

diff --git a/src/internal.h b/src/internal.h
index 4641fc154015af5cb5c0012762f55842e5e14911..0fa097c7dcf084d483ffb675c2346fbb6984480d 100644 (file)
--- a/src/internal.h
+++ b/src/internal.h
@@ -232,8 +232,7 @@
     do {                                                                \
         unsigned long __unsuppflags = flags & ~(supported);             \
         if (__unsuppflags) {                                            \
-            virReportErrorHelper(NULL,                                  \
-                                 VIR_FROM_THIS,                         \
+            virReportErrorHelper(VIR_FROM_THIS,                         \
                                  VIR_ERR_INVALID_ARG,                   \
                                  __FILE__,                              \
                                  __FUNCTION__,                          \

diff --git a/src/libvirt-qemu.c b/src/libvirt-qemu.c
index d914ac87c5c5b1787520e07cf0c94db3ac68d395..46727c843c3db983f1db4a0c3eda9629c8772b26 100644 (file)
--- a/src/libvirt-qemu.c
+++ b/src/libvirt-qemu.c
@@ -29,11 +29,11 @@
 #include "libvirt/libvirt-qemu.h"
 
 #define virLibConnError(conn, error, info)                               \
-    virReportErrorHelper(conn, VIR_FROM_NONE, error, NULL, __FUNCTION__, \
+    virReportErrorHelper(VIR_FROM_NONE, error, NULL, __FUNCTION__,       \
                          __LINE__, info)
 
 #define virLibDomainError(domain, error, info)                          \
-    virReportErrorHelper(NULL, VIR_FROM_DOM, error, NULL, __FUNCTION__, \
+    virReportErrorHelper(VIR_FROM_DOM, error, NULL, __FUNCTION__,       \
                          __LINE__, info)
 
 int

diff --git a/src/libvirt.c b/src/libvirt.c
index 0da98859fc6037bdc6bfa9b97188721a176aeb38..9e6784bcaefdafcdae794d2b79e0b3a088d59a2c 100644 (file)
--- a/src/libvirt.c
+++ b/src/libvirt.c
@@ -469,37 +469,37 @@ DllMain (HINSTANCE instance ATTRIBUTE_UNUSED,
 #endif
 
 #define virLibConnError(code, ...)                                \
-    virReportErrorHelper(NULL, VIR_FROM_NONE, code, __FILE__,     \
+    virReportErrorHelper(VIR_FROM_NONE, code, __FILE__,           \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 #define virLibDomainError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_DOM, code, __FILE__,      \
+    virReportErrorHelper(VIR_FROM_DOM, code, __FILE__,            \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 #define virLibNetworkError(code, ...)                             \
-    virReportErrorHelper(NULL, VIR_FROM_NETWORK, code, __FILE__,  \
+    virReportErrorHelper(VIR_FROM_NETWORK, code, __FILE__,        \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 #define virLibStoragePoolError(code, ...)                         \
-    virReportErrorHelper(NULL, VIR_FROM_STORAGE, code, __FILE__,  \
+    virReportErrorHelper(VIR_FROM_STORAGE, code, __FILE__,        \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 #define virLibStorageVolError(code, ...)                          \
-    virReportErrorHelper(NULL, VIR_FROM_STORAGE, code, __FILE__,  \
+    virReportErrorHelper(VIR_FROM_STORAGE, code, __FILE__,        \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 #define virLibInterfaceError(code, ...)                           \
-    virReportErrorHelper(NULL, VIR_FROM_INTERFACE, code, __FILE__,\
+    virReportErrorHelper(VIR_FROM_INTERFACE, code, __FILE__,      \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 #define virLibNodeDeviceError(code, ...)                          \
-    virReportErrorHelper(NULL, VIR_FROM_NODEDEV, code, __FILE__,  \
+    virReportErrorHelper(VIR_FROM_NODEDEV, code, __FILE__,        \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 #define virLibSecretError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_SECRET, code, __FILE__,   \
+    virReportErrorHelper(VIR_FROM_SECRET, code, __FILE__,         \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 #define virLibStreamError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_STREAMS, code, __FILE__,  \
+    virReportErrorHelper(VIR_FROM_STREAMS, code, __FILE__,        \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 #define virLibNWFilterError(code, ...)                            \
-    virReportErrorHelper(NULL, VIR_FROM_NWFILTER, code, __FILE__, \
+    virReportErrorHelper(VIR_FROM_NWFILTER, code, __FILE__,       \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
-#define virLibDomainSnapshotError(code, ...)                            \
-    virReportErrorHelper(NULL, VIR_FROM_DOMAIN_SNAPSHOT, code, __FILE__, \
+#define virLibDomainSnapshotError(code, ...)                       \
+    virReportErrorHelper(VIR_FROM_DOMAIN_SNAPSHOT, code, __FILE__, \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 
@@ -1056,7 +1056,7 @@ do_open (const char *name,
              STRCASEEQ(ret->uri->scheme, "xenapi") ||
 #endif
              false)) {
-            virReportErrorHelper(NULL, VIR_FROM_NONE, VIR_ERR_INVALID_ARG,
+            virReportErrorHelper(VIR_FROM_NONE, VIR_ERR_INVALID_ARG,
                                  __FILE__, __FUNCTION__, __LINE__,
                                  _("libvirt was built without the '%s' driver"),
                                  ret->uri->scheme);

diff --git a/src/libxl/libxl_conf.h b/src/libxl/libxl_conf.h
index f2f0d8ac0a21d97efbf5e60e918b675bdbed9551..8c877860858382d93db8e79449ea21f8c8e601f1 100644 (file)
--- a/src/libxl/libxl_conf.h
+++ b/src/libxl/libxl_conf.h
@@ -87,7 +87,7 @@ struct _libxlDomainObjPrivate {
 
 
 # define libxlError(code, ...)                                     \
-    virReportErrorHelper(NULL, VIR_FROM_LIBXL, code, __FILE__,     \
+    virReportErrorHelper(VIR_FROM_LIBXL, code, __FILE__,           \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 virCapsPtr

diff --git a/src/lxc/lxc_conf.h b/src/lxc/lxc_conf.h
index f820d6dd603446899692048031defadd57617f70..4f1ead3e1201a62e56bb051c0c546a063eff8f7a 100644 (file)
--- a/src/lxc/lxc_conf.h
+++ b/src/lxc/lxc_conf.h
@@ -66,7 +66,7 @@ int lxcLoadDriverConfig(lxc_driver_t *driver);
 virCapsPtr lxcCapsInit(void);
 
 # define lxcError(code, ...)                                             \
-    virReportErrorHelper(NULL, VIR_FROM_LXC, code, __FILE__,            \
+    virReportErrorHelper(VIR_FROM_LXC, code, __FILE__,                   \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #endif /* LXC_CONF_H */

diff --git a/src/lxc/veth.c b/src/lxc/veth.c
index 26bf4ff88b0447de068cd4b0e90828607da67962..a00aa23da9250dabb87547c6fb5664c476d4aaf5 100644 (file)
--- a/src/lxc/veth.c
+++ b/src/lxc/veth.c
@@ -27,7 +27,7 @@
 #define VIR_FROM_THIS VIR_FROM_LXC
 
 #define vethError(code, ...)                                  \
-    virReportErrorHelper(NULL, VIR_FROM_LXC, code, __FILE__,  \
+    virReportErrorHelper(VIR_FROM_LXC, code, __FILE__,        \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /* Functions */

diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
index f3952d4a29f64c103873a62f83b4ac71e40c99a8..8b5c1b613aa0e310ab7dad9c9086669df8a38fe1 100644 (file)
--- a/src/network/bridge_driver.c
+++ b/src/network/bridge_driver.c
@@ -70,7 +70,7 @@
 #define VIR_FROM_THIS VIR_FROM_NETWORK
 
 #define networkReportError(code, ...)                                   \
-    virReportErrorHelper(NULL, VIR_FROM_NETWORK, code, __FILE__,        \
+    virReportErrorHelper(VIR_FROM_NETWORK, code, __FILE__,              \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /* Main driver state */

diff --git a/src/nodeinfo.c b/src/nodeinfo.c
index e1c9ad33a629671f4f9fa47dcc45d3c3f152de4f..facac150d85596cb66b200407152bc37f6c82e6a 100644 (file)
--- a/src/nodeinfo.c
+++ b/src/nodeinfo.c
@@ -51,7 +51,7 @@
 #define VIR_FROM_THIS VIR_FROM_NONE
 
 #define nodeReportError(code, ...)                                      \
-    virReportErrorHelper(NULL, VIR_FROM_NONE, code, __FILE__,           \
+    virReportErrorHelper(VIR_FROM_NONE, code, __FILE__,                 \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #ifdef __linux__

diff --git a/src/openvz/openvz_conf.h b/src/openvz/openvz_conf.h
index 4673fa6b463345c1bba68982b01a3d77ed4ea834..9a57551e7835abce8037d0c2cfab0fb7ee0e01e1 100644 (file)
--- a/src/openvz/openvz_conf.h
+++ b/src/openvz/openvz_conf.h
@@ -34,7 +34,7 @@
 # include "threads.h"
 
 # define openvzError(code, ...)                                            \
-        virReportErrorHelper(NULL, VIR_FROM_OPENVZ, code, __FILE__,        \
+        virReportErrorHelper(VIR_FROM_OPENVZ, code, __FILE__,              \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 

diff --git a/src/phyp/phyp_driver.c b/src/phyp/phyp_driver.c
index ae2ed7009cfdb0f2267e1f139f26c072383bc482..8f8c3ba6d483a3589694e8bc8a4c0487c63c1aa8 100644 (file)
--- a/src/phyp/phyp_driver.c
+++ b/src/phyp/phyp_driver.c
@@ -65,7 +65,7 @@
 #define VIR_FROM_THIS VIR_FROM_PHYP
 
 #define PHYP_ERROR(code, ...)                                                 \
-    virReportErrorHelper(NULL, VIR_FROM_PHYP, code, __FILE__, __FUNCTION__,   \
+    virReportErrorHelper(VIR_FROM_PHYP, code, __FILE__, __FUNCTION__,         \
                          __LINE__, __VA_ARGS__)
 
 /*

diff --git a/src/qemu/qemu_conf.h b/src/qemu/qemu_conf.h
index 94918f6e214a36367eba6e86800c437a18f93c7e..f2bfa1eebbf31974b7eb9ace70e47f31b51c8033 100644 (file)
--- a/src/qemu/qemu_conf.h
+++ b/src/qemu/qemu_conf.h
@@ -146,7 +146,7 @@ struct _qemuDomainCmdlineDef {
 # define QEMUD_MIGRATION_NUM_PORTS 64
 
 # define qemuReportError(code, ...)                                      \
-    virReportErrorHelper(NULL, VIR_FROM_QEMU, code, __FILE__,           \
+    virReportErrorHelper(VIR_FROM_QEMU, code, __FILE__,                  \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 

diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index b979f71c83a3ab3e0b483762c6cb3bb689d76169..e30780cc938be070eae1e9bd9d1bd07ac38f0bc0 100644 (file)
--- a/src/remote/remote_driver.c
+++ b/src/remote/remote_driver.c
@@ -242,7 +242,7 @@ static int remoteAuthPolkit (virConnectPtr conn, struct private_data *priv, int
 #endif /* HAVE_POLKIT */
 
 #define remoteError(code, ...)                                    \
-    virReportErrorHelper(NULL, VIR_FROM_REMOTE, code, __FILE__,   \
+    virReportErrorHelper(VIR_FROM_REMOTE, code, __FILE__,         \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 static virDomainPtr get_nonnull_domain (virConnectPtr conn, remote_nonnull_domain domain);
@@ -8644,8 +8644,7 @@ remoteStreamHasError(virStreamPtr st) {
     }
 
     VIR_DEBUG0("Raising async error");
-    virRaiseErrorFull(st->conn,
-                      __FILE__, __FUNCTION__, __LINE__,
+    virRaiseErrorFull(__FILE__, __FUNCTION__, __LINE__,
                       privst->err.domain,
                       privst->err.code,
                       privst->err.level,
@@ -10908,8 +10907,7 @@ cleanup:
              * convert missing remote entry points into the unsupported
              * feature error
              */
-            virRaiseErrorFull(flags & REMOTE_CALL_IN_OPEN ? NULL : conn,
-                              __FILE__, __FUNCTION__, __LINE__,
+            virRaiseErrorFull(__FILE__, __FUNCTION__, __LINE__,
                               thiscall->err.domain,
                               VIR_ERR_NO_SUPPORT,
                               thiscall->err.level,
@@ -10921,8 +10919,7 @@ cleanup:
                               "%s", *thiscall->err.message);
             rv = -1;
         } else {
-            virRaiseErrorFull(flags & REMOTE_CALL_IN_OPEN ? NULL : conn,
-                              __FILE__, __FUNCTION__, __LINE__,
+            virRaiseErrorFull(__FILE__, __FUNCTION__, __LINE__,
                               thiscall->err.domain,
                               thiscall->err.code,
                               thiscall->err.level,

diff --git a/src/security/security_manager.h b/src/security/security_manager.h
index 3f88801870a4058437d2c2cbc6e455e23fd259e6..8d7c220b0088e29f89bed4e1a61e31afa34d071c 100644 (file)
--- a/src/security/security_manager.h
+++ b/src/security/security_manager.h
@@ -24,7 +24,7 @@
 # define VIR_SECURITY_MANAGER_H__
 
 # define virSecurityReportError(code, ...)                          \
-    virReportErrorHelper(NULL, VIR_FROM_SECURITY, code, __FILE__,   \
+    virReportErrorHelper(VIR_FROM_SECURITY, code, __FILE__,         \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 

diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 17f5ad9ae7538c5039a6c3c51a5cd18f45e08acb..0978214a6a1f1059d9c2510f832a115ba58bb016 100644 (file)
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -117,7 +117,7 @@ static const virNodeInfo defaultNodeInfo = {
 
 
 #define testError(code, ...)                                      \
-        virReportErrorHelper(NULL, VIR_FROM_TEST, code, __FILE__, \
+        virReportErrorHelper(VIR_FROM_TEST, code, __FILE__,       \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 static int testClose(virConnectPtr conn);

diff --git a/src/uml/uml_conf.h b/src/uml/uml_conf.h
index 64df5f7684104ac01b6bcb1a4e1da32e7830fb70..1105f84080aa38ef7137aa6a129a0176c18098b3 100644 (file)
--- a/src/uml/uml_conf.h
+++ b/src/uml/uml_conf.h
@@ -64,7 +64,7 @@ struct uml_driver {
 
 
 # define umlReportError(code, ...)                                      \
-    virReportErrorHelper(NULL, VIR_FROM_UML, code, __FILE__,            \
+    virReportErrorHelper(VIR_FROM_UML, code, __FILE__,                  \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 virCapsPtr  umlCapsInit               (void);

diff --git a/src/util/command.c b/src/util/command.c
index 2e475a0ac05db8278aaf472211844cb6faa88ec8..862a913c20a5d57bc8b28475e0415bf089b372c0 100644 (file)
--- a/src/util/command.c
+++ b/src/util/command.c
@@ -38,7 +38,7 @@
 #define VIR_FROM_THIS VIR_FROM_NONE
 
 #define virCommandError(code, ...)                                      \
-    virReportErrorHelper(NULL, VIR_FROM_NONE, code, __FILE__,           \
+    virReportErrorHelper(VIR_FROM_NONE, code, __FILE__,                 \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 enum {

diff --git a/src/util/conf.c b/src/util/conf.c
index 71a344fefd7b632e5889a0dba9c8222a1f7cb3f5..4b8afb8f524cd30872af7ab385c3ea181cd532dc 100644 (file)
--- a/src/util/conf.c
+++ b/src/util/conf.c
@@ -100,13 +100,13 @@ virConfError(virConfParserCtxtPtr ctxt,
 
     /* Construct the string 'filename:line: info' if we have that. */
     if (ctxt && ctxt->filename) {
-        virRaiseError(NULL, NULL, NULL, VIR_FROM_CONF, error, VIR_ERR_ERROR,
+        virRaiseError(NULL, NULL, VIR_FROM_CONF, error, VIR_ERR_ERROR,
                         info, ctxt->filename, NULL,
                         ctxt->line, 0,
                         "%s:%d: %s", ctxt->filename, ctxt->line, info);
     } else {
         format = virErrorMsg(error, info);
-        virRaiseError(NULL, NULL, NULL, VIR_FROM_CONF, error, VIR_ERR_ERROR,
+        virRaiseError(NULL, NULL, VIR_FROM_CONF, error, VIR_ERR_ERROR,
                         info, NULL, NULL,
                         ctxt ? ctxt->line : 0, 0,
                         format, info);

diff --git a/src/util/event_poll.c b/src/util/event_poll.c
index cd1ff4a0f9ff0ad93f3594cf125377b927712cb7..c5eedd3d4d6c39363ac0d1f74f0d04b2e1be39f4 100644 (file)
--- a/src/util/event_poll.c
+++ b/src/util/event_poll.c
@@ -44,7 +44,7 @@
 #define VIR_FROM_THIS VIR_FROM_EVENT
 
 #define virEventError(code, ...)                                    \
-    virReportErrorHelper(NULL, VIR_FROM_EVENT, code, __FILE__,      \
+    virReportErrorHelper(VIR_FROM_EVENT, code, __FILE__,            \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 static int virEventPollInterruptLocked(void);

diff --git a/src/util/hooks.c b/src/util/hooks.c
index 819c95cce5babbe5121fb9a322f3acb2ebdf50ff..a409d773e1d40dd8121b04ff7aee2de75d437f9c 100644 (file)
--- a/src/util/hooks.c
+++ b/src/util/hooks.c
@@ -42,7 +42,7 @@
 #define VIR_FROM_THIS VIR_FROM_HOOK
 
 #define virHookReportError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_HOOK, code, __FILE__,      \
+    virReportErrorHelper(VIR_FROM_HOOK, code, __FILE__,            \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #define LIBVIRT_HOOK_DIR SYSCONFDIR "/libvirt/hooks"

diff --git a/src/util/hostusb.c b/src/util/hostusb.c
index 2d6e41482a9bd7792634d849eeb5e0319dde412e..d5b478b70511d27a17caefeaaa26d734f0e7562d 100644 (file)
--- a/src/util/hostusb.c
+++ b/src/util/hostusb.c
@@ -55,7 +55,7 @@ struct _usbDevice {
 #define VIR_FROM_THIS VIR_FROM_NONE
 
 #define usbReportError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_NONE, code, __FILE__,  \
+    virReportErrorHelper(VIR_FROM_NONE, code, __FILE__,        \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 static int usbSysReadFile(const char *f_name, const char *d_name,

diff --git a/src/util/interface.c b/src/util/interface.c
index fe58823be17fb9449cbc15b5ead59be3e5a2eb30..5e1987adc4d3a5c519d33f85cb36f44d2512cfcf 100644 (file)
--- a/src/util/interface.c
+++ b/src/util/interface.c
@@ -42,7 +42,7 @@
 #include "files.h"
 
 #define ifaceError(code, ...) \
-        virReportErrorHelper(NULL, VIR_FROM_NET, code, __FILE__, \
+        virReportErrorHelper(VIR_FROM_NET, code, __FILE__, \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #if __linux__

diff --git a/src/util/iptables.c b/src/util/iptables.c
index 59f5cc7a51c2bbe6d1fd9b538e01523628bd7ef3..76d412c12d96c2710c9def10ade37192feb70396 100644 (file)
--- a/src/util/iptables.c
+++ b/src/util/iptables.c
@@ -46,7 +46,7 @@
 
 #define VIR_FROM_THIS VIR_FROM_NONE
 #define iptablesError(code, ...)                                        \
-    virReportErrorHelper(NULL, VIR_FROM_THIS, code, __FILE__,           \
+    virReportErrorHelper(VIR_FROM_THIS, code, __FILE__,                 \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 enum {

diff --git a/src/util/json.c b/src/util/json.c
index be47f64031f6f68872bb2b3e409ec994e9a8d044..0daeae9092e247701d15b2766813db18471480dc 100644 (file)
--- a/src/util/json.c
+++ b/src/util/json.c
@@ -37,7 +37,7 @@
 /* XXX fixme */
 #define VIR_FROM_THIS VIR_FROM_NONE
 #define virJSONError(code, ...)                                         \
-    virReportErrorHelper(NULL, VIR_FROM_NONE, code, __FILE__,           \
+    virReportErrorHelper(VIR_FROM_NONE, code, __FILE__,                 \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 

diff --git a/src/util/macvtap.c b/src/util/macvtap.c
index 346eaf69c5ac3820c2f6e4310c63b89a5294885f..a7af0cb7817e1b2d921a23e9a91bcd5d6615811e 100644 (file)
--- a/src/util/macvtap.c
+++ b/src/util/macvtap.c
@@ -63,7 +63,7 @@
 # define VIR_FROM_THIS VIR_FROM_NET
 
 # define macvtapError(code, ...)                                           \
-        virReportErrorHelper(NULL, VIR_FROM_NET, code, __FILE__,           \
+        virReportErrorHelper(VIR_FROM_NET, code, __FILE__,                 \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 # define MACVTAP_NAME_PREFIX   "macvtap"

diff --git a/src/util/network.c b/src/util/network.c
index 33028aaaac3f54df4f73bde634ad6f0f7a32d89f..eb16e0c254625fb0d683ffae6a0dab7f7b70ff99 100644 (file)
--- a/src/util/network.c
+++ b/src/util/network.c
@@ -18,7 +18,7 @@
 
 #define VIR_FROM_THIS VIR_FROM_NONE
 #define virSocketError(code, ...)                                       \
-    virReportErrorHelper(NULL, VIR_FROM_THIS, code, __FILE__,           \
+    virReportErrorHelper(VIR_FROM_THIS, code, __FILE__,                 \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /*

diff --git a/src/util/pci.c b/src/util/pci.c
index ff950d15565e8737e458b03d6f6044e6171a4aee..945f32a9b6b52aeb4fa097bb77bc703f0cbdcc31 100644 (file)
--- a/src/util/pci.c
+++ b/src/util/pci.c
@@ -82,7 +82,7 @@ struct _pciDeviceList {
 #define VIR_FROM_THIS VIR_FROM_NONE
 
 #define pciReportError(code, ...)                              \
-    virReportErrorHelper(NULL, VIR_FROM_NONE, code, __FILE__,  \
+    virReportErrorHelper(VIR_FROM_NONE, code, __FILE__,        \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /* Specifications referenced in comments:

diff --git a/src/util/sexpr.c b/src/util/sexpr.c
index da3d4b3b3a35590873f197ae7a717820b729de3f..d6668f87f8fe15b5613fa9926d59e4541140b4c1 100644 (file)
--- a/src/util/sexpr.c
+++ b/src/util/sexpr.c
@@ -25,8 +25,8 @@
 #define VIR_FROM_THIS VIR_FROM_SEXPR
 
 #define virSexprError(code, ...)                                           \
-        virReportErrorHelper(NULL, VIR_FROM_SEXPR, code, __FILE__,         \
-                               __FUNCTION__, __LINE__, __VA_ARGS__)
+        virReportErrorHelper(VIR_FROM_SEXPR, code, __FILE__,               \
+                             __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /**
  * sexpr_new:

diff --git a/src/util/stats_linux.c b/src/util/stats_linux.c
index 5397320075051c1b2232ea6ba93539aa309bbd91..173cdc5dc727b0f4501d9f852c0de377cfa96d2c 100644 (file)
--- a/src/util/stats_linux.c
+++ b/src/util/stats_linux.c
@@ -30,7 +30,7 @@
 # define VIR_FROM_THIS VIR_FROM_STATS_LINUX
 
 # define virStatsError(code, ...)                               \
-    virReportErrorHelper(NULL, VIR_FROM_THIS, code, __FILE__,  \
+    virReportErrorHelper(VIR_FROM_THIS, code, __FILE__,         \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 

diff --git a/src/util/sysinfo.c b/src/util/sysinfo.c
index 2b764ae21c4942765f835ca114de373031deb9d2..a865d2591be4d53b388a5e7e5c4adc13df566e18 100644 (file)
--- a/src/util/sysinfo.c
+++ b/src/util/sysinfo.c
@@ -41,7 +41,7 @@
 #define VIR_FROM_THIS VIR_FROM_SYSINFO
 
 #define virSmbiosReportError(code, ...)                               \
-    virReportErrorHelper(NULL, VIR_FROM_SYSINFO, code, __FILE__,      \
+    virReportErrorHelper(VIR_FROM_SYSINFO, code, __FILE__,            \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #define SYSINFO_SMBIOS_DECODER "dmidecode"

diff --git a/src/util/util.c b/src/util/util.c
index 526f51c0f696193fc7600a8b7a6546f979daba54..d4d2610b1e5f02b03058b19ed19e4000a11a9e44 100644 (file)
--- a/src/util/util.c
+++ b/src/util/util.c
@@ -89,7 +89,7 @@ verify(sizeof(gid_t) <= sizeof (unsigned int) &&
 #define VIR_FROM_THIS VIR_FROM_NONE
 
 #define virUtilError(code, ...)                                            \
-        virReportErrorHelper(NULL, VIR_FROM_NONE, code, __FILE__,          \
+        virReportErrorHelper(VIR_FROM_NONE, code, __FILE__,                \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /* Like read(), but restarts after EINTR */

diff --git a/src/util/virterror.c b/src/util/virterror.c
index b7d8924a5d2095ae430597ae6f5d053dbab2cfba..fbb4a4526c976b1640b6ca01f0187867e2ec5d6c 100644 (file)
--- a/src/util/virterror.c
+++ b/src/util/virterror.c
@@ -663,7 +663,6 @@ virDispatchError(virConnectPtr conn)
 
 /**
  * virRaiseErrorFull:
- * @conn: the connection to the hypervisor if available
  * @filename: filename where error was raised
  * @funcname: function name where error was raised
  * @linenr: line number where error was raised
@@ -682,8 +681,7 @@ virDispatchError(virConnectPtr conn)
  * immediately if a callback is found and store it for later handling.
  */
 void
-virRaiseErrorFull(virConnectPtr conn ATTRIBUTE_UNUSED,
-                  const char *filename ATTRIBUTE_UNUSED,
+virRaiseErrorFull(const char *filename ATTRIBUTE_UNUSED,
                   const char *funcname,
                   size_t linenr,
                   int domain,
@@ -1214,7 +1212,6 @@ virErrorMsg(virErrorNumber error, const char *info)
 /**
  * virReportErrorHelper:
  *
- * @conn: the connection to the hypervisor if available
  * @domcode: the virErrorDomain indicating where it's coming from
  * @errcode: the virErrorNumber code for the error
  * @filename: Source file error is dispatched from
@@ -1226,8 +1223,7 @@ virErrorMsg(virErrorNumber error, const char *info)
  * Helper function to do most of the grunt work for individual driver
  * ReportError
  */
-void virReportErrorHelper(virConnectPtr conn,
-                          int domcode,
+void virReportErrorHelper(int domcode,
                           int errcode,
                           const char *filename,
                           const char *funcname,
@@ -1248,7 +1244,7 @@ void virReportErrorHelper(virConnectPtr conn,
     }
 
     virerr = virErrorMsg(errcode, (errorMessage[0] ? errorMessage : NULL));
-    virRaiseErrorFull(conn, filename, funcname, linenr,
+    virRaiseErrorFull(filename, funcname, linenr,
                       domcode, errcode, VIR_ERR_ERROR,
                       virerr, errorMessage, NULL,
                       -1, -1, virerr, errorMessage);
@@ -1324,7 +1320,7 @@ void virReportSystemErrorFull(int domcode,
     if (!msgDetail)
         msgDetail = errnoDetail;
 
-    virRaiseErrorFull(NULL, filename, funcname, linenr,
+    virRaiseErrorFull(filename, funcname, linenr,
                       domcode, VIR_ERR_SYSTEM_ERROR, VIR_ERR_ERROR,
                       msg, msgDetail, NULL, -1, -1, msg, msgDetail);
     errno = save_errno;
@@ -1348,7 +1344,7 @@ void virReportOOMErrorFull(int domcode,
     const char *virerr;
 
     virerr = virErrorMsg(VIR_ERR_NO_MEMORY, NULL);
-    virRaiseErrorFull(NULL, filename, funcname, linenr,
+    virRaiseErrorFull(filename, funcname, linenr,
                       domcode, VIR_ERR_NO_MEMORY, VIR_ERR_ERROR,
                       virerr, NULL, NULL, -1, -1, virerr, NULL);
 }

diff --git a/src/util/virterror_internal.h b/src/util/virterror_internal.h
index 8f32f4169de7956fed1e58fcc518f773b2bf9b18..df4b1d21b680554f13b789865fb16b10e0762db4 100644 (file)
--- a/src/util/virterror_internal.h
+++ b/src/util/virterror_internal.h
@@ -33,8 +33,7 @@ extern void *virUserData;
  *                                                                     *
  ************************************************************************/
 int virErrorInitialize(void);
-void virRaiseErrorFull(virConnectPtr conn,
-                       const char *filename,
+void virRaiseErrorFull(const char *filename,
                        const char *funcname,
                        size_t linenr,
                        int domain,
@@ -46,22 +45,22 @@ void virRaiseErrorFull(virConnectPtr conn,
                        int int1,
                        int int2,
                        const char *fmt, ...)
-    ATTRIBUTE_FMT_PRINTF(13, 14);
+    ATTRIBUTE_FMT_PRINTF(12, 13);
 
 /* Includes 'dom' and 'net' for compatbility, but they're ignored */
-# define virRaiseError(conn, dom, net, domain, code, level,              \
+# define virRaiseError(dom, net, domain, code, level,              \
                       str1, str2, str3, int1, int2, msg, ...)           \
-    virRaiseErrorFull(conn, __FILE__, __FUNCTION__, __LINE__,           \
+    virRaiseErrorFull(__FILE__, __FUNCTION__, __LINE__,           \
                       domain, code, level, str1, str2, str3, int1, int2, \
                       msg, __VA_ARGS__)
 
 const char *virErrorMsg(virErrorNumber error, const char *info);
-void virReportErrorHelper(virConnectPtr conn, int domcode, int errcode,
+void virReportErrorHelper(int domcode, int errcode,
                           const char *filename ATTRIBUTE_UNUSED,
                           const char *funcname ATTRIBUTE_UNUSED,
                           size_t linenr ATTRIBUTE_UNUSED,
                           const char *fmt, ...)
-  ATTRIBUTE_FMT_PRINTF(7, 8);
+  ATTRIBUTE_FMT_PRINTF(6, 7);
 
 void virReportSystemErrorFull(int domcode,
                               int theerrno,

diff --git a/src/util/xml.c b/src/util/xml.c
index 2c50e147bea34113f0502d5665bedfbd20ac9adb..d2989e2a635274b9208dd64db5689ced9e69db19 100644 (file)
--- a/src/util/xml.c
+++ b/src/util/xml.c
@@ -26,7 +26,7 @@
 #define VIR_FROM_THIS VIR_FROM_XML
 
 #define virGenericReportError(from, code, ...)                          \
-        virReportErrorHelper(NULL, from, code, __FILE__,                \
+        virReportErrorHelper(from, code, __FILE__,                      \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #define virXMLError(code, ...)                                          \

diff --git a/src/vbox/vbox_driver.c b/src/vbox/vbox_driver.c
index 9526ee41ad96df5edc00b5efa0092c0967ecab13..fc43b8c5a7963d8014283732f49ee8729c0fde17 100644 (file)
--- a/src/vbox/vbox_driver.c
+++ b/src/vbox/vbox_driver.c
@@ -66,7 +66,7 @@ static virDriver vboxDriverDummy;
 #define VIR_FROM_THIS VIR_FROM_VBOX
 
 #define vboxError(code, ...) \
-        virReportErrorHelper(NULL, VIR_FROM_VBOX, code, __FILE__, \
+        virReportErrorHelper(VIR_FROM_VBOX, code, __FILE__, \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 int vboxRegister(void) {

diff --git a/src/vbox/vbox_tmpl.c b/src/vbox/vbox_tmpl.c
index 3ca34ddbdf5ce38e357554e628d047749aeb0deb..8241d34e594acd7efeba45abecd0526e04cdb656 100644 (file)
--- a/src/vbox/vbox_tmpl.c
+++ b/src/vbox/vbox_tmpl.c
@@ -133,7 +133,7 @@ typedef IMediumAttachment IHardDiskAttachment;
 #endif /* VBOX_API_VERSION >= 3001 */
 
 #define vboxError(code, ...) \
-        virReportErrorHelper(NULL, VIR_FROM_VBOX, code, __FILE__, \
+        virReportErrorHelper(VIR_FROM_VBOX, code, __FILE__, \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #define DEBUGPRUnichar(msg, strUtf16) \

diff --git a/src/vmware/vmware_conf.h b/src/vmware/vmware_conf.h
index e47ce62ca3dbd28bb0dd859d6a57ddc0ae2daec8..b6d9d606747e34e7b2ad3f6d77c25ae54c36e44a 100644 (file)
--- a/src/vmware/vmware_conf.h
+++ b/src/vmware/vmware_conf.h
@@ -31,7 +31,7 @@
 # define PROGRAM_SENTINAL ((char *)0x1)
 
 # define vmwareError(code, ...) \
-         virReportErrorHelper(NULL, VIR_FROM_VMWARE, code, __FILE__, \
+         virReportErrorHelper(VIR_FROM_VMWARE, code, __FILE__, \
                               __FUNCTION__, __LINE__, __VA_ARGS__)
 
 # define TYPE_PLAYER        0

diff --git a/src/vmx/vmx.c b/src/vmx/vmx.c
index b0d32188a2153ac8126dc7dd1a6ec93f86fcbf99..daeedc31546d1e241e7b9c2696c7decfc2c0fd79 100644 (file)
--- a/src/vmx/vmx.c
+++ b/src/vmx/vmx.c
@@ -471,7 +471,7 @@ def->parallels[0]...
 #define VIR_FROM_THIS VIR_FROM_NONE
 
 #define VMX_ERROR(code, ...)                                                  \
-    virReportErrorHelper(NULL, VIR_FROM_NONE, code, __FILE__, __FUNCTION__,   \
+    virReportErrorHelper(VIR_FROM_NONE, code, __FILE__, __FUNCTION__,         \
                          __LINE__, __VA_ARGS__)
 
 #define VMX_BUILD_NAME_EXTRA(_suffix, _extra)                                 \

diff --git a/src/xen/block_stats.c b/src/xen/block_stats.c
index 1918257aefa5e12e8cf3918b7173419f08ee7d17..a28212c5d5d9723cf50b3b38eed941fe1feadb6d 100644 (file)
--- a/src/xen/block_stats.c
+++ b/src/xen/block_stats.c
@@ -42,8 +42,7 @@
  * Handle a stats error.
  */
 static void
-statsErrorFunc (virConnectPtr conn,
-                virErrorNumber error, const char *func, const char *info,
+statsErrorFunc (virErrorNumber error, const char *func, const char *info,
                 int value)
 {
     char fullinfo[1000];
@@ -55,7 +54,7 @@ statsErrorFunc (virConnectPtr conn,
         fullinfo[sizeof (fullinfo) - 1] = 0;
         info = fullinfo;
     }
-    virRaiseError(conn, NULL, NULL, VIR_FROM_STATS_LINUX, error,
+    virRaiseError(NULL, NULL, VIR_FROM_STATS_LINUX, error,
                     VIR_ERR_ERROR,
                     errmsg, info, NULL, value, 0, errmsg, info,
                     value);
@@ -180,8 +179,8 @@ check_bd_connected (xenUnifiedPrivatePtr priv, int device, int domid)
 }
 
 static int
-read_bd_stats (virConnectPtr conn, xenUnifiedPrivatePtr priv,
-               int device, int domid, struct _virDomainBlockStats *stats)
+read_bd_stats(xenUnifiedPrivatePtr priv,
+              int device, int domid, struct _virDomainBlockStats *stats)
 {
     stats->rd_req   = read_bd_stat (device, domid, "rd_req");
     stats->rd_bytes = read_bd_stat (device, domid, "rd_sect");
@@ -195,8 +194,8 @@ read_bd_stats (virConnectPtr conn, xenUnifiedPrivatePtr priv,
     if (stats->rd_req == -1 && stats->rd_bytes == -1 &&
         stats->wr_req == -1 && stats->wr_bytes == -1 &&
         stats->errs == -1) {
-        statsErrorFunc (conn, VIR_ERR_INTERNAL_ERROR, __FUNCTION__,
-                        "Failed to read any block statistics", domid);
+        statsErrorFunc(VIR_ERR_INTERNAL_ERROR, __FUNCTION__,
+                       "Failed to read any block statistics", domid);
         return -1;
     }
 
@@ -208,8 +207,8 @@ read_bd_stats (virConnectPtr conn, xenUnifiedPrivatePtr priv,
         stats->wr_req == 0 && stats->wr_bytes == 0 &&
         stats->errs == 0 &&
         !check_bd_connected (priv, device, domid)) {
-        statsErrorFunc (conn, VIR_ERR_INTERNAL_ERROR, __FUNCTION__,
-                        "Frontend block device not connected", domid);
+        statsErrorFunc(VIR_ERR_INTERNAL_ERROR, __FUNCTION__,
+                       "Frontend block device not connected", domid);
         return -1;
     }
 
@@ -218,18 +217,18 @@ read_bd_stats (virConnectPtr conn, xenUnifiedPrivatePtr priv,
      */
     if (stats->rd_bytes > 0) {
         if (stats->rd_bytes >= ((unsigned long long)1)<<(63-9)) {
-            statsErrorFunc (conn, VIR_ERR_INTERNAL_ERROR, __FUNCTION__,
-                            "stats->rd_bytes would overflow 64 bit counter",
-                            domid);
+            statsErrorFunc(VIR_ERR_INTERNAL_ERROR, __FUNCTION__,
+                           "stats->rd_bytes would overflow 64 bit counter",
+                           domid);
             return -1;
         }
         stats->rd_bytes *= 512;
     }
     if (stats->wr_bytes > 0) {
         if (stats->wr_bytes >= ((unsigned long long)1)<<(63-9)) {
-            statsErrorFunc (conn, VIR_ERR_INTERNAL_ERROR, __FUNCTION__,
-                            "stats->wr_bytes would overflow 64 bit counter",
-                            domid);
+            statsErrorFunc(VIR_ERR_INTERNAL_ERROR, __FUNCTION__,
+                           "stats->wr_bytes would overflow 64 bit counter",
+                           domid);
             return -1;
         }
         stats->wr_bytes *= 512;
@@ -270,7 +269,7 @@ disk_re_match(const char *regex, const char *path, int *part)
 }
 
 int
-xenLinuxDomainDeviceID(virConnectPtr conn, int domid, const char *path)
+xenLinuxDomainDeviceID(int domid, const char *path)
 {
     int major, minor;
     int part;
@@ -347,20 +346,20 @@ xenLinuxDomainDeviceID(virConnectPtr conn, int domid, const char *path)
      * beginning of the strings for better error messages
      */
     else if (strlen(mod_path) >= 7 && STRPREFIX(mod_path, "/dev/sd"))
-        statsErrorFunc (conn, VIR_ERR_INVALID_ARG, __FUNCTION__,
-                        "invalid path, device names must be in the range sda[1-15] - sdiv[1-15]",
-                        domid);
+        statsErrorFunc(VIR_ERR_INVALID_ARG, __FUNCTION__,
+                       "invalid path, device names must be in the range sda[1-15] - sdiv[1-15]",
+                       domid);
     else if (strlen(mod_path) >= 7 && STRPREFIX(mod_path, "/dev/hd"))
-        statsErrorFunc (conn, VIR_ERR_INVALID_ARG, __FUNCTION__,
-                        "invalid path, device names must be in the range hda[1-63] - hdt[1-63]",
-                        domid);
+        statsErrorFunc(VIR_ERR_INVALID_ARG, __FUNCTION__,
+                       "invalid path, device names must be in the range hda[1-63] - hdt[1-63]",
+                       domid);
     else if (strlen(mod_path) >= 8 && STRPREFIX(mod_path, "/dev/xvd"))
-        statsErrorFunc (conn, VIR_ERR_INVALID_ARG, __FUNCTION__,
-                        "invalid path, device names must be in the range xvda[1-15] - xvdiz[1-15]",
-                        domid);
+        statsErrorFunc(VIR_ERR_INVALID_ARG, __FUNCTION__,
+                       "invalid path, device names must be in the range xvda[1-15] - xvdiz[1-15]",
+                       domid);
     else
-        statsErrorFunc (conn, VIR_ERR_INVALID_ARG, __FUNCTION__,
-                        "unsupported path, use xvdN, hdN, or sdN", domid);
+        statsErrorFunc(VIR_ERR_INVALID_ARG, __FUNCTION__,
+                       "unsupported path, use xvdN, hdN, or sdN", domid);
 
     VIR_FREE(mod_path);
 
@@ -373,12 +372,12 @@ xenLinuxDomainBlockStats (xenUnifiedPrivatePtr priv,
                           const char *path,
                           struct _virDomainBlockStats *stats)
 {
-    int device = xenLinuxDomainDeviceID(dom->conn, dom->id, path);
+    int device = xenLinuxDomainDeviceID(dom->id, path);
 
     if (device < 0)
         return -1;
 
-    return read_bd_stats (dom->conn, priv, device, dom->id, stats);
+    return read_bd_stats(priv, device, dom->id, stats);
 }
 
 #endif /* __linux__ */

diff --git a/src/xen/block_stats.h b/src/xen/block_stats.h
index ba113d74ef3015d57bb445d82010a713db7e7a6f..c94f6458fdb73670fe6e369247f364e895d40105 100644 (file)
--- a/src/xen/block_stats.h
+++ b/src/xen/block_stats.h
@@ -19,7 +19,7 @@ extern int xenLinuxDomainBlockStats (xenUnifiedPrivatePtr priv,
                                      virDomainPtr dom, const char *path,
                                      struct _virDomainBlockStats *stats);
 
-extern int xenLinuxDomainDeviceID(virConnectPtr conn, int domid, const char *dev);
+extern int xenLinuxDomainDeviceID(int domid, const char *dev);
 
 # endif /* __linux__ */
 

diff --git a/src/xen/xen_driver.c b/src/xen/xen_driver.c
index 9f4772213e1b82fc4d384de80f5b2445717200eb..2a07b7bd049f9bfcd5bd53ef9cf96d1e05bc2cc2 100644 (file)
--- a/src/xen/xen_driver.c
+++ b/src/xen/xen_driver.c
@@ -79,7 +79,7 @@ static int inside_daemon;
 #endif
 
 #define xenUnifiedError(code, ...)                                         \
-        virReportErrorHelper(NULL, VIR_FROM_XEN, code, __FILE__,           \
+        virReportErrorHelper(VIR_FROM_XEN, code, __FILE__,                 \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /**

diff --git a/src/xen/xen_hypervisor.c b/src/xen/xen_hypervisor.c
index 8a9dae5afad5996caa736382f0226da46daa8e89..9a5b41d1ed9741dfc138133047ca17c383206085 100644 (file)
--- a/src/xen/xen_hypervisor.c
+++ b/src/xen/xen_hypervisor.c
@@ -843,7 +843,7 @@ struct xenUnifiedDriver xenHypervisorDriver = {
 
 #define virXenError(code, ...)                                             \
         if (in_init == 0)                                                  \
-            virReportErrorHelper(NULL, VIR_FROM_XEN, code, __FILE__,       \
+            virReportErrorHelper(VIR_FROM_XEN, code, __FILE__,             \
                                  __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /**
@@ -870,11 +870,11 @@ virXenErrorFunc(virErrorNumber error, const char *func, const char *info,
     if (func != NULL) {
         snprintf(fullinfo, 999, "%s: %s", func, info);
         fullinfo[999] = 0;
-        virRaiseError(NULL, NULL, NULL, VIR_FROM_XEN, error, VIR_ERR_ERROR,
+        virRaiseError(NULL, NULL, VIR_FROM_XEN, error, VIR_ERR_ERROR,
                         errmsg, fullinfo, NULL, value, 0, errmsg, fullinfo,
                         value);
     } else {
-        virRaiseError(NULL, NULL, NULL, VIR_FROM_XEN, error, VIR_ERR_ERROR,
+        virRaiseError(NULL, NULL, VIR_FROM_XEN, error, VIR_ERR_ERROR,
                         errmsg, info, NULL, value, 0, errmsg, info,
                         value);
     }

diff --git a/src/xen/xen_inotify.c b/src/xen/xen_inotify.c
index 5a997e601f106e331de852bb763a116e7ac8de80..d809c45489a95d9c3ea5e7b99170093477a8b94a 100644 (file)
--- a/src/xen/xen_inotify.c
+++ b/src/xen/xen_inotify.c
@@ -46,7 +46,7 @@
 #define VIR_FROM_THIS VIR_FROM_XEN_INOTIFY
 
 #define virXenInotifyError(code, ...)                                   \
-        virReportErrorHelper(NULL, VIR_FROM_XEN_INOTIFY, code, __FILE__,      \
+        virReportErrorHelper(VIR_FROM_XEN_INOTIFY, code, __FILE__,      \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 struct xenUnifiedDriver xenInotifyDriver = {

diff --git a/src/xen/xend_internal.c b/src/xen/xend_internal.c
index 57422d3b2e1e086ade672786c32f9d36b911d91d..b608a437157084404f70891ffa783ce58dd56d39 100644 (file)
--- a/src/xen/xend_internal.c
+++ b/src/xen/xend_internal.c
@@ -68,7 +68,7 @@ virDomainXMLDevID(virDomainPtr domain,
                   int ref_len);
 
 #define virXendError(code, ...)                                            \
-        virReportErrorHelper(NULL, VIR_FROM_XEND, code, __FILE__,          \
+        virReportErrorHelper(VIR_FROM_XEND, code, __FILE__,                \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #define virXendErrorInt(code, ival)                                        \

diff --git a/src/xen/xm_internal.c b/src/xen/xm_internal.c
index 9225808bca1464c7fd00b9425c5ce4f6e8a0c75e..db47a0206f61e6fe181cbfc2b35398e7590f311a 100644 (file)
--- a/src/xen/xm_internal.c
+++ b/src/xen/xm_internal.c
@@ -121,7 +121,7 @@ struct xenUnifiedDriver xenXMDriver = {
 };
 
 #define xenXMError(code, ...)                                              \
-        virReportErrorHelper(NULL, VIR_FROM_XENXM, code, __FILE__,         \
+        virReportErrorHelper(VIR_FROM_XENXM, code, __FILE__,               \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #ifndef WITH_XEN_INOTIFY

diff --git a/src/xen/xs_internal.c b/src/xen/xs_internal.c
index d9aad1f56d04d8e341bce42558a3d9c8a3a6b199..c318f6ce973154d81957168f8a69dace8253ab51 100644 (file)
--- a/src/xen/xs_internal.c
+++ b/src/xen/xs_internal.c
@@ -83,7 +83,7 @@ struct xenUnifiedDriver xenStoreDriver = {
 };
 
 #define virXenStoreError(code, ...)                                  \
-        virReportErrorHelper(NULL, VIR_FROM_XENSTORE, code, __FILE__,      \
+        virReportErrorHelper(VIR_FROM_XENSTORE, code, __FILE__,      \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /************************************************************************

diff --git a/src/xenapi/xenapi_driver.c b/src/xenapi/xenapi_driver.c
index 60b23c7182f2184ef3cda855c795b5e7ded3fdc1..3fbdcc654d544bcd61b93a37076be077d2b69c5b 100644 (file)
--- a/src/xenapi/xenapi_driver.c
+++ b/src/xenapi/xenapi_driver.c
@@ -44,7 +44,7 @@
 #define VIR_FROM_THIS VIR_FROM_XENAPI
 
 #define xenapiError(code, ...)                                    \
-        virReportErrorHelper(NULL, VIR_FROM_THIS, code, __FILE__, \
+        virReportErrorHelper(VIR_FROM_THIS, code, __FILE__,       \
                              __FUNCTION__, __LINE__, __VA_ARGS__)
 
 /*

diff --git a/src/xenapi/xenapi_utils.c b/src/xenapi/xenapi_utils.c
index f50610ad07855e372e58f707306566607718a772..ae20bf7797df80a8771fb369b52fd6c749e91e6f 100644 (file)
--- a/src/xenapi/xenapi_utils.c
+++ b/src/xenapi/xenapi_utils.c
@@ -386,11 +386,11 @@ xenapiSessionErrorHandle(virConnectPtr conn, virErrorNumber errNum,
 
     if (buf == NULL && priv != NULL && priv->session != NULL) {
         char *ret = returnErrorFromSession(priv->session);
-        virReportErrorHelper(conn, VIR_FROM_XENAPI, errNum, filename, func, lineno, _("%s"), ret);
+        virReportErrorHelper(VIR_FROM_XENAPI, errNum, filename, func, lineno, _("%s"), ret);
         xen_session_clear_error(priv->session);
         VIR_FREE(ret);
     } else {
-        virReportErrorHelper(conn, VIR_FROM_XENAPI, errNum, filename, func, lineno, _("%s"), buf);
+        virReportErrorHelper(VIR_FROM_XENAPI, errNum, filename, func, lineno, _("%s"), buf);
     }
 }
 

diff --git a/src/xenxs/xenxs_private.h b/src/xenxs/xenxs_private.h
index 9bf1223d2d0fed90f6aadc94e20ff612231f8b75..60d27d22fee2d776ab0f4ff994eb8090fd68a71b 100644 (file)
--- a/src/xenxs/xenxs_private.h
+++ b/src/xenxs/xenxs_private.h
@@ -57,7 +57,7 @@
 # define VIR_FROM_THIS VIR_FROM_NONE
 
 # define XENXS_ERROR(code, ...)                                               \
-    virReportErrorHelper(NULL, VIR_FROM_NONE, code, __FILE__, __FUNCTION__,   \
+    virReportErrorHelper(VIR_FROM_NONE, code, __FILE__, __FUNCTION__,         \
                          __LINE__, __VA_ARGS__)
 
 #endif /* __VIR_XENXS_PRIVATE_H__ */