Skip to content

screen: Update version to 4.9.1 #3240

Merged
merged 1 commit into from
May 13, 2025
Merged

screen: Update version to 4.9.1 #3240

merged 1 commit into from
May 13, 2025

Conversation

donald
Copy link
Collaborator

@donald donald commented May 12, 2025

Remove setuid-root from screen and update to 4.9.1.
Requires mariux64/mxtools#463
Update screen to 4.9.1.

See https://www.openwall.com/lists/oss-security/2025/05/12/1

Update screen to 4.9.1.

More importantly, don't install screen as setuid-root. this is
only required for Multiuser Sessions, which are probably not used in our
environment.

Screen has a lot of setuid-root related security problems. [1]

Do not update to 5.0.0 yet.

[1]: https://www.openwall.com/lists/oss-security/2025/05/12/1
@pmenzel
Copy link
Collaborator

pmenzel commented May 12, 2025

It’s a little confusing as you reference the email regarding the security issues, but 4.9.1 does not contain anything about this as it’s from 2023. So only the setuid-root relates to the oss-security posting.

@donald
Copy link
Collaborator Author

donald commented May 13, 2025

The point of the PR is "Remove setuid-root from screen". The update (to latest 4.9) is just by the way. How can this be expressed more clearly?

From the referenced document:

Given all this, we don't recommend to install Screen setuid-root at all
at the moment (neither version 5.0.0 nor the older 4.9 versions).

And I agree with that. We've had a root exploit with screen ourself before and were not happy with how it was handled [1].

I wasn't aware that setuid-root is only needed for a feature, which might be unused in our environment. So removing setuid-root is the way to go.

[1] https://lists.gnu.org/archive/html/screen-devel/2017-03/msg00001.html

@donald donald merged commit 76184a7 into master May 13, 2025
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