[PR #1296] [MERGED] WIP: --novideo option #3912

Closed
opened 2026-05-05 10:10:15 -06:00 by gitea-mirror · 0 comments
Owner

📋 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: masterHead: novideo


📝 Commits (1)

📊 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-dev allows video devices as of 1328d523ed, 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.

## 📋 Pull Request Information **Original PR:** https://github.com/netblue30/firejail/pull/1296 **Author:** [@Fred-Barclay](https://github.com/Fred-Barclay) **Created:** 5/22/2017 **Status:** ✅ Merged **Merged:** 5/23/2017 **Merged by:** [@netblue30](https://github.com/netblue30) **Base:** `master` ← **Head:** `novideo` --- ### 📝 Commits (1) - [`cfbcbf2`](https://github.com/netblue30/firejail/commit/cfbcbf2c95455373aa2570827c52b7b87d80cfef) --novideo option ### 📊 Changes **8 files changed** (+389 additions, -361 deletions) <details> <summary>View changed files</summary> 📝 `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) </details> ### 📄 Description Since `--private-dev` allows video devices as of 1328d523ed62dfcdb205580ce7c126813b592cb7, 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. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
gitea-mirror 2026-05-05 10:10:15 -06:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/firejail#3912
No description provided.