From: Patrick McHardy <kaber@trash.net>
To: Shan Wei <shanwei@cn.fujitsu.com>
Cc: David Miller <davem@davemloft.net>,
netfilter-devel@vger.kernel.org,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: Re: [RFC][PATCH] IP: Send a fragment reassembly time exceeded packet when enabling connection track
Date: Tue, 05 Jan 2010 06:44:35 +0100 [thread overview]
Message-ID: <4B42D1C3.3020207@trash.net> (raw)
In-Reply-To: <4B3191E7.8060509@cn.fujitsu.com>
Shan Wei wrote:
> Default, a host may send a fragment reassembly time exceeded packet
> (ICMP Time Exceeded Message with code value of 1) when defraging fragments timeout.
> But, when enabling connection track, a host can't send the packet.
>
> Because, the module of nf_defrag_ipv4 selected by connection track is registered
> in PRE_ROUTING HOOK and assembles all accepted fragments(here, not begin to routing).
> After defrag timeout, the host can't send fragment reassembly time exceeded packet,
> because of lack of router information.
>
> RFC 792 says:
>>>>> If a host reassembling a fragmented datagram cannot complete the
>>>>> reassembly due to missing fragments within its time limit it
>>>>> discards the datagram, and it may send a time exceeded message.
>>>>>
>>>>> If fragment zero is not available then no time exceeded need be
>>>>> sent at all.
>>>>>
>>>>>
>>>>> Read more: http://www.faqs.org/rfcs/rfc792.html#ixzz0aOXRD7Wp
>
> So, the patch try to fix it with filling router information before sending fragment reassembly
> time exceeded packet when defrag timeout.
I guess the question is whether we really want to send an ICMP
message in this case. The above quote applies to end hosts,
while conntrack is also (probably more commonly) used on routers,
which normally shouldn't attempt reassembly. I can see no real
downside to this though except that it makes it quite easy to
discover firewalls, but that shouldn't be a real problem.
next prev parent reply other threads:[~2010-01-05 5:44 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-23 3:43 [RFC][PATCH] IP: Send a fragment reassembly time exceeded packet when enabling connection track Shan Wei
2010-01-05 5:44 ` Patrick McHardy [this message]
2010-01-13 3:15 ` Shan Wei
2010-01-13 8:27 ` Patrick McHardy
2010-01-14 9:18 ` Shan Wei
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4B42D1C3.3020207@trash.net \
--to=kaber@trash.net \
--cc=davem@davemloft.net \
--cc=netdev@vger.kernel.org \
--cc=netfilter-devel@vger.kernel.org \
--cc=shanwei@cn.fujitsu.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.