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 #3965] firejail fails to start if iBus is not in use, but .config/ibus/bus/* exists. #2481

New issue
Closed
opened 2026-05-05 09:09:57 -06:00 by gitea-mirror · 5 comments
gitea-mirror commented 2026-05-05 09:09:57 -06:00
Owner
Copy link

Originally created by @oranenj on GitHub (Feb 9, 2021).
Original GitHub issue: https://github.com/netblue30/firejail/issues/3965

I happened to have this in my .config:

cat .config/ibus/bus/75b2ea12f9fc4116ad6d2c56e1f24f6e-unix-0 
IBUS_ADDRESS=
IBUS_DAEMON_PID=

This causes firejail to exit immediately on startup with the following cryptic error message:

$ firejail --debug
Autoselecting /bin/bash as shell
Command name #/bin/bash#
Attempting to find default.profile...
Found default.profile profile in /etc/firejail directory
Reading profile /etc/firejail/default.profile
Found disable-common.inc profile in /etc/firejail directory
Reading profile /etc/firejail/disable-common.inc
Found disable-passwdmgr.inc profile in /etc/firejail directory
Reading profile /etc/firejail/disable-passwdmgr.inc
Found disable-programs.inc profile in /etc/firejail directory
Reading profile /etc/firejail/disable-programs.inc

** Note: you can use --noprofile to disable default.profile **

DISPLAY=:0 parsed as 0
Using the local network stack
Parent pid 6162, child pid 6163
Initializing child process
Host network configured
PID namespace installed
Mounting tmpfs on /run/firejail/mnt directory
Creating empty /run/firejail/mnt/seccomp directory
Creating empty /run/firejail/mnt/seccomp/seccomp.protocol file
Creating empty /run/firejail/mnt/seccomp/seccomp.postexec file
Creating empty /run/firejail/mnt/seccomp/seccomp.postexec32 file
IBUS_ADDRESS=
Error: invalid --env setting
Error: proc 6162 cannot sync with peer: unexpected EOF
Peer 6163 unexpectedly exited with status 1

My desktop environment does not use IBus and I'm not sure where those files came from, but removing them allows firejail to run normally. I'm assuming this code is the problem: 06e6dfe31a/src/firejail/sandbox.c (L778)

firejail should probably ignore the environment variables in those files if they are malformed or empty.

Originally created by @oranenj on GitHub (Feb 9, 2021). Original GitHub issue: https://github.com/netblue30/firejail/issues/3965 I happened to have this in my .config: ``` cat .config/ibus/bus/75b2ea12f9fc4116ad6d2c56e1f24f6e-unix-0 IBUS_ADDRESS= IBUS_DAEMON_PID= ``` This causes firejail to exit immediately on startup with the following cryptic error message: ``` $ firejail --debug Autoselecting /bin/bash as shell Command name #/bin/bash# Attempting to find default.profile... Found default.profile profile in /etc/firejail directory Reading profile /etc/firejail/default.profile Found disable-common.inc profile in /etc/firejail directory Reading profile /etc/firejail/disable-common.inc Found disable-passwdmgr.inc profile in /etc/firejail directory Reading profile /etc/firejail/disable-passwdmgr.inc Found disable-programs.inc profile in /etc/firejail directory Reading profile /etc/firejail/disable-programs.inc ** Note: you can use --noprofile to disable default.profile ** DISPLAY=:0 parsed as 0 Using the local network stack Parent pid 6162, child pid 6163 Initializing child process Host network configured PID namespace installed Mounting tmpfs on /run/firejail/mnt directory Creating empty /run/firejail/mnt/seccomp directory Creating empty /run/firejail/mnt/seccomp/seccomp.protocol file Creating empty /run/firejail/mnt/seccomp/seccomp.postexec file Creating empty /run/firejail/mnt/seccomp/seccomp.postexec32 file IBUS_ADDRESS= Error: invalid --env setting Error: proc 6162 cannot sync with peer: unexpected EOF Peer 6163 unexpectedly exited with status 1 ``` My desktop environment does not use IBus and I'm not sure where those files came from, but removing them allows firejail to run normally. I'm assuming this code is the problem: https://github.com/netblue30/firejail/blob/06e6dfe31ab23f0ed4e56799efffc03dfb1e2c0e/src/firejail/sandbox.c#L778 firejail should probably ignore the environment variables in those files if they are malformed or empty.
gitea-mirror 2026-05-05 09:09:57 -06:00
  • closed this issue
  • added the
    bug
         label
gitea-mirror commented 2026-05-05 09:10:00 -06:00
Author
Owner
Copy link

@oranenj commented on GitHub (Feb 9, 2021):

Looking at the code, the problem seems to be the env_store function is called with SETENV and not SETENV_ALLOW_EMPTY, causing it to fail.

<!-- gh-comment-id:776122421 --> @oranenj commented on GitHub (Feb 9, 2021): Looking at the code, the problem seems to be the `env_store` function is called with `SETENV` and not `SETENV_ALLOW_EMPTY`, causing it to fail.
gitea-mirror commented 2026-05-05 09:10:01 -06:00
Author
Owner
Copy link

@topimiettinen commented on GitHub (Feb 11, 2021):

Current Firejail moves the environment variables away, so most of them are not used by the main setuid executable nor the unprivileged helper programs. The variables are restored for the final application. With this in mind, I think the checks for empty variables could be lifted.

<!-- gh-comment-id:777501120 --> @topimiettinen commented on GitHub (Feb 11, 2021): Current Firejail moves the environment variables away, so most of them are not used by the main setuid executable nor the unprivileged helper programs. The variables are restored for the final application. With this in mind, I think the checks for empty variables could be lifted.
gitea-mirror commented 2026-05-05 09:10:01 -06:00
Author
Owner
Copy link

@topimiettinen commented on GitHub (Feb 11, 2021):

@oranenj Could you test #3974?

<!-- gh-comment-id:777539427 --> @topimiettinen commented on GitHub (Feb 11, 2021): @oranenj Could you test #3974?
gitea-mirror commented 2026-05-05 09:10:02 -06:00
Author
Owner
Copy link

@oranenj commented on GitHub (Feb 11, 2021):

@topimiettinen built and seems to work. Old version of firejail fails, the patched one runs fine. I tested running my sandboxed browser with it, and nothing obvious broke, so LGTM.

<!-- gh-comment-id:777600525 --> @oranenj commented on GitHub (Feb 11, 2021): @topimiettinen built and seems to work. Old version of firejail fails, the patched one runs fine. I tested running my sandboxed browser with it, and nothing obvious broke, so LGTM.
gitea-mirror commented 2026-05-05 09:10:02 -06:00
Author
Owner
Copy link

@topimiettinen commented on GitHub (Feb 11, 2021):

Thanks for testing!

<!-- gh-comment-id:777695854 --> @topimiettinen commented on GitHub (Feb 11, 2021): Thanks for testing!
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#2481
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?