On Tue, Feb 18, 2014 at 11:02:13PM +0200, Julian Anastasov wrote:
> Thanks for testing! This patch needs some tuning,
> refer to Documentation/CodingStyle for the rules.
> checkpatch.pl reports for the problems:
Thanks. I was a reading it, but something with impatience and
lazy, and worst of all, I still don't know an easy way to blame
without cloning the repository :-(
<snipped proof of lazy incompetence>
> I agree for the comment but not sure if it is
> appropriate for bugfixes that go to stable kernels.
> Also, the format should be icmp_offset=%u, not %d.
Thanks for yet another oversight ;-). And it would definitely not
be needed here. I do wonder about the EnterFunction though.
I could not clearly see what really generatd the "invalid header"
error message in the log.
> Also, we should mention the problematic commit
> and to CC the authors. You can tune/borrow from the
> following example:
> [PATCH] ipvs: fix wrong icmp_offset in ip_vs_nat_icmp_v6
> Fix regression introduced in 3.8 with commit 63dca2c0b0e7a9
> ("ipvs: Fix faulty IPv6 extension header handling in IPVS").
> Calling ipv6_find_hdr with protocol (IPPROTO_ICMPV6) is not
> supported, use -1 instead. Solves problem of damaged IPv6
> headers in NAT-ed ICMP packets.
Jesper and Hans, I think that ipv6_find_hdr should have a target
of -1 to find the protocol header, and any other next-header
target will be valid after that.
Anyway, I will read the committing patches again and reduce the
patch to just a change -1.
If I see any "invalid headers" again on the firewall, I will add
The current patch is live using 3.12.11
( http://www.speurders.nl/ and http://www.spitsnieuws.nl/ f.i. )
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html