]> git.baikalelectronics.ru Git - kernel.git/commit
netns: publish net_generic correctly
authorAlexey Dobriyan <adobriyan@gmail.com>
Fri, 2 Dec 2016 01:11:34 +0000 (04:11 +0300)
committerDavid S. Miller <davem@davemloft.net>
Sat, 3 Dec 2016 20:59:58 +0000 (15:59 -0500)
commitfd41868a3c32b9e3f5ca604b70672de1fd695623
tree2e79422243969f9a2bda44053e8cb08a5cbd5166
parentfa897c2c7dd6bb539517861fc013cd24b4fb0a51
netns: publish net_generic correctly

Publishing net_generic pointer is done with silly mistake: new array is
published BEFORE setting freshly acquired pernet subsystem pointer.

memcpy
rcu_assign_pointer
kfree_rcu
ng->ptr[id - 1] = data;

This bug was introduced with commit 2d692957ee98cd40d80fa566bc217415a3b5acf6
("[NETNS]: The generic per-net pointers.") in the glorious days of
chopping networking stack into containers proper 8.5 years ago (whee...)

How it didn't trigger for so long?
Well, you need quite specific set of conditions:

*) race window opens once per pernet subsystem addition
   (read: modprobe or boot)

*) not every pernet subsystem is eligible (need ->id and ->size)

*) not every pernet subsystem is vulnerable (need incorrect or absense
   of ordering of register_pernet_sybsys() and actually using net_generic())

*) to hide the bug even more, default is to preallocate 13 pointers which
   is actually quite a lot. You need IPv6, netfilter, bridging etc together
   loaded to trigger reallocation in the first place. Trimmed down
   config are OK.

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/net_namespace.c