Message ID | 1407363429-3870-1-git-send-email-pablo@netfilter.org |
---|---|
State | Awaiting Upstream |
Delegated to: | Pablo Neira |
Headers | show |
From: Pablo Neira Ayuso <pablo@netfilter.org> Date: Thu, 7 Aug 2014 00:17:09 +0200 > cb1ce2e ("ipv6: Implement automatic flow label generation on transmit") > accidentally uses socket option 64, which is already used by ip6tables: > > IP6T_SO_SET_REPLACE / IP6T_SO_GET_INFO 64 > IP6T_SO_SET_ADD_COUNTERS / IP6T_SO_GET_ENTRIES 65 > > There is comment include/uapi/linux/in6.h warning about that. > > Allocate 70 for this, which seems to be unused instead. > > Cc: Tom Herbert <therbert@google.com> > Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org> Applied, thanks Pablo. The way we maintain this number space seems very error prone, FWIW. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, Aug 07, 2014 at 12:17:09AM +0200, Pablo Neira Ayuso wrote: > cb1ce2e ("ipv6: Implement automatic flow label generation on transmit") > accidentally uses socket option 64, which is already used by ip6tables: > > IP6T_SO_SET_REPLACE / IP6T_SO_GET_INFO 64 > IP6T_SO_SET_ADD_COUNTERS / IP6T_SO_GET_ENTRIES 65 > > There is comment include/uapi/linux/in6.h warning about that. > > Allocate 70 for this, which seems to be unused instead. > > Cc: Tom Herbert <therbert@google.com> > Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org> Just tested this, ip6tables works fine now. thanks Pablo. Dave -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/include/uapi/linux/in6.h b/include/uapi/linux/in6.h index 22b7a69..74a2a17 100644 --- a/include/uapi/linux/in6.h +++ b/include/uapi/linux/in6.h @@ -233,7 +233,6 @@ struct in6_flowlabel_req { #if 0 /* not yet */ #define IPV6_USE_MIN_MTU 63 #endif -#define IPV6_AUTOFLOWLABEL 64 /* * Netfilter (1) @@ -262,6 +261,7 @@ struct in6_flowlabel_req { * IP6T_SO_ORIGINAL_DST 80 */ +#define IPV6_AUTOFLOWLABEL 70 /* RFC5014: Source address selection */ #define IPV6_ADDR_PREFERENCES 72
cb1ce2e ("ipv6: Implement automatic flow label generation on transmit") accidentally uses socket option 64, which is already used by ip6tables: IP6T_SO_SET_REPLACE / IP6T_SO_GET_INFO 64 IP6T_SO_SET_ADD_COUNTERS / IP6T_SO_GET_ENTRIES 65 There is comment include/uapi/linux/in6.h warning about that. Allocate 70 for this, which seems to be unused instead. Cc: Tom Herbert <therbert@google.com> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org> --- include/uapi/linux/in6.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)