mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[GH-ISSUE #2142] Firefox 61.0.1.glibc2.7 Appimage with multi-process windows enabled does not successfully load pages #1450
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#1450
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?
Originally created by @crass on GitHub (Oct 6, 2018).
Original GitHub issue: https://github.com/netblue30/firejail/issues/2142
Using up to date master with #2141 added, the
61.0.1.glibc2.7firefox appimage displays, with saved tabs when using a pre-existing profile. However none of the tabs load, includingfile:///urls. Version56.0.2.glibc2.3.4works just fine. I would presume all the latest profile fixes are in master, is that a wrong assumption? Has anyone else gotten 61 non-appimage to run?https://bintray.com/probono/AppImages/Firefox/61.0.1.glibc2.7#files
@chiraag-nataraj commented on GitHub (Oct 6, 2018):
I currently have a non-appimage
62.0.3running just fine on Debian sid/experimental with a more restrictive profile than the default (mine's here).@crass commented on GitHub (Oct 8, 2018):
I've tried the latest appimage
62.0.2and get the same symptoms. I've disabled multi-process windows (see bottom), and now both are working as expected. Do you have multi-process windows enabled?@chiraag-nataraj commented on GitHub (Oct 8, 2018):
Yup, I've had it enabled since they first made it available.
@crass commented on GitHub (Oct 8, 2018):
Ok, I've narrowed it down to this code. If I remove
arg_appimagefrom the conditional and recompile, then the firefox appimage will load content in tabs when mutli-process windows is turned on (ie. work as expected). Basically that code is causing any specificseccompconfiguration (which firefox.profile has an needs) to be thrown out and to use a default config.@netblue30, so what was the rationale behind including appimages here? It seems that the decision was made in commit
2399e09e66, but its not clear to me why. It seems to me that appimages must not be any more strict than the corresponding non-appimage profile (the program will need all the rules of the non-appimage profile plus perhaps some permission unique to appimages).@crass commented on GitHub (Oct 16, 2018):
After digging deeper, the appimage will load with
enforce_filtersifchrootis removed from@defaults. Afterstraceing the appimage, it can be seen thatchrootand before thatcapsetwithCAP_SYS_CHROOTis being called. It appears that the sandbox is setting up a user namespace where it is root inside the sandbox, and thus can give itself capabilities.Does anyone have thoughts on how to solve this? I'm thinking that appimage mode should only use the default seccomp if one wasn't provided. If one was provided the user clearly wanted to use that one. Also, in the vast majority of cases, the user will specify a specific profile for an appimage because the command name matching will not generally work for appimages. So, the user provided profile will have the appropriate seccomp, which may not be as restrictive as the default seccomp.