github-starred/firejail
2
0
Fork 0
mirror of https://github.com/netblue30/firejail.git synced 2026-05-15 14:16:14 -06:00
Code Issues 531 Projects Packages Wiki Activity Actions 6

[GH-ISSUE #2854] Standard notes not working #1784

New issue
Closed
opened 2026-05-05 08:27:05 -06:00 by gitea-mirror · 18 comments
gitea-mirror commented 2026-05-05 08:27:05 -06:00
Owner
Copy link

Originally created by @Edu4rdSHL on GitHub (Jul 12, 2019).
Original GitHub issue: https://github.com/netblue30/firejail/issues/2854

I'm trying to run standardnotes-desktop from ArchLinux and I got:

sechacklabs@SecHackLabs ~> standardnotes-desktop 
Reading profile /etc/firejail/standardnotes-desktop.profile
Reading profile /etc/firejail/globals.local
Reading profile /etc/firejail/disable-common.inc
Reading profile /etc/firejail/disable-devel.inc
Reading profile /etc/firejail/disable-exec.inc
Reading profile /etc/firejail/disable-interpreters.inc
Reading profile /etc/firejail/disable-passwdmgr.inc
Reading profile /etc/firejail/disable-programs.inc
Reading profile /etc/firejail/whitelist-var-common.inc
Parent pid 22575, child pid 22576
Warning: skipping alternatives for private /etc
Warning: skipping pki for private /etc
Warning: skipping crypto-policies for private /etc
Private /etc installed in 22.47 ms
Warning: An abstract unix socket for session D-BUS might still be available. Use --net or remove unix from --protocol set.
Warning: cleaning all supplementary groups
Warning: cleaning all supplementary groups
Warning: /sbin directory link was not blacklisted
Warning: /usr/sbin directory link was not blacklisted
Child process initialized in 114.61 ms

Parent is shutting down, bye...

I tried removing unix from https://github.com/netblue30/firejail/blob/master/etc/standardnotes-desktop.profile#L36 but doesn't worked:

sechacklabs@SecHackLabs ~> standardnotes-desktop 
Reading profile /etc/firejail/standardnotes-desktop.profile
Reading profile /etc/firejail/globals.local
Reading profile /etc/firejail/disable-common.inc
Reading profile /etc/firejail/disable-devel.inc
Reading profile /etc/firejail/disable-exec.inc
Reading profile /etc/firejail/disable-interpreters.inc
Reading profile /etc/firejail/disable-passwdmgr.inc
Reading profile /etc/firejail/disable-programs.inc
Reading profile /etc/firejail/whitelist-var-common.inc
Parent pid 22535, child pid 22536
Warning: skipping alternatives for private /etc
Warning: skipping pki for private /etc
Warning: skipping crypto-policies for private /etc
Private /etc installed in 20.54 ms
Warning: cleaning all supplementary groups
Warning: cleaning all supplementary groups
Warning: /sbin directory link was not blacklisted
Warning: /usr/sbin directory link was not blacklisted
Child process initialized in 105.03 ms

Parent is shutting down, bye...
Originally created by @Edu4rdSHL on GitHub (Jul 12, 2019). Original GitHub issue: https://github.com/netblue30/firejail/issues/2854 I'm trying to run `standardnotes-desktop` from ArchLinux and I got: ``` sechacklabs@SecHackLabs ~> standardnotes-desktop Reading profile /etc/firejail/standardnotes-desktop.profile Reading profile /etc/firejail/globals.local Reading profile /etc/firejail/disable-common.inc Reading profile /etc/firejail/disable-devel.inc Reading profile /etc/firejail/disable-exec.inc Reading profile /etc/firejail/disable-interpreters.inc Reading profile /etc/firejail/disable-passwdmgr.inc Reading profile /etc/firejail/disable-programs.inc Reading profile /etc/firejail/whitelist-var-common.inc Parent pid 22575, child pid 22576 Warning: skipping alternatives for private /etc Warning: skipping pki for private /etc Warning: skipping crypto-policies for private /etc Private /etc installed in 22.47 ms Warning: An abstract unix socket for session D-BUS might still be available. Use --net or remove unix from --protocol set. Warning: cleaning all supplementary groups Warning: cleaning all supplementary groups Warning: /sbin directory link was not blacklisted Warning: /usr/sbin directory link was not blacklisted Child process initialized in 114.61 ms Parent is shutting down, bye... ``` I tried removing `unix` from https://github.com/netblue30/firejail/blob/master/etc/standardnotes-desktop.profile#L36 but doesn't worked: ``` sechacklabs@SecHackLabs ~> standardnotes-desktop Reading profile /etc/firejail/standardnotes-desktop.profile Reading profile /etc/firejail/globals.local Reading profile /etc/firejail/disable-common.inc Reading profile /etc/firejail/disable-devel.inc Reading profile /etc/firejail/disable-exec.inc Reading profile /etc/firejail/disable-interpreters.inc Reading profile /etc/firejail/disable-passwdmgr.inc Reading profile /etc/firejail/disable-programs.inc Reading profile /etc/firejail/whitelist-var-common.inc Parent pid 22535, child pid 22536 Warning: skipping alternatives for private /etc Warning: skipping pki for private /etc Warning: skipping crypto-policies for private /etc Private /etc installed in 20.54 ms Warning: cleaning all supplementary groups Warning: cleaning all supplementary groups Warning: /sbin directory link was not blacklisted Warning: /usr/sbin directory link was not blacklisted Child process initialized in 105.03 ms Parent is shutting down, bye... ```
gitea-mirror commented 2026-05-05 08:27:08 -06:00
Author
Owner
Copy link

@rusty-snake commented on GitHub (Jul 12, 2019):

@Edu4rdSHL What is in your globals.local?

removing unix

That will fully break standart-notes.

<!-- gh-comment-id:510804489 --> @rusty-snake commented on GitHub (Jul 12, 2019): @Edu4rdSHL What is in your `globals.local`? > removing `unix` That will fully break standart-notes.
gitea-mirror commented 2026-05-05 08:27:09 -06:00
Author
Owner
Copy link

@veloute commented on GitHub (Jul 12, 2019):

experiencing the same problem.
after a few minutes of playing around, seccomp and apparmor seem to have something to do with it.
try commenting both options out and see if that works for you (as a temp solution).

<!-- gh-comment-id:510852358 --> @veloute commented on GitHub (Jul 12, 2019): experiencing the same problem. after a few minutes of playing around, seccomp and apparmor seem to have something to do with it. try commenting both options out and see if that works for you (as a temp solution).
gitea-mirror commented 2026-05-05 08:27:10 -06:00
Author
Owner
Copy link

@rusty-snake commented on GitHub (Jul 12, 2019):

Related to #2821:

riot-desktop and standartnotes-desktop use both electron, right?

electron recently had an update with fixes of his own sandbox [1, 2, 3].

Maybe all electron-apps now need the chroot syscall.

@Edu4rdSHL try commet apparmor and seccomp as @veloute proposed and add the following:

seccomp.drop @clock,@cpu-emulation,@debug,@module,@obsolete,@raw-io,@reboot,@resources,@swap,acct,add_key,bpf,fanotify_init,io_cancel,io_destroy,io_getevents,io_setup,io_submit,ioprio_set,kcmp,keyctl,mincore,mount,name_to_handle_at,nfsservctl,ni_syscall,open_by_handle_at,pivot_root,remap_file_pages,request_key,setdomainname,sethostname,syslog,umount,umount2,userfaultfd,vhangup,vmsplice
<!-- gh-comment-id:510864625 --> @rusty-snake commented on GitHub (Jul 12, 2019): Related to #2821: riot-desktop and standartnotes-desktop use both electron, right? electron recently had an update with fixes of his own sandbox [[1], [2], [3]]. [1]: https://electronjs.org/releases/stable#release-notes-for-v506 [2]: https://github.com/electron/electron/pull/18798 [3]: https://github.com/electron/electron/commit/4318ad8f3a517fd6714b9a0361ef9de4e680b902 Maybe all electron-apps now need the `chroot` syscall. @Edu4rdSHL try commet `apparmor` and `seccomp` as @veloute proposed and add the following: ``` seccomp.drop @clock,@cpu-emulation,@debug,@module,@obsolete,@raw-io,@reboot,@resources,@swap,acct,add_key,bpf,fanotify_init,io_cancel,io_destroy,io_getevents,io_setup,io_submit,ioprio_set,kcmp,keyctl,mincore,mount,name_to_handle_at,nfsservctl,ni_syscall,open_by_handle_at,pivot_root,remap_file_pages,request_key,setdomainname,sethostname,syslog,umount,umount2,userfaultfd,vhangup,vmsplice ```
gitea-mirror commented 2026-05-05 08:27:11 -06:00
Author
Owner
Copy link

@veloute commented on GitHub (Jul 12, 2019):

we should probably add an include line for the electron profile in standardnotes-desktop's profile, right?

<!-- gh-comment-id:510865295 --> @veloute commented on GitHub (Jul 12, 2019): we should probably add an include line for the electron profile in standardnotes-desktop's profile, right?
gitea-mirror commented 2026-05-05 08:27:12 -06:00
Author
Owner
Copy link

@rusty-snake commented on GitHub (Jul 12, 2019):

electron.profile also still have seccomp.

But Streamlineing all electron-app-profile would may good. (But there are more than only standartnotes-desktop. e.g. teams-for-linux, telegram-desktop, ...)

I think the most *-desktop and some more.

<!-- gh-comment-id:510866454 --> @rusty-snake commented on GitHub (Jul 12, 2019): [electron.profile](https://github.com/netblue30/firejail/blob/master/etc/electron.profile) also still have `seccomp`. But Streamlineing all electron-app-profile would may good. (But there are more than only standartnotes-desktop. e.g. teams-for-linux, telegram-desktop, ...) I think the most `*-desktop` and some more.
gitea-mirror commented 2026-05-05 08:27:13 -06:00
Author
Owner
Copy link

@veloute commented on GitHub (Jul 12, 2019):

can confirm those changes are working for me; now just to wait for @Edu4rdSHL and make a pr with the changes if that's the fix needed.

<!-- gh-comment-id:510869094 --> @veloute commented on GitHub (Jul 12, 2019): can confirm those changes are working for me; now just to wait for @Edu4rdSHL and make a pr with the changes if that's the fix needed.
gitea-mirror commented 2026-05-05 08:27:14 -06:00
Author
Owner
Copy link

@Edu4rdSHL commented on GitHub (Jul 13, 2019):

Related to #2821:

riot-desktop and standartnotes-desktop use both electron, right?

electron recently had an update with fixes of his own sandbox [1, 2, 3].

Maybe all electron-apps now need the chroot syscall.

@Edu4rdSHL try commet apparmor and seccomp as @veloute proposed and add the following:

seccomp.drop @clock,@cpu-emulation,@debug,@module,@obsolete,@raw-io,@reboot,@resources,@swap,acct,add_key,bpf,fanotify_init,io_cancel,io_destroy,io_getevents,io_setup,io_submit,ioprio_set,kcmp,keyctl,mincore,mount,name_to_handle_at,nfsservctl,ni_syscall,open_by_handle_at,pivot_root,remap_file_pages,request_key,setdomainname,sethostname,syslog,umount,umount2,userfaultfd,vhangup,vmsplice

I tried it and worked, but now I've a question. In globals.local I had apparmor to enable it globally, now I need to remove it to get standard notes working.

Is not possible to have standard notes working with apparmor?

<!-- gh-comment-id:511072991 --> @Edu4rdSHL commented on GitHub (Jul 13, 2019): > Related to #2821: > > riot-desktop and standartnotes-desktop use both electron, right? > > electron recently had an update with fixes of his own sandbox [[1](https://electronjs.org/releases/stable#release-notes-for-v506), [2](https://github.com/electron/electron/pull/18798), [3](https://github.com/electron/electron/commit/4318ad8f3a517fd6714b9a0361ef9de4e680b902)]. > > Maybe all electron-apps now need the `chroot` syscall. > > @Edu4rdSHL try commet `apparmor` and `seccomp` as @veloute proposed and add the following: > > ``` > seccomp.drop @clock,@cpu-emulation,@debug,@module,@obsolete,@raw-io,@reboot,@resources,@swap,acct,add_key,bpf,fanotify_init,io_cancel,io_destroy,io_getevents,io_setup,io_submit,ioprio_set,kcmp,keyctl,mincore,mount,name_to_handle_at,nfsservctl,ni_syscall,open_by_handle_at,pivot_root,remap_file_pages,request_key,setdomainname,sethostname,syslog,umount,umount2,userfaultfd,vhangup,vmsplice > ``` I tried it and worked, but now I've a question. In `globals.local` I had `apparmor` to enable it globally, now I need to remove it to get standard notes working. Is not possible to have standard notes working with apparmor?
gitea-mirror commented 2026-05-05 08:27:15 -06:00
Author
Owner
Copy link

@veloute commented on GitHub (Jul 13, 2019):

create a standardnotes-desktop.local file in either /etc/firejail/ or /home/$USER/.config/firejail/ with the line'ignore apparmor

<!-- gh-comment-id:511073216 --> @veloute commented on GitHub (Jul 13, 2019): create a standardnotes-desktop.local file in either /etc/firejail/ or /home/$USER/.config/firejail/ with the line'`ignore apparmor`
gitea-mirror commented 2026-05-05 08:27:16 -06:00
Author
Owner
Copy link

@Edu4rdSHL commented on GitHub (Jul 13, 2019):

@veloute thanks, I know that, but my question is if denitively isn't possible to have the standard notes app with apparmor.

<!-- gh-comment-id:511073348 --> @Edu4rdSHL commented on GitHub (Jul 13, 2019): @veloute thanks, I know that, but my question is if denitively isn't possible to have the standard notes app with apparmor.
gitea-mirror commented 2026-05-05 08:27:17 -06:00
Author
Owner
Copy link

@rusty-snake commented on GitHub (Jul 13, 2019):

@Edu4rdSHL firejail-default is the apparmor profile that is used by firejail. Site-specific additions and overrides: local/firejail-local
Maybe this line in firejail-default causing the issue capability sys_chroot,.
I can' tell you more since I do not use AA.

EDIT: Why can I add reactions to my own posts ❓ 🤔

<!-- gh-comment-id:511104838 --> @rusty-snake commented on GitHub (Jul 13, 2019): @Edu4rdSHL `firejail-default` is the apparmor profile that is used by firejail. Site-specific additions and overrides: `local/firejail-local` Maybe this line in firejail-default causing the issue `capability sys_chroot,`. I can' tell you more since I do not use AA. EDIT: Why can I add reactions to my own posts :question: :thinking:
gitea-mirror commented 2026-05-05 08:27:18 -06:00
Author
Owner
Copy link

@veloute commented on GitHub (Jul 13, 2019):

fixed with dae5a61751

<!-- gh-comment-id:511111325 --> @veloute commented on GitHub (Jul 13, 2019): fixed with https://github.com/netblue30/firejail/commit/dae5a6175109d7f48c4d9e2cfec0d183fb068e44
gitea-mirror commented 2026-05-05 08:27:19 -06:00
Author
Owner
Copy link

@Vincent43 commented on GitHub (Jul 13, 2019):

@Edu4rdSHL Please post journalctl --grep=DENIED after failed attempt with apparmor enabled.

Maybe this line in firejail-default causing the issue capability sys_chroot,

This line only allows sys_chroot capability so it can't really break anything.

<!-- gh-comment-id:511128712 --> @Vincent43 commented on GitHub (Jul 13, 2019): @Edu4rdSHL Please post `journalctl --grep=DENIED` after failed attempt with `apparmor` enabled. > Maybe this line in firejail-default causing the issue capability sys_chroot, This line only allows `sys_chroot` capability so it can't really break anything.
gitea-mirror commented 2026-05-05 08:27:19 -06:00
Author
Owner
Copy link

@Edu4rdSHL commented on GitHub (Jul 13, 2019):

Hello, @Vincent43 I got it:

Jul 13 13:58:42 SecHackLabs audit[3764]: AVC apparmor="DENIED" operation="open" profile="firejail-default" name="/proc/15/comm" pid=3764 comm="electron" requested_mask="w" denied_mask="w" fsuid=1000 ouid=1000
Jul 13 13:58:42 SecHackLabs kernel: audit: type=1400 audit(1563044322.022:195): apparmor="DENIED" operation="open" profile="firejail-default" name="/proc/15/comm" pid=3764 comm="electron" requested_mask="w" denied_mask="w" fsuid=1000 ouid=1000
<!-- gh-comment-id:511146074 --> @Edu4rdSHL commented on GitHub (Jul 13, 2019): Hello, @Vincent43 I got it: ``` Jul 13 13:58:42 SecHackLabs audit[3764]: AVC apparmor="DENIED" operation="open" profile="firejail-default" name="/proc/15/comm" pid=3764 comm="electron" requested_mask="w" denied_mask="w" fsuid=1000 ouid=1000 Jul 13 13:58:42 SecHackLabs kernel: audit: type=1400 audit(1563044322.022:195): apparmor="DENIED" operation="open" profile="firejail-default" name="/proc/15/comm" pid=3764 comm="electron" requested_mask="w" denied_mask="w" fsuid=1000 ouid=1000 ```
gitea-mirror commented 2026-05-05 08:27:19 -06:00
Author
Owner
Copy link

@Vincent43 commented on GitHub (Jul 13, 2019):

@Edu4rdSHL Ok, try adding /proc/@{PID}/comm w, in /etc/apparmor.d/local/firejail-local and see if that helps.

You have to run apparmor_parser -r /etc/apparmor.d/firejail-default afterwards.

<!-- gh-comment-id:511148833 --> @Vincent43 commented on GitHub (Jul 13, 2019): @Edu4rdSHL Ok, try adding `/proc/@{PID}/comm w,` in `/etc/apparmor.d/local/firejail-local` and see if that helps. You have to run `apparmor_parser -r /etc/apparmor.d/firejail-default` afterwards.
gitea-mirror commented 2026-05-05 08:27:20 -06:00
Author
Owner
Copy link

@Edu4rdSHL commented on GitHub (Jul 13, 2019):

Worked, thanks.

<!-- gh-comment-id:511151917 --> @Edu4rdSHL commented on GitHub (Jul 13, 2019): Worked, thanks.
gitea-mirror commented 2026-05-05 08:27:20 -06:00
Author
Owner
Copy link

@Vincent43 commented on GitHub (Jul 14, 2019):

@Edu4rdSHL I will add it in default apparmor profile and re-enable apparmor in standard-notes then.

<!-- gh-comment-id:511198710 --> @Vincent43 commented on GitHub (Jul 14, 2019): @Edu4rdSHL I will add it in default apparmor profile and re-enable apparmor in standard-notes then.
gitea-mirror commented 2026-05-05 08:27:21 -06:00
Author
Owner
Copy link

@Vincent43 commented on GitHub (Jul 14, 2019):

Fixed with 2eca1252e3 . I think everything is covered now.

<!-- gh-comment-id:511199529 --> @Vincent43 commented on GitHub (Jul 14, 2019): Fixed with https://github.com/netblue30/firejail/commit/2eca1252e3491f098f036483855e3402882ebc54 . I think everything is covered now.
gitea-mirror commented 2026-05-05 08:27:21 -06:00
Author
Owner
Copy link

@Edu4rdSHL commented on GitHub (Jul 14, 2019):

Thanks for the fix.

<!-- gh-comment-id:511200118 --> @Edu4rdSHL commented on GitHub (Jul 14, 2019): Thanks for the fix.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
serialize_remounts_v2
landlock_v4
landlock
release-0.9.64
release-0.9.62
gitlab_ci
LTSbase
fred_ctests
docs
0.9.80
0.9.80-rc1
0.9.78
0.9.76
0.9.76-rc4
0.9.76-rc3
0.9.76-rc2
0.9.76-rc1
0.9.74
landlock-split
0.9.72
0.9.72rc1
0.9.70
0.9.68
0.9.68rc1
0.9.66
0.9.66rc1
0.9.64.4
0.9.64.2
0.9.64
0.9.64rc1
0.9.62.4
0.9.62.2
0.9.62
0.9.56.2-LTS
0.9.60
0.9.60-rc1
0.9.58.2
0.9.58
0.9.58-rc1
0.9.56-LTS-release
0.9.56-LTS
0.9.56
0.9.56-rc1
0.9.54
0.9.54-rc2
0.9.54-rc1
0.9.52
0.9.38.12
0.9.50
0.9.50-rc1
0.9.48
0.9.46
0.9.46-rc1
0.9.44.10
0.9.44.8
0.9.44.6
0.9.38.10
0.9.44.4
0.9.38.8
0.9.44.2
0.9.44
0.9.44-rc1
0.9.38.4
0.9.42
0.9.42-rc2
0.9.38.2
0.9.42-rc1
disable-globalcfg
0.9.40
0.9.40-rc1
0.9.38
0.9.38-rc1
0.9.36
0.9.36-rc1
0.9.34
0.9.34-rc1
0.9.32
0.9.32-rc1
0.9.30
0.9.30-rc1
Labels
Clear 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

Mirrored from GitHub Pull Request

  
question

   
question_old

   
removal

   
runtime-permissions

   
sandbox-ipc

   
security

   
stale

   
wiki

   
wiki

   
wontfix

   
wordpress

   
workaround
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
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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#1784
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?