Messages in this thread | | | From | (Eric W. Biederman) | Date | Thu, 14 Feb 2013 23:10:46 -0800 | Subject | Re: [PATCH] tcp: sysctl to disable TCP simultaneous connect |
| |
Kees Cook <keescook@chromium.org> writes:
> On Thu, Feb 14, 2013 at 9:30 PM, Eric W. Biederman > <ebiederm@xmission.com> wrote: >> Kees Cook <keescook@chromium.org> writes: >> >>> 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. >> >> Chrome OS has no plans to implement webrtc? Last I had read that >> support had been added to the release versions of Chrome, and was in the >> development builds of firefox. I really don't belive that there are >> many systems that don't intend to run a web browser. > > I haven't looked at the internals of webrtc. Are you implying some > feature in it relies on the TCP simultaneous connect?
I am saying that yes.
webrtc is built on ICE (interactivity connectivity establishment). ICE support for TCP (RFC6544) uses TCP simultaneous connect. webrtc supports tcp connections.
Eric
| |