include/net/netlink.h: In function ‘nlmsg_parse’:
include/net/netlink.h:368:21: warning: comparison between signed and
    unsigned integer expressions [-Wsign-compare]
include/net/netlink.h: In function ‘nlmsg_validate’:
include/net/netlink.h:401:21: warning: comparison between signed and
    unsigned integer expressions [-Wsign-compare]

Seen with gcc (GCC) 4.6.3 20120306 (Red Hat 4.6.3-2).

The result of nlmsg_msg_size() is always compared to
nlmsghdr.nlmsg_len, which is unsigned.

Clean up: nlmsg_padlen() is no longer used.

Signed-off-by: Chuck Lever <chuck.le...@oracle.com>
---

 include/net/netlink.h |   11 +----------
 1 files changed, 1 insertions(+), 10 deletions(-)

diff --git a/include/net/netlink.h b/include/net/netlink.h
index e025043..27c7ab7 100644
--- a/include/net/netlink.h
+++ b/include/net/netlink.h
@@ -258,7 +258,7 @@ extern int          nla_append(struct sk_buff *skb, int 
attrlen,
  * nlmsg_msg_size - length of netlink message not including padding
  * @payload: length of message payload
  */
-static inline int nlmsg_msg_size(int payload)
+static inline unsigned int nlmsg_msg_size(int payload)
 {
        return NLMSG_HDRLEN + payload;
 }
@@ -273,15 +273,6 @@ static inline int nlmsg_total_size(int payload)
 }
 
 /**
- * nlmsg_padlen - length of padding at the message's tail
- * @payload: length of message payload
- */
-static inline int nlmsg_padlen(int payload)
-{
-       return nlmsg_total_size(payload) - nlmsg_msg_size(payload);
-}
-
-/**
  * nlmsg_data - head of message payload
  * @nlh: netlink message header
  */


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel&#174; Ethernet, visit 
http://communities.intel.com/community/wired

Reply via email to