Thread (49 messages) 49 messages, 11 authors, 2017-11-30

Re: [PATCH v5 next 0/5] Improve Module autoloading infrastructure

From: Djalal Harouni <hidden>
Date: 2017-11-27 21:31:58
Also in: linux-security-module, lkml

Hi Linus,

On Mon, Nov 27, 2017 at 8:12 PM, Linus Torvalds
[off-list ref] wrote:
On Mon, Nov 27, 2017 at 11:02 AM, Linus Torvalds
[off-list ref] wrote:
quoted
Now, the above will not necessarily work with a legacy /dev/ directory
where al the nodes have been pre-populated, and opening the device
node is supposed to load the module. So _historically_ we did indeed
load modules as normal users. But does that really happen any more?
Sadly, it looks like bluetoothd actually does expect to load the
bt-proto-XYZ modules with no capabilities at all.

So apparently we really do depend on not needing capabilities for
module loading.

Oh well.
Yes DCCP is unprivileged, tun and all tunneling, some md drivers, some
crypto, and device drivers... fs modules can be loaded inside
usernamespaces, and maybe when some request requires external symbols
too...

However tunneling helps to solve real usecases, so that's why the
backward compatibility and opt-in.

I do perfectly understand that opt-in is not the best choice, however
these patchset includes a per process tree, and given that lot of code
is running in containers and sandboxes, it is better than nothing. I
will follow up later with patches to the major ones especially when we
force the flag by default. Ubuntu was said to be owned in a past
security contest due to this kind of things, and now since they have
ubuntu snaps or apps they can set the flag, and others will follow.

Thanks!
                 Linus


-- 
tixxdz
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help