syslogs: a bit better wording in explanation how buffering
authorDenis Vlasenko <vda.linux@googlemail.com>
Sat, 6 Jan 2007 22:08:53 +0000 (22:08 -0000)
committerDenis Vlasenko <vda.linux@googlemail.com>
Sat, 6 Jan 2007 22:08:53 +0000 (22:08 -0000)
is allocated

sysklogd/syslogd.c

index fbb85338abe145339707c087f79e3ed299dbc7ff..01120757b70d206d5981f210d47253cf5566755e 100644 (file)
@@ -58,16 +58,16 @@ static struct sockaddr_in remoteAddr;
 
 /* We are using bb_common_bufsiz1 for buffering: */
 enum { MAX_READ = (BUFSIZ/6) & ~0xf };
-/* We recv into this... (size: MAX_READ ~== BUFSIZ/6) */
+/* We recv into RECVBUF... (size: MAX_READ ~== BUFSIZ/6) */
 #define RECVBUF  bb_common_bufsiz1
-/* ...then copy here, escaping control chars */
-/* (can grow x2 + 1 max ~== BUFSIZ/3) */
+/* ...then copy to PARSEBUF, escaping control chars */
+/* (can grow x2 max ~== BUFSIZ/3) */
 #define PARSEBUF (bb_common_bufsiz1 + MAX_READ)
-/* ...then sprintf into this, adding timestamp (15 chars),
+/* ...then sprintf into PRINTBUF, adding timestamp (15 chars),
  * host (64), fac.prio (20) to the message */
 /* (growth by: 15 + 64 + 20 + delims = ~110) */
-#define PRINTBUF (bb_common_bufsiz1 + 3*MAX_READ + 0x10)
-/* totals: BUFSIZ/6 + BUFSIZ/3 + BUFSIZ/3 = BUFSIZ - BUFSIZ/6
+#define PRINTBUF (bb_common_bufsiz1 + 3*MAX_READ)
+/* totals: BUFSIZ == BUFSIZ/6 + BUFSIZ/3 + (BUFSIZ/3+BUFSIZ/6)
  * -- we have BUFSIZ/6 extra at the ent of PRINTBUF
  * which covers needed ~110 extra bytes (and much more) */
 
@@ -467,7 +467,7 @@ static void do_syslogd(void)
        int sock_fd;
        fd_set fds;
 
-       /* Set up signal handlers. */
+       /* Set up signal handlers */
        signal(SIGINT, quit_signal);
        signal(SIGTERM, quit_signal);
        signal(SIGQUIT, quit_signal);
@@ -510,7 +510,7 @@ static void do_syslogd(void)
 
                if (select(sock_fd + 1, &fds, NULL, NULL, NULL) < 0) {
                        if (errno == EINTR) {
-                               /* alarm may have happened. */
+                               /* alarm may have happened */
                                continue;
                        }
                        bb_perror_msg_and_die("select");
@@ -604,6 +604,5 @@ int syslogd_main(int argc, char **argv)
        }
        umask(0);
        do_syslogd();
-
-       return EXIT_SUCCESS;
+       /* return EXIT_SUCCESS; */
 }