From cad2a4caedb6274c59682dee630f5a42d059fd0d Mon Sep 17 00:00:00 2001
From: Matthias Bolte <matthias.bolte@googlemail.com>
Date: Fri, 12 Feb 2010 00:18:54 +0100
Subject: [PATCH] Convert virSecurityReportError into a macro

The virRaiseError macro inside of virSecurityReportError expands to
virRaiseErrorFull and includes the __FILE__, __FUNCTION__ and __LINE__
information. But this three values are always the same for every call
to virSecurityReportError and do not reflect the actual error context.

Converting virSecurityReportError into a macro results in getting the
correct __FILE__, __FUNCTION__ and __LINE__ information.
---
 src/security/security_driver.c | 18 ------------------
 src/security/security_driver.h |  6 +++---
 2 files changed, 3 insertions(+), 21 deletions(-)

diff --git a/src/security/security_driver.c b/src/security/security_driver.c
index 27945a6697..4c98190e10 100644
--- a/src/security/security_driver.c
+++ b/src/security/security_driver.c
@@ -90,24 +90,6 @@ virSecurityDriverStartup(virSecurityDriverPtr *drv,
     return -2;
 }
 
-void
-virSecurityReportError(int code, const char *fmt, ...)
-{
-    va_list args;
-    char errorMessage[1024];
-
-    if (fmt) {
-        va_start(args, fmt);
-        vsnprintf(errorMessage, sizeof(errorMessage) - 1, fmt, args);
-        va_end(args);
-    } else
-        errorMessage[0] = '\0';
-
-    virRaiseError(NULL, NULL, NULL, VIR_FROM_SECURITY, code,
-                  VIR_ERR_ERROR, NULL, NULL, NULL, -1, -1, "%s",
-                  errorMessage);
-}
-
 /*
  * Helpers
  */
diff --git a/src/security/security_driver.h b/src/security/security_driver.h
index 8860d81431..15671b3601 100644
--- a/src/security/security_driver.h
+++ b/src/security/security_driver.h
@@ -88,9 +88,9 @@ int virSecurityDriverStartup(virSecurityDriverPtr *drv,
 int
 virSecurityDriverVerify(virDomainDefPtr def);
 
-void
-virSecurityReportError(int code, const char *fmt, ...)
-    ATTRIBUTE_FMT_PRINTF(2, 3);
+#define virSecurityReportError(code, fmt...)                        \
+    virReportErrorHelper(NULL, VIR_FROM_SECURITY, code, __FILE__,   \
+                         __FUNCTION__, __LINE__, fmt)
 
 /* Helpers */
 void virSecurityDriverInit(virSecurityDriverPtr drv);