mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[PR #3087] [MERGED] Fix Brave's native sandbox #4637
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#4637
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/3087
Author: @ghost
Created: 12/17/2019
Status: ✅ Merged
Merged: 12/19/2019
Merged by: @undefined
Base:
master← Head:brave-fix📝 Commits (4)
e751ceaAllow user access to /proc/config.gze368095Fix Brave's native sandbox7bacc9eMove /proc/config.gz to disable-common.inca9ace9bMove /proc/config.gz to disable-common.inc📊 Changes
3 files changed (+8 additions, -1 deletions)
View changed files
📝
etc/brave.profile(+3 -0)📝
etc/disable-common.inc(+3 -0)📝
src/firejail/fs.c(+2 -1)📄 Description
IMPORTANT: leave this unmerged until collaborators have had ample opportunity to review.
This is a tentative fix for #2944. The patch enables using
noblacklist /proc/config.gzin/etc/firejail/brave.profile. Momentarily this is only needed for that one profile, so I'm not entirely convinced this is the best way to fix #2914. Thoughts?🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.