mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[PR #1296] [MERGED] WIP: --novideo option #3912
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#3912
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/1296
Author: @Fred-Barclay
Created: 5/22/2017
Status: ✅ Merged
Merged: 5/23/2017
Merged by: @netblue30
Base:
master← Head:novideo📝 Commits (1)
cfbcbf2--novideo option📊 Changes
8 files changed (+389 additions, -361 deletions)
View changed files
📝
src/firejail/firejail.h(+11 -11)📝
src/firejail/fs_dev.c(+23 -13)📝
src/firejail/main.c(+145 -141)📝
src/firejail/profile.c(+67 -63)📝
src/firejail/sandbox.c(+76 -68)📝
src/firejail/usage.c(+21 -20)📝
src/man/firejail-profile.txt(+6 -7)📝
src/man/firejail.txt(+40 -38)📄 Description
Since
--private-devallows video devices as of1328d523ed, this gives us a way to block video devices in almost exactly the same way as--nosound.Still a work in progress. Code needs cleanup and improvement, but
it does block /dev/video* in all of my tests so far.
Please review and let me know your thoughts, but don't merge! 😄 It's still got some issues to be fixed.
EDIT: sorry for the large diff. My editor (Atom) automatically removes trailing whitespace.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.