From mboxrd@z Thu Jan 1 00:00:00 1970 From: "H. Peter Anvin" Subject: Re: [PATCH] tcp: sysctl to disable TCP simultaneous connect Date: Thu, 07 Feb 2013 11:29:08 -0800 Message-ID: <51140084.3080902@zytor.com> References: <20130207175240.GA12520@www.outflux.net> <20130207103950.662698ea@nehalam.linuxnetplumber.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Stephen Hemminger , LKML , Rob Landley , "David S. Miller" , Alexey Kuznetsov , James Morris , Hideaki YOSHIFUJI , Patrick McHardy , Eric Dumazet , Neil Horman , Yuchung Cheng , Shan Wei , "linux-doc@vger.kernel.org" , netdev@vger.kernel.org, Willy Tarreau To: Kees Cook Return-path: In-Reply-To: Sender: linux-doc-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On 02/07/2013 10:44 AM, Kees Cook wrote: >> >> This patch probably also breaks TCP STUNT that is used by some applications for NAT >> traversal. > > The patch would not break it -- it defaults the sysctl to staying enabled. > > If you mean the documentation should be updated, sure, that's easy to do. > > David: I know you aren't a fan of this patch, but I'd like to try to > convince you. :) This leaves the feature enabled and add a toggle for > systems (like Chrome OS) that don't want to risk this DoS at all. > There are so very many other toggle, I don't see why this one would be > a problem to add. > It is not just STUNT, but in NAT-less configurations behind stateful firewalls (which is expected to be the norm for IPv6), TCP rendezvous via crossed SYN is a very effective way to establish peer-to-peer connections. -hpa