[libvirt] [PATCH 02/10] nwfilter: use consistent OOM reporting

Eric Blake eblake at redhat.com
Wed Aug 18 23:45:05 UTC 2010


* src/nwfilter/nwfilter_driver.c (nwfilterLog): Delete.
(nwfilterDriverStartup): Use virReportOOMError instead.
---

No point making printf uses harder to audit by hiding them in a macro,
especially when this file already uses virReportOOMError elsewhere.

 src/nwfilter/nwfilter_driver.c |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/src/nwfilter/nwfilter_driver.c b/src/nwfilter/nwfilter_driver.c
index 0e8241e..bda50f9 100644
--- a/src/nwfilter/nwfilter_driver.c
+++ b/src/nwfilter/nwfilter_driver.c
@@ -42,9 +42,6 @@

 #define VIR_FROM_THIS VIR_FROM_NWFILTER

-#define nwfilterLog(msg...) fprintf(stderr, msg)
-
-
 static virNWFilterDriverStatePtr driverState;

 static int nwfilterDriverShutdown(void);
@@ -95,7 +92,6 @@ nwfilterDriverStartup(int privileged) {
             goto error;

         if (virAsprintf(&base, "%s/.libvirt", userdir) == -1) {
-            nwfilterLog("out of memory in virAsprintf");
             VIR_FREE(userdir);
             goto out_of_memory;
         }
@@ -118,7 +114,7 @@ nwfilterDriverStartup(int privileged) {
     return 0;

 out_of_memory:
-    nwfilterLog("virNWFilterStartup: out of memory");
+    virReportOOMError();

 error:
     VIR_FREE(base);
-- 
1.7.2.1




More information about the libvir-list mailing list