netdev - Re: [PATCH] kmod: don't load module unless req process has CAP_SYS

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: <20170515.141444.2109254753464149113.davem@davemloft.net>
Date:   Mon, 15 May 2017 14:14:44 -0400 (EDT)
From:   David Miller <davem@...emloft.net>
To:     maheshb@...gle.com
Cc:     gregkh@...uxfoundation.org, ebiederm@...ssion.com,
        mahesh@...dewar.net, mingo@...nel.org,
        linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
        keescook@...omium.org, edumazet@...gle.com
Subject: Re: [PATCH] kmod: don't load module unless req process has
 CAP_SYS_MODULE

From: Mahesh Bandewar (महेश बंडेवार) <maheshb@...gle.com>
Date: Mon, 15 May 2017 10:59:55 -0700

> The current behavior is already breaking things. e.g. unprivileged
> process can be root inside it's own user-ns. This will allow it to
> create IPtable rules causing contracking module to be loaded in
> default-ns affecting every flow on the server (not just the namespace
> that user or an unprivileged process is attached to). Cases that I
> mentioned above are just the tip of an iceberg.

Yes, that is certainly undesirable.

But is it really a module loading problem?  Perhaps we need to look
more deeply into how conntract behaves by default wrt. namespaces.

If we've given the user the ability to be root in his or her own
namespace, then we should let them do root stuff in there.

The only problem is when "doing root stuff in there" has an
undesirable impact upon the rest of the system.

And that's needs to be looked into on a facility by facility basis,
rather then just sprinkling "no module loading" test here and there,
or even unconditionally.