mherrb/seatd
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert "libseat: Check euid before using builtin"

Browse source 
This reverts commit 1ae6c3b3dd.

A user might want to run the builtin server as non root, if they have
permission to use the devices.

The check was originally copied from wlroots's direct backend. It was reverted
in fa05d3cde68d with a detailed explanation of why root priviledges are not
always necessary to use the DRM device.
This commit is contained in:
Greg Depoire--Ferrer 2021-08-04 07:55:28 +02:00 • committed by Kenny Levinsen
parent 6e7a1db32d
commit d9ae4c3010
1 changed files with 0 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

5
libseat/backend/seatd.c
View file

@ -606,11 +606,6 @@ static struct libseat *builtin_open_seat(struct libseat_seat_listener *listener,
return NULL; return NULL;
} }
if (geteuid() != 0) {
log_error("Built-in seatd instance requires root privileges");
return NULL;
}
pid_t pid = fork(); pid_t pid = fork();
if (pid == -1) { if (pid == -1) {
log_errorf("Could not fork: %s", strerror(errno)); log_errorf("Could not fork: %s", strerror(errno));