ntpd: do not create pid file if not backgrounded
authorDenys Vlasenko <vda.linux@googlemail.com>
Wed, 27 Mar 2019 12:37:40 +0000 (13:37 +0100)
committerDenys Vlasenko <vda.linux@googlemail.com>
Wed, 27 Mar 2019 12:47:38 +0000 (13:47 +0100)
function                                             old     new   delta
ntp_init                                             995    1021     +26
ntpd_main                                           1267    1257     -10
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 1/1 up/down: 26/-10)             Total: 16 bytes

Reported by Alfonso Ranieri <alforan@tin.it>

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
networking/ntpd.c

index b2e77929e0c654c33025e013eebcd3c65fd73f1f..027cfe783a469af019ad2a94dbe56a42f9dac3f5 100644 (file)
@@ -2585,6 +2585,10 @@ static NOINLINE void ntp_init(char **argv)
                /* -l but no peers: "stratum 1 server" mode */
                G.stratum = 1;
        }
+
+       if (!(opts & OPT_n)) /* only if backgrounded: */
+               write_pidfile_std_path_and_ext("ntpd");
+
        /* If network is up, syncronization occurs in ~10 seconds.
         * We give "ntpd -q" 10 seconds to get first reply,
         * then another 50 seconds to finish syncing.
@@ -2641,8 +2645,6 @@ int ntpd_main(int argc UNUSED_PARAM, char **argv)
         */
        cnt = G.peer_cnt * (INITIAL_SAMPLES + 1);
 
-       write_pidfile_std_path_and_ext("ntpd");
-
        while (!bb_got_signal) {
                llist_t *item;
                unsigned i, j;