adjtimex: make it NOFORK
authorDenys Vlasenko <vda.linux@googlemail.com>
Sun, 6 Aug 2017 18:00:21 +0000 (20:00 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sun, 6 Aug 2017 18:00:21 +0000 (20:00 +0200)
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
NOFORK_NOEXEC.lst
miscutils/adjtimex.c

index 99af2435725083f83c2b634ccfad84f702a4ce09..97b7d8b20968368597b26f02606453305dc952de 100644 (file)
@@ -39,7 +39,7 @@ acpid - daemon
 add-shell
 addgroup
 adduser
-adjtimex
+adjtimex - NOFORK
 ar - runner
 arch - NOFORK
 arp - complex, rare
index a39c4f5cfa464514c2ea5b5a1111e8a61e0bddc6..ce6f8ccd8fde829118b268613fae8545bf4186a6 100644 (file)
@@ -18,7 +18,7 @@
 //config:      Adjtimex reads and optionally sets adjustment parameters for
 //config:      the Linux clock adjustment algorithm.
 
-//applet:IF_ADJTIMEX(APPLET(adjtimex, BB_DIR_SBIN, BB_SUID_DROP))
+//applet:IF_ADJTIMEX(APPLET_NOFORK(adjtimex, adjtimex, BB_DIR_SBIN, BB_SUID_DROP, adjtimex))
 
 //kbuild:lib-$(CONFIG_ADJTIMEX) += adjtimex.o
 
@@ -116,6 +116,11 @@ int adjtimex_main(int argc UNUSED_PARAM, char **argv)
                txc.modes |= ADJ_TICK;
        }
 
+       /* It's NOFORK applet because the code is very simple:
+        * just some printf. No opens, no allocs.
+        * If you need to make it more complex, feel free to downgrade to NOEXEC
+        */
+
        ret = adjtimex(&txc);
        if (ret < 0)
                bb_perror_nomsg_and_die();