mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[GH-ISSUE #595] Firejail needs root to display version(?!) #419
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#419
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 @chiraag-nataraj on GitHub (Jun 25, 2016).
Original GitHub issue: https://github.com/netblue30/firejail/issues/595
Steps to reproduce:
firejail bashfirejail --versionInstead of getting the version, I get
@netblue30 commented on GitHub (Jun 26, 2016):
Did you install it SUID?
@chiraag-nataraj commented on GitHub (Jun 26, 2016):
Yup. (This is within the jail, hence the different UID and GID).
I think I wasn't clear - this is within the jail (not outside).
@netblue30 commented on GitHub (Jun 27, 2016):
Not a problem, the jail actually will prevent a new jail for starting - firejail is a SUID executable, and SUID executables are disabled.
@chiraag-nataraj commented on GitHub (Jun 27, 2016):
Huh...okay. I realized that, but it's interesting that it would have to set the UID and GID to root before it even did the argument parsing (which would tell it that it just has to display the version).
@netblue30 commented on GitHub (Jun 27, 2016):
Good point! I'll have to fix it.
@chiraag-nataraj commented on GitHub (Jun 27, 2016):
😄
@netblue30 commented on GitHub (Jun 28, 2016):
Fixed, thanks for the bug.