mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[PR #4042] [MERGED] private-lib hardening #5011
Labels
No labels
LTS merge
LTS merge
bug
bug
converted-to-discussion
doc-todo
documentation
duplicate
enhancement
file-transfer
firecfg
firejail-in-firejail
firetools
graphics
help wanted
information_old
installation
invalid
modif
moved
needinfo
networking
notabug
notourbug
old-version
overlayfs
packaging
profile-request
pull-request
question
question_old
removal
runtime-permissions
sandbox-ipc
security
stale
wiki
wiki
wontfix
wordpress
workaround
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/firejail#5011
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/netblue30/firejail/pull/4042
Author: @smitsohu
Created: 3/3/2021
Status: ✅ Merged
Merged: 3/5/2021
Merged by: @netblue30
Base:
master← Head:privatelib6📝 Commits (1)
dd2c1e5private-lib hardening📊 Changes
1 file changed (+29 additions, -2 deletions)
View changed files
📝
src/firejail/fs_lib.c(+29 -2)📄 Description
Ensure that libraries are loaded from below a default ld.so search path
It is reasonable for firejail to expect that unprivileged users have
no write permission on these paths; lax permissions there would
mean that the system is probably screwed anyway.
A somewhat similar check exists already in
private-bin:3565217343/src/firejail/fs_bin.c (L166)From a functional perspective this should be no limitation, because
private-libnot need to bother about libraries from paths that arenot masked in the end.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.