net: nla_align_64bit() needs to test the right pointer.
authorDavid S. Miller <davem@davemloft.net>
Wed, 20 Apr 2016 19:32:54 +0000 (15:32 -0400)
committerDavid S. Miller <davem@davemloft.net>
Wed, 20 Apr 2016 19:32:54 +0000 (15:32 -0400)
Netlink messages are appended, one object at a time, to the end of
the SKB.  Therefore we need to test skb_tail_pointer() not skb->data
for alignment.

Fixes: 35c5845957c7 ("net: Add helpers for 64-bit aligning netlink attributes.")
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/netlink.h

index cf95df1fa14b03737d2fef3b95f2de5ad15a8b3b..3c1fd92a52c8274174f7e2057f36bfa3626eddb0 100644 (file)
@@ -1250,7 +1250,7 @@ static inline int nla_align_64bit(struct sk_buff *skb, int padattr)
         * nlattr header for next attribute, will make nla_data()
         * 8-byte aligned.
         */
-       if (IS_ALIGNED((unsigned long)skb->data, 8) &&
+       if (IS_ALIGNED((unsigned long)skb_tail_pointer(skb), 8) &&
            !nla_reserve(skb, padattr, 0))
                return -EMSGSIZE;
 #endif