Skip to content

tmpfiles.d: Add run/uscreens #463

Merged
merged 1 commit into from
May 13, 2025
Merged

tmpfiles.d: Add run/uscreens #463

merged 1 commit into from
May 13, 2025

Conversation

donald
Copy link
Collaborator

@donald donald commented May 12, 2025

We plan to remove setuid-root from screen. For that to work, /run/uscreens must be writable by normal users.

We plan to remove setuid-root from screen. For that to work,
/run/uscreens must be writable by normal users.
@pmenzel
Copy link
Contributor

pmenzel commented May 12, 2025

Debian has:

$ grep screen /lib/tmpfiles.d/*
/lib/tmpfiles.d/screen-cleanup.conf:d /run/screen 0777 root utmp

Is uscreens a widespread solution?

@pmenzel
Copy link
Contributor

pmenzel commented May 12, 2025

Ah, we configure it that way:

    bee_configure \
        --with-socket-dir=/run/uscreens \

@donald donald merged commit 60ca0d4 into master May 13, 2025
@pmenzel
Copy link
Contributor

pmenzel commented May 15, 2025

I think you said something about fixing something up on the running systems. Could you document, what you did? Just change the permission on /run/uscreens?

@donald
Copy link
Collaborator Author

donald commented May 17, 2025

From memory:

pdist push ALL --files /etc/tmpfiles.d/run-uscreens.conf --post "systemd-tmpfiles --create" --para 10

However, this was wrong, because the old (setuid) screen didn't like the (01777) mode, so we got errors and I had to push the new screen binary as well (pdist push ALL --files /usr/bin --para 10)

Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants