Re: problem forwarding IP fragments with DF bit set (caused by ipv4: fix path MTU discovery with connection tracking)

[Patrick McHardy <kaber@trash.net>]

On Tue, Apr 29, 2014 at 04:42:15PM +0200, Maxime Bizon wrote:
> 
> On Tue, 2014-04-29 at 15:33 +0100, Patrick McHardy wrote:
> 
> > That is the correct behaviour.
> 
> is it ?
> 
> say the remote host is sending two 1000 bytes fragments, with DF set to
> me and my MTU is 1500, I'm suppose to route these packets to an
> interface with the same MTU.
> 
> without conntrack both fragments are correctly forwarded
> 
> with conntrack, the original packet is re-assembled, then instead of
> forwarding it, we send a "frag needed mtu = 1500" to the remote host,
> which never sent any packets bigger than 1500 at the beginning.

Right, that is not correct of course. We save the original packet size
and should either refragment to that size or send an ICMP frag required
if the original size exceeds the outgoing MTU.

So your patch does look correct, however we should probably only set
local_df in conntrack defrag.