Balazs Scheidler wrote: > +static bool > +socket_mt6_v1(const struct sk_buff *skb, const struct xt_match_param *par) > +{ > + struct ipv6hdr *iph = ipv6_hdr(skb); > + struct udphdr _hdr, *hp = NULL; > + struct sock *sk; > + struct in6_addr *daddr, *saddr; > + __be16 dport, sport; > + int thoff; > + u8 tproto; > + const struct xt_socket_mtinfo1 *info = (struct xt_socket_mtinfo1 *) par->matchinfo; > + > + tproto = ipv6_find_hdr(skb, &thoff, -1, NULL); > + if (tproto < 0) { > + pr_debug("socket match: Unable to find transport header in IPv6 packet, dropping\n"); > + return NF_DROP; > + } > + > + if (tproto == IPPROTO_UDP || tproto == IPPROTO_TCP) { > + hp = skb_header_pointer(skb, thoff, > + sizeof(_hdr), &_hdr); > + if (hp == NULL) > + return false; > + > + saddr = &iph->saddr; > + sport = hp->source; > + daddr = &iph->daddr; > + dport = hp->dest; > + > + } else if (tproto == IPPROTO_ICMP) { > + if (extract_icmp6_fields(skb, thoff, &tproto, &saddr, &daddr, > + &sport, &dport)) > + return false; > + } else { > + return false; > + } Shouldn't this be IPPROTO_ICMPV6? -Brian -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html