Le vendredi 25 novembre 2011 à 14:19 +0000, David Laight a écrit : > > + addr1 = (__force u32) ip6->saddr.s6_addr32[3]; > > + addr2 = (__force u32) ip6->daddr.s6_addr32[3]; > ... > > + ports.v32 = * (__force u32 *) (skb->data + nhoff); > > Is this code even vaguely portable?? Yes it is. > I suspect the 'ports' bit has serious endianness problems. We dont care of endianness here, and we document it with the (__force u32) cast. > I'm also not sure whether linux guarantees the alignment > of skb->data here. It is guaranteed in whole linux stack. -- 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