diff options
author | наб <nabijaczleweli@nabijaczleweli.xyz> | 2022-04-05 20:50:55 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-04-05 20:50:55 +0000 |
commit | ed715283de8e65a30d777e9576399ab75014b6fe (patch) | |
tree | 90d296e2c1537bf4e05b922bc759c73233f06071 | |
parent | 4d972ab5aefb2499609f6327d0cfa5ab52856205 (diff) | |
download | src-ed715283de8e65a30d777e9576399ab75014b6fe.tar.gz src-ed715283de8e65a30d777e9576399ab75014b6fe.zip |
cmd: zed: rc: drop "should be owned by root and 0600"
It doesn't matter, 0600 are Weird Permissions, and it's even weirder to
spec them for no reason ‒ it's perfectly fine if it's the usual 0:0 644,
or literally anything else, so long as unprivileged users can't edit it
(which (a) 644 accomplishes and (b) is at the administrator's
discretion, it's not unheard of to have adm users and having it
be 664 in that case is just as good; it's not our place to say)
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
Closes #12544
Closes #13276
-rw-r--r-- | cmd/zed/zed.d/Makefile.am | 1 | ||||
-rw-r--r-- | cmd/zed/zed.d/zed.rc | 4 |
2 files changed, 1 insertions, 4 deletions
diff --git a/cmd/zed/zed.d/Makefile.am b/cmd/zed/zed.d/Makefile.am index 3c0f5c3dd1b2..3efcfb3629cf 100644 --- a/cmd/zed/zed.d/Makefile.am +++ b/cmd/zed/zed.d/Makefile.am @@ -54,7 +54,6 @@ install-data-hook: -L "$(DESTDIR)$(zedconfdir)/$${f}" || \ ln -s "$(zedexecdir)/$${f}" "$(DESTDIR)$(zedconfdir)"; \ done - chmod 0600 "$(DESTDIR)$(zedconfdir)/zed.rc" # False positive: 1>&"${ZED_FLOCK_FD}" looks suspiciously similar to a >&filename bash extension CHECKBASHISMS_IGNORE = -e 'should be >word 2>&1' -e '&"$${ZED_FLOCK_FD}"' diff --git a/cmd/zed/zed.d/zed.rc b/cmd/zed/zed.d/zed.rc index a8fdc86c3431..3c58a2c281f5 100644 --- a/cmd/zed/zed.d/zed.rc +++ b/cmd/zed/zed.d/zed.rc @@ -1,7 +1,5 @@ ## -# zed.rc -# -# This file should be owned by root and permissioned 0600. +# zed.rc – ZEDLET configuration. ## # shellcheck disable=SC2034 |