netdev - Re: Add a SOCK_DESTROY operation to close sockets from userspace

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAKD1Yr2mCs_v2posCVz=YwhCPvGVH+=VCBpWWf2TB-2aDj5R+w@mail.gmail.com>
Date:	Fri, 20 Nov 2015 07:55:44 +0900
From:	Lorenzo Colitti <lorenzo@...gle.com>
To:	David Miller <davem@...emloft.net>
Cc:	Eric Dumazet <eric.dumazet@...il.com>,
	Maciej Żenczykowski <zenczykowski@...il.com>,
	Hannes Frederic Sowa <hannes@...essinduktion.org>,
	Stephen Hemminger <stephen@...workplumber.org>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	Eric Dumazet <edumazet@...gle.com>, Erik Kline <ek@...gle.com>,
	Dmitry Torokhov <dtor@...gle.com>
Subject: Re: Add a SOCK_DESTROY operation to close sockets from userspace

On Fri, Nov 20, 2015 at 1:33 AM, David Miller <davem@...emloft.net> wrote:
>> Every-time I make a change in linux TCP stack, this code breaks, and
>> this a  real pain because Android changes need to be carried over to
>> vendors.
>
> I'm very glad that you felt the pain enough that you finally had
> to reluctantly try and upstream something.  I'm sorry that doing
> hackish things locally in the Android kernel tree is so painful :-/

Actually, when SIOCKILLADDR crashes users' phones, Eric is not the one
that needs to fix this. I am. Because I am not a TCP expert or a
hardcode kernel developer, I ask Eric for help, and he is good enough
to provide it.

> Android folks really do not care about upstream, and it is probably
> bottom of their priority.  Their actions consistently support this.

Speaking as the one of the Android networking team leads, I would love
it if that changed. It would certainly make my life easier if these
features were upstream. I am trying to upstream all of them so that I
don't have to worry about ensuring that all devices pick them up as
opposed to shipping kernels that lack important commits, and thus
contain subtle bugs that cause real networking problems for real
users.

Pretty much everything we have done in networking code in the past two
years we have tried to upstream - see patches by myself and Erik Kline
that got merged. Some of the things we do are ugly hacks that we've
were expedient at the time, like SIOCKILLADDR. Eric rightly says it's
evil and racy. For those, we try to come up with something that's a
better solution, and we send it upstream. This is one of the results.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html