chattr,lsattr,tune2fs: make them NOEXEC
authorDenys Vlasenko <vda.linux@googlemail.com>
Sat, 5 Aug 2017 18:38:04 +0000 (20:38 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sat, 5 Aug 2017 18:38:04 +0000 (20:38 +0200)
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
NOFORK_NOEXEC.lst
e2fsprogs/chattr.c
e2fsprogs/lsattr.c
e2fsprogs/tune2fs.c

index 78d06f3f51bc6840acd8457ac49021d51de6d732..0b6528d94f13d8c2d94af6c88dfb7b02d8d76d89 100644 (file)
@@ -61,7 +61,7 @@ bzip2 - runner
 cal - runner: cal -n9999
 cat - runner
 chat - needs ^C to work
-chattr - runner
+chattr - noexec. runner
 chgrp - noexec. runner
 chmod - noexec. runner
 chown - noexec. runner
@@ -204,7 +204,7 @@ lpd - daemon
 lpq - runner
 lpr - runner
 ls - noexec. runner
-lsattr - runner. noexec candidate (ls is, why not this one?)
+lsattr - noexec. runner
 lsmod - noexec
 lsof - complex
 lspci - noexec candidate, too rare to bother for nofork
@@ -366,7 +366,7 @@ truncate - NOFORK
 tty - NOFORK
 ttysize - NOFORK
 tunctl
-tune2fs - leaks: open+xfunc
+tune2fs - noexec. leaks: open+xfunc
 ubiattach
 ubidetach
 ubimkvol
index bb870a99061653de2076fe59c715a08dfa13e47a..76a5253b62d1b5c0da3973eec0df152c92549e60 100644 (file)
@@ -15,7 +15,7 @@
 //config:      help
 //config:      chattr changes the file attributes on a second extended file system.
 
-//applet:IF_CHATTR(APPLET(chattr, BB_DIR_BIN, BB_SUID_DROP))
+//applet:IF_CHATTR(APPLET_NOEXEC(chattr, chattr, BB_DIR_BIN, BB_SUID_DROP, chattr))
 
 //kbuild:lib-$(CONFIG_CHATTR) += chattr.o e2fs_lib.o
 
index 756d26832bf87979db9a27a4c2c496a23baf1536..56c1187c172ee6834c64f749a51e28ef44a8cab5 100644 (file)
@@ -16,7 +16,8 @@
 //config:      help
 //config:      lsattr lists the file attributes on a second extended file system.
 
-//applet:IF_LSATTR(APPLET(lsattr, BB_DIR_BIN, BB_SUID_DROP))
+//applet:IF_LSATTR(APPLET_NOEXEC(lsattr, lsattr, BB_DIR_BIN, BB_SUID_DROP, lsattr))
+/* ls is NOEXEC, so we should be too! ;) */
 
 //kbuild:lib-$(CONFIG_LSATTR) += lsattr.o e2fs_lib.o
 
index 95411db5f950d2dd6d1cd1d61c801a5abf592519..9f14b26ecbf76fc6cc544aed102efc062c70b16f 100644 (file)
@@ -13,7 +13,7 @@
 //config:      tune2fs allows the system administrator to adjust various tunable
 //config:      filesystem parameters on Linux ext2/ext3 filesystems.
 
-//applet:IF_TUNE2FS(APPLET(tune2fs, BB_DIR_SBIN, BB_SUID_DROP))
+//applet:IF_TUNE2FS(APPLET_NOEXEC(tune2fs, tune2fs, BB_DIR_SBIN, BB_SUID_DROP, tune2fs))
 
 //TODO alias to "tune2fs -L LABEL": //applet:IF_E2LABEL(APPLET_ODDNAME(e2label, tune2fs, BB_DIR_SBIN, BB_SUID_DROP, e2label))