On Tue, May 09, 2017 at 05:51:13PM +0200, Florian Westphal wrote:
> nft add rule ip6 f i meta l4proto ipv6-icmp icmpv6 type nd-router-advert
> <cmdline>:1:50-60: Error: conflicting protocols specified: unknown vs. icmpv6
>
> add icmpv6 to nexthdr list so base gets updated correctly.

I would like to see a new test coming with this patch. So we can
easily relate the fix with the new test.

> Reported-by: Thomas Woerner <[email protected]>
> Signed-off-by: Florian Westphal <[email protected]>

Acked-by: Pablo Neira Ayuso <[email protected]>

>  src/proto.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/proto.c b/src/proto.c
> index 79e9dbf2b33e..fcdfbe73c735 100644
> --- a/src/proto.c
> +++ b/src/proto.c
> @@ -779,6 +779,7 @@ const struct proto_desc proto_inet_service = {
>               PROTO_LINK(IPPROTO_TCP,         &proto_tcp),
>               PROTO_LINK(IPPROTO_DCCP,        &proto_dccp),
>               PROTO_LINK(IPPROTO_SCTP,        &proto_sctp),
> +             PROTO_LINK(IPPROTO_ICMPV6,      &proto_icmp6),
>       },
>       .templates      = {
>               [0]     = PROTO_META_TEMPLATE("l4proto", &inet_protocol_type, 
> NFT_META_L4PROTO, 8),
> -- 
> 2.10.2
> 
> --
> To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to