Skip to content

Commit cac0be5

Browse files
nabijaczleweliandrewc12
authored andcommitted
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 <[email protected]> Signed-off-by: Ahelenia Ziemiańska <[email protected]> Closes openzfs#12544 Closes openzfs#13276
1 parent cb271c7 commit cac0be5

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

cmd/zed/zed.d/Makefile.am

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@ install-data-hook:
5454
-L "$(DESTDIR)$(zedconfdir)/$${f}" || \
5555
ln -s "$(zedexecdir)/$${f}" "$(DESTDIR)$(zedconfdir)"; \
5656
done
57-
chmod 0600 "$(DESTDIR)$(zedconfdir)/zed.rc"
5857

5958
# False positive: 1>&"${ZED_FLOCK_FD}" looks suspiciously similar to a >&filename bash extension
6059
CHECKBASHISMS_IGNORE = -e 'should be >word 2>&1' -e '&"$${ZED_FLOCK_FD}"'

cmd/zed/zed.d/zed.rc

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
##
2-
# zed.rc
3-
#
4-
# This file should be owned by root and permissioned 0600.
2+
# zed.rc – ZEDLET configuration.
53
##
64
# shellcheck disable=SC2034
75

0 commit comments

Comments
 (0)