diff options
author | Patrick Steinhardt <ps@pks.im> | 2020-09-03 21:25:33 +0200 |
---|---|---|
committer | Simon Ser <contact@emersion.fr> | 2020-09-04 11:39:25 +0200 |
commit | fa05d3cde68de73df4cac9762f6623c6001651cc (patch) | |
tree | c35444bcde0cd73761ac8d400cbb6bbc03a3f6e1 /backend/session/logind.c | |
parent | efe6414640aa15cfad975dec3db427936c0450a4 (diff) |
session: Don't refuse unprivileged creation of "direct" backend
When starting a compositor that's using the "direct" session backend,
wlroots needs to handle calls to `drmSetMaster()` and `drmDropMaster()`.
As both calls used to require `CAP_SYS_ADMIN`, wlroots thus simply
refused starting in case the process doesn't enjoy evelated privileges.
Permission rules have changed since linux.git commit 45bc3d26c95a (drm:
rework SET_MASTER and DROP_MASTER perm handling, 2020-03-19). As a
result, starting with Linux v5.8, both ioctls will now also succeed if
the process is currently or has been the DRM master. And as the first
process to open render nodes will become the DRM master automatically,
this effectively means that process elevation is not strictly required
in all setups anymore.
So let's drop the `geteuid() != 0` permission check to allow those new
rules to do their magic.
Diffstat (limited to 'backend/session/logind.c')
0 files changed, 0 insertions, 0 deletions