Merge branch 'enforce-global-flow-dissector'
authorAlexei Starovoitov <ast@kernel.org>
Tue, 8 Oct 2019 03:16:34 +0000 (20:16 -0700)
committerAlexei Starovoitov <ast@kernel.org>
Tue, 8 Oct 2019 03:16:34 +0000 (20:16 -0700)
commit72ccd9200f218a7eb2933a93e69a79a1ac984d83
treea20d775996eee47d44f32888ab7cece80a115885
parent4564a8bb57e4240c08ce2db5194bfa2e71f57730
parent1d9626dc08bf0f5c6932b98e0f4dc5d6b305786f
Merge branch 'enforce-global-flow-dissector'

Stanislav Fomichev says:

====================
While having a per-net-ns flow dissector programs is convenient for
testing, security-wise it's better to have only one vetted global
flow dissector implementation.

Let's have a convention that when BPF flow dissector is installed
in the root namespace, child namespaces can't override it.

The intended use-case is to attach global BPF flow dissector
early from the init scripts/systemd. Attaching global dissector
is prohibited if some non-root namespace already has flow dissector
attached. Also, attaching to non-root namespace is prohibited
when there is flow dissector attached to the root namespace.

v3:
* drop extra check and empty line (Andrii Nakryiko)

v2:
* EPERM -> EEXIST (Song Liu)
* Make sure we don't have dissector attached to non-root namespaces
  when attaching the global one (Andrii Nakryiko)
====================

Signed-off-by: Alexei Starovoitov <ast@kernel.org>