Bug 41205 - Harden permissions for weston-launch
Summary: Harden permissions for weston-launch
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: weston (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: Yuri N. Sedunov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-10-24 23:52 MSK by Vladimir D. Seleznev
Modified: 2021-10-31 08:47 MSK (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Vladimir D. Seleznev 2021-10-24 23:52:04 MSK
Сейчас права на weston-launch установлены в 6711/-rws--s--x. При этом weston-launch нужен для того, чтобы мочь запустить сам weston в случае отсутствия logind.

Т.к. weston-launch в любом случае проверяет, что запустивший исполняемый файл пользователь входит в группу weston-launch, то я предлагаю при установке пакета создавать системную группу с таким именем, а права на запуск исполняемого файла выставить в 6710/-rws--s---.
Comment 1 Yuri N. Sedunov 2021-10-25 01:20:51 MSK
Нужен ли кому weston на системах без logind?
Comment 2 Vladimir D. Seleznev 2021-10-25 01:28:55 MSK
(In reply to Yuri N. Sedunov from comment #1)
> Нужен ли кому weston на системах без logind?

Ну вот я один из пользователей.
Comment 3 Yuri N. Sedunov 2021-10-25 01:44:53 MSK
Не хотите ли доверить членам группы xgrp запуск weston на таких системах?
Comment 4 Vladimir D. Seleznev 2021-10-25 01:52:44 MSK
(In reply to Yuri N. Sedunov from comment #3)
> Не хотите ли доверить членам группы xgrp запуск weston на таких системах?

Это был бы отличный вариант, но тогда придётся запатчить weston-launch:

https://github.com/wayland-project/weston/blob/58dba727c910665c7d4891b8b991226e4d3b28f2/libweston/weston-launch.c#L165
Comment 5 Yuri N. Sedunov 2021-10-25 02:05:07 MSK
Бинго! Предлагаю вам сделать такой патч для weston, чтобы можно было задавать группу в meson_options.txt. Со своей стороны обязуюсь одобрить задание в сизиф прежде, чем ваш патч примет апстрим.
Comment 6 Vladimir D. Seleznev 2021-10-31 06:41:23 MSK
(In reply to Yuri N. Sedunov from comment #5)
> Бинго! Предлагаю вам сделать такой патч для weston, чтобы можно было
> задавать группу в meson_options.txt. Со своей стороны обязуюсь одобрить
> задание в сизиф прежде, чем ваш патч примет апстрим.

https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/718
Comment 7 Repository Robot 2021-10-31 08:47:37 MSK
weston-9.0.0-alt1.2 -> sisyphus:

 Sun Oct 31 2021 Vladimir D. Seleznev <vseleznv@altlinux> 9.0.0-alt1.2
 - weston-launch: changed perms mode, subject of xgrp group (closes #41205)