mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[PR #319] [MERGED] Add compile-time option to restrict --net= to root only #3596
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#3596
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/319
Author: @yumkam
Created: 2/23/2016
Status: ✅ Merged
Merged: 2/24/2016
Merged by: @netblue30
Base:
master← Head:network-restricted📝 Commits (1)
4db1a65Add compile-time option to restrict --net= to root only📊 Changes
2 files changed (+9 additions, -0 deletions)
View changed files
📝
configure.ac(+3 -0)📝
src/firejail/main.c(+6 -0)📄 Description
./configure --enable-network=restricted allows only
--net=noneto non-root users.Other variants delegate too much power to non-root users and dangerous (it completely bypasses system-wide firewall and routing, it allows introducing arbitrary-chosen MAC and IP interfaces on LAN [disregarding DHCP policy], etc).
Root already had power to twiddle with anything, so no sense to restrain her, and
--net=nonelooks safe enough (and still useful) for ordinary users (and permitted viaunshare -rnanyway).See #290
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.