[libvirt] [PATCH] error: Fix macro checking for NULL arguments

Peter Krempa pkrempa at redhat.com
Tue Jun 19 13:58:32 UTC 2012


Macro virCheckNullArgGoto is supposed to check for NULL argument but
checks non-NULL instead.
---
 src/internal.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/internal.h b/src/internal.h
index 1b1598b..b09609e 100644
--- a/src/internal.h
+++ b/src/internal.h
@@ -241,21 +241,21 @@

 # define virCheckNonNullArgReturn(argname, retval)  \
     do {                                            \
         if (argname == NULL) {                      \
             virReportInvalidNullArg(argname);       \
             return retval;                          \
         }                                           \
     } while (0)
 # define virCheckNullArgGoto(argname, label)        \
     do {                                            \
-        if (argname == NULL) {                      \
+        if (argname != NULL) {                      \
             virReportInvalidNullArg(argname);       \
             goto label;                             \
         }                                           \
     } while (0)
 # define virCheckNonNullArgGoto(argname, label)     \
     do {                                            \
         if (argname == NULL) {                      \
             virReportInvalidNonNullArg(argname);    \
             goto label;                             \
         }                                           \
-- 
1.7.8.6




More information about the libvir-list mailing list