datapath: Make VERIFY_NUL_STRING return a negative error code.

This is the common kernel convention.  None of the callers currently do
anything with the return code except for test whether it is nonzero, so
they don't need any changes.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>

diff --git a/datapath/linux-2.6/compat-2.6/include/net/netlink.h b/datapath/linux-2.6/compat-2.6/include/net/netlink.h
index c11a170087f3283229c1e4affe4dfd5881d20dca..a23deee1251b80906f64188979e46b2590d94b7b 100644 (file)
--- a/datapath/linux-2.6/compat-2.6/include/net/netlink.h
+++ b/datapath/linux-2.6/compat-2.6/include/net/netlink.h
@@ -11,7 +11,7 @@ static inline int VERIFY_NUL_STRING(struct nlattr *attr)
 {
        return (!attr || (nla_len(attr)
                          && memchr(nla_data(attr), '\0', nla_len(attr)))
-               ? 0 : EINVAL);
+               ? 0 : -EINVAL);
 }
 #else
 static inline int VERIFY_NUL_STRING(struct nlattr *attr)