From 8a820b2732ec1672401cb5117437adcdc652b18b Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Sat, 6 Jan 2007 22:08:53 +0000 Subject: syslogs: a bit better wording in explanation how buffering is allocated --- sysklogd/syslogd.c | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/sysklogd/syslogd.c b/sysklogd/syslogd.c index fbb8533..0112075 100644 --- a/sysklogd/syslogd.c +++ b/sysklogd/syslogd.c @@ -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; */ } -- cgit v1.1