]> granicus.if.org Git - ipset/commitdiff
ipset: Backports for the nla_put_net64() API changes
authorNeutron Soutmun <neo.neutron@gmail.com>
Thu, 23 Jun 2016 10:22:11 +0000 (17:22 +0700)
committerJozsef Kadlecsik <kadlec@blackhole.kfki.hu>
Tue, 28 Jun 2016 08:19:35 +0000 (10:19 +0200)
* Backports the patch "libnl: nla_put_net64():align on a 64-bit area" [1]
  by Nicolas Dichtel <nicolas.dichtel@6wind.com>

* Since the nla_put_net64() API has been changed, therefore, the
  ip_set_compat.h.in should provides the macro IPSET_NLA_PUT_NET64 that
  point to the nla_put_net64() with appropriate number of arguments.

  The build script should distinguish the API changes by detect for
  the existence of nla_put_64bit() function in include/net/netlink.h.
  This function was added in the same patches set and called by
  the nla_put_be64() that called by nla_put_net64() respectively.

[1] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit?id=e9bbe898cbe89b17ad3993c136aa13d0431cd537

Signed-off-by: Neutron Soutmun <neo.neutron@gmail.com>
Signed-off-by: Jozsef Kadlecsik <kadlec@blackhole.kfki.hu>
configure.ac
kernel/include/linux/netfilter/ipset/ip_set_compat.h.in
kernel/include/linux/netfilter/ipset/ip_set_counter.h
kernel/include/linux/netfilter/ipset/ip_set_skbinfo.h
kernel/include/uapi/linux/netfilter/ipset/ip_set.h

index 7156ab84deb56e7ea3cabe0a14fa2b7376a23ba5..b73b3cc7e706e330ff5c7476b8f768745efaf8ec 100644 (file)
@@ -250,6 +250,16 @@ else
        AC_SUBST(HAVE_NLA_PUT_BE64, undef)
 fi
 
+AC_MSG_CHECKING([kernel source for nla_put_64bit])
+if test -f $ksourcedir/include/net/netlink.h && \
+   $GREP -q 'nla_put_64bit' $ksourcedir/include/net/netlink.h; then
+       AC_MSG_RESULT(yes)
+       AC_SUBST(HAVE_NLA_PUT_64BIT, define)
+else
+       AC_MSG_RESULT(no)
+       AC_SUBST(HAVE_NLA_PUT_64BIT, undef)
+fi
+
 AC_MSG_CHECKING([kernel source for portid in nl_info])
 if test -f $ksourcedir/include/linux/netlink.h && \
    $AWK '/^struct netlink_skb_parms/ {for(i=1; i<=5; i++) {getline; print}}' $ksourcedir/include/linux/netlink.h | $GREP -q 'portid;'; then
index 15d076376e1817e92495028334830a6abbb077ec..fe24255843fac4bb08c2f425885c9a57a868dac1 100644 (file)
@@ -10,6 +10,7 @@
 #@HAVE_ETHER_ADDR_EQUAL@ HAVE_ETHER_ADDR_EQUAL
 #@HAVE_NLA_PUT_BE16@ HAVE_NLA_PUT_BE16
 #@HAVE_NLA_PUT_BE64@ HAVE_NLA_PUT_BE64
+#@HAVE_NLA_PUT_64BIT@ HAVE_NLA_PUT_64BIT
 #@HAVE_NL_INFO_PORTID@ HAVE_NL_INFO_PORTID
 #define HAVE_NETLINK_DUMP_START_ARGS   @HAVE_NETLINK_DUMP_START_ARGS@
 #@HAVE_NS_CAPABLE@ HAVE_NS_CAPABLE
@@ -160,6 +161,12 @@ static inline int nla_put_net64(struct sk_buff *skb, int attrtype, __be64 value)
 }
 #endif
 
+#ifdef HAVE_NLA_PUT_64BIT
+#define IPSET_NLA_PUT_NET64(skb, t, v, pa) nla_put_be64(skb, t, v, pa)
+#else
+#define IPSET_NLA_PUT_NET64(skb, t, v, pa) nla_put_be64(skb, t, v)
+#endif
+
 #ifdef HAVE_NL_INFO_PORTID
 #define NETLINK_PORTID(skb)    NETLINK_CB(skb).portid
 #else
index d61e0e5085c42af7c6ddadc550bd847bab951b80..f2f1e8f2508f0a2d0179e66f7bd78eceb38a9977 100644 (file)
@@ -53,10 +53,12 @@ ip_set_update_counter(struct ip_set_counter *counter,
 static inline bool
 ip_set_put_counter(struct sk_buff *skb, const struct ip_set_counter *counter)
 {
-       return nla_put_net64(skb, IPSET_ATTR_BYTES,
-                            cpu_to_be64(ip_set_get_bytes(counter))) ||
-              nla_put_net64(skb, IPSET_ATTR_PACKETS,
-                            cpu_to_be64(ip_set_get_packets(counter)));
+       return IPSET_NLA_PUT_NET64(skb, IPSET_ATTR_BYTES,
+                                  cpu_to_be64(ip_set_get_bytes(counter)),
+                                  IPSET_ATTR_PAD) ||
+              IPSET_NLA_PUT_NET64(skb, IPSET_ATTR_PACKETS,
+                                  cpu_to_be64(ip_set_get_packets(counter)),
+                                  IPSET_ATTR_PAD);
 }
 
 static inline void
index b03dd6a3a5a2cc4f642cf4ee1126a56c4d2b0a62..c6df3a1c2d65d0b550c1328b94a806fa241fc835 100644 (file)
@@ -23,9 +23,10 @@ ip_set_put_skbinfo(struct sk_buff *skb, const struct ip_set_skbinfo *skbinfo)
 {
        /* Send nonzero parameters only */
        return ((skbinfo->skbmark || skbinfo->skbmarkmask) &&
-               nla_put_net64(skb, IPSET_ATTR_SKBMARK,
-                             cpu_to_be64((u64)skbinfo->skbmark << 32 |
-                                         skbinfo->skbmarkmask))) ||
+               IPSET_NLA_PUT_NET64(skb, IPSET_ATTR_SKBMARK,
+                                   cpu_to_be64((u64)skbinfo->skbmark << 32 |
+                                               skbinfo->skbmarkmask),
+                                   IPSET_ATTR_PAD)) ||
               (skbinfo->skbprio &&
                nla_put_net32(skb, IPSET_ATTR_SKBPRIO,
                              cpu_to_be32(skbinfo->skbprio))) ||
index def91b9d308457a5b1ee9d2eafb5fe79c7af3fe1..4a6776f7c09556449a54d559abcf4fe18dcec3c3 100644 (file)
@@ -118,6 +118,7 @@ enum {
        IPSET_ATTR_SKBMARK,
        IPSET_ATTR_SKBPRIO,
        IPSET_ATTR_SKBQUEUE,
+       IPSET_ATTR_PAD,
        __IPSET_ATTR_ADT_MAX,
 };
 #define IPSET_ATTR_ADT_MAX     (__IPSET_ATTR_ADT_MAX - 1)