Syslogd will not go to background if "-n" is given. Better help

and argument checking

Changelog

@@ -65,6 +65,8 @@
 	* Implemented 'ls -R' (enabled by enabling BB_FEATURE_LS_RECURSIVE)
 	* Implemented "ping -s", fixed error messages and argument parsing -
 	    Pavel Roskin
+	* Syslogd will not go to background if "-n" is given. Better help
+	    and argument checking -- Pavel Roskin
 	* More doc updates
 
 

docs/busybox.pod

@@ -1530,11 +1530,10 @@ Linux system and kernel (provides klogd) logging utility.
 Note that this version of syslogd/klogd ignores /etc/syslog.conf.
 
 Options:
-
+	-m NUM		Interval between MARK lines (default=20min, 0=off)
-        -m      Change the mark timestamp interval. default=20min. 0=off
+	-n		Run as a foreground process
-        -n      Do not fork into the background (for when run by init)
+	-K		Do not start up the klogd process
-        -K      Do not start up the klogd process (by default syslogd spawns klogd).
+	-O FILE		Use an alternate log file (default=/var/log/messages)
-        -O      Specify an alternate log file.  default=/var/log/messages
 
 -------------------------------
 
@@ -1949,4 +1948,4 @@ Enrique Zanardi <ezanardi@ull.es>
 
 =cut
 
-# $Id: busybox.pod,v 1.37 2000/06/07 20:38:15 proski Exp $
+# $Id: busybox.pod,v 1.38 2000/06/07 21:08:25 proski Exp $

sysklogd/syslogd.c

@@ -69,12 +69,12 @@ static const char syslogd_usage[] =
 	"\nLinux system and kernel (provides klogd) logging utility.\n"
 	"Note that this version of syslogd/klogd ignores /etc/syslog.conf.\n\n"
 	"Options:\n"
-	"\t-m\tChange the mark timestamp interval. default=20min. 0=off\n"
+	"\t-m NUM\t\tInterval between MARK lines (default=20min, 0=off)\n"
-	"\t-n\tDo not fork into the background (for when run by init)\n"
+	"\t-n\t\tRun as a foreground process\n"
 #ifdef BB_KLOGD
-	"\t-K\tDo not start up the klogd process (by default syslogd spawns klogd).\n"
+	"\t-K\t\tDo not start up the klogd process\n"
 #endif
-	"\t-O\tSpecify an alternate log file.  default=/var/log/messages\n"
+	"\t-O FILE\t\tUse an alternate log file (default=/var/log/messages)\n"
 #endif
 	;
 
@@ -170,6 +170,49 @@ static void domark(int sig)
 	}
 }
 
+#define BUFSIZE 1023
+static void serveConnection (int conn) __attribute__ ((noreturn));
+static void serveConnection (int conn)
+{
+	char   buf[ BUFSIZE + 1 ];
+	int    n_read;
+
+	while ((n_read = read (conn, buf, BUFSIZE )) > 0) {
+
+		int           pri = (LOG_USER | LOG_NOTICE);
+		char          line[ BUFSIZE + 1 ];
+		unsigned char c;
+
+		char *p = buf, *q = line;
+
+		buf[ n_read - 1 ] = '\0';
+
+		while (p && (c = *p) && q < &line[ sizeof (line) - 1 ]) {
+			if (c == '<') {
+			/* Parse the magic priority number. */
+				pri = 0;
+				while (isdigit (*(++p))) {
+					pri = 10 * pri + (*p - '0');
+				}
+				if (pri & ~(LOG_FACMASK | LOG_PRIMASK))
+					pri = (LOG_USER | LOG_NOTICE);
+			} else if (c == '\n') {
+				*q++ = ' ';
+			} else if (iscntrl (c) && (c < 0177)) {
+				*q++ = '^';
+				*q++ = c ^ 0100;
+			} else {
+				*q++ = c;
+			}
+			p++;
+		}
+		*q = '\0';
+		/* Now log it */
+		logMessage (pri, line);
+	}
+	exit (0);
+}
+
 static void doSyslogd (void) __attribute__ ((noreturn));
 static void doSyslogd (void)
 {
@@ -251,47 +294,8 @@ static void doSyslogd (void)
 						continue;
 					}
 
-					if (pid > 0) {
+					if (pid == 0)
-
+						serveConnection (conn);
-#						define BUFSIZE 1023
-						char   buf[ BUFSIZE + 1 ];
-						int    n_read;
-
-						while ((n_read = read (conn, buf, BUFSIZE )) > 0) {
-
-							int           pri = (LOG_USER | LOG_NOTICE);
-							char          line[ BUFSIZE + 1 ];
-							unsigned char c;
-
-							char *p = buf, *q = line;
-
-							buf[ n_read - 1 ] = '\0';
-
-							while (p && (c = *p) && q < &line[ sizeof (line) - 1 ]) {
-								if (c == '<') {
-								/* Parse the magic priority number. */
-									pri = 0;
-									while (isdigit (*(++p))) {
-										pri = 10 * pri + (*p - '0');
-									}
-									if (pri & ~(LOG_FACMASK | LOG_PRIMASK))
-										pri = (LOG_USER | LOG_NOTICE);
-								} else if (c == '\n') {
-									*q++ = ' ';
-								} else if (iscntrl (c) && (c < 0177)) {
-									*q++ = '^';
-									*q++ = c ^ 0100;
-								} else {
-									*q++ = c;
-								}
-								p++;
-							}
-							*q = '\0';
-							/* Now log it */
-							logMessage (pri, line);
-						}
-						exit (0);
-					}
 					close (conn);
 				}
 			}
@@ -428,6 +432,9 @@ extern int syslogd_main(int argc, char **argv)
 		}
 	}
 
+	if (argc > 0)
+		usage(syslogd_usage);
+
 	/* Store away localhost's name before the fork */
 	gethostname(LocalHostName, sizeof(LocalHostName));
 	if ((p = strchr(LocalHostName, '.'))) {

syslogd.c

@@ -69,12 +69,12 @@ static const char syslogd_usage[] =
 	"\nLinux system and kernel (provides klogd) logging utility.\n"
 	"Note that this version of syslogd/klogd ignores /etc/syslog.conf.\n\n"
 	"Options:\n"
-	"\t-m\tChange the mark timestamp interval. default=20min. 0=off\n"
+	"\t-m NUM\t\tInterval between MARK lines (default=20min, 0=off)\n"
-	"\t-n\tDo not fork into the background (for when run by init)\n"
+	"\t-n\t\tRun as a foreground process\n"
 #ifdef BB_KLOGD
-	"\t-K\tDo not start up the klogd process (by default syslogd spawns klogd).\n"
+	"\t-K\t\tDo not start up the klogd process\n"
 #endif
-	"\t-O\tSpecify an alternate log file.  default=/var/log/messages\n"
+	"\t-O FILE\t\tUse an alternate log file (default=/var/log/messages)\n"
 #endif
 	;
 
@@ -170,6 +170,49 @@ static void domark(int sig)
 	}
 }
 
+#define BUFSIZE 1023
+static void serveConnection (int conn) __attribute__ ((noreturn));
+static void serveConnection (int conn)
+{
+	char   buf[ BUFSIZE + 1 ];
+	int    n_read;
+
+	while ((n_read = read (conn, buf, BUFSIZE )) > 0) {
+
+		int           pri = (LOG_USER | LOG_NOTICE);
+		char          line[ BUFSIZE + 1 ];
+		unsigned char c;
+
+		char *p = buf, *q = line;
+
+		buf[ n_read - 1 ] = '\0';
+
+		while (p && (c = *p) && q < &line[ sizeof (line) - 1 ]) {
+			if (c == '<') {
+			/* Parse the magic priority number. */
+				pri = 0;
+				while (isdigit (*(++p))) {
+					pri = 10 * pri + (*p - '0');
+				}
+				if (pri & ~(LOG_FACMASK | LOG_PRIMASK))
+					pri = (LOG_USER | LOG_NOTICE);
+			} else if (c == '\n') {
+				*q++ = ' ';
+			} else if (iscntrl (c) && (c < 0177)) {
+				*q++ = '^';
+				*q++ = c ^ 0100;
+			} else {
+				*q++ = c;
+			}
+			p++;
+		}
+		*q = '\0';
+		/* Now log it */
+		logMessage (pri, line);
+	}
+	exit (0);
+}
+
 static void doSyslogd (void) __attribute__ ((noreturn));
 static void doSyslogd (void)
 {
@@ -251,47 +294,8 @@ static void doSyslogd (void)
 						continue;
 					}
 
-					if (pid > 0) {
+					if (pid == 0)
-
+						serveConnection (conn);
-#						define BUFSIZE 1023
-						char   buf[ BUFSIZE + 1 ];
-						int    n_read;
-
-						while ((n_read = read (conn, buf, BUFSIZE )) > 0) {
-
-							int           pri = (LOG_USER | LOG_NOTICE);
-							char          line[ BUFSIZE + 1 ];
-							unsigned char c;
-
-							char *p = buf, *q = line;
-
-							buf[ n_read - 1 ] = '\0';
-
-							while (p && (c = *p) && q < &line[ sizeof (line) - 1 ]) {
-								if (c == '<') {
-								/* Parse the magic priority number. */
-									pri = 0;
-									while (isdigit (*(++p))) {
-										pri = 10 * pri + (*p - '0');
-									}
-									if (pri & ~(LOG_FACMASK | LOG_PRIMASK))
-										pri = (LOG_USER | LOG_NOTICE);
-								} else if (c == '\n') {
-									*q++ = ' ';
-								} else if (iscntrl (c) && (c < 0177)) {
-									*q++ = '^';
-									*q++ = c ^ 0100;
-								} else {
-									*q++ = c;
-								}
-								p++;
-							}
-							*q = '\0';
-							/* Now log it */
-							logMessage (pri, line);
-						}
-						exit (0);
-					}
 					close (conn);
 				}
 			}
@@ -428,6 +432,9 @@ extern int syslogd_main(int argc, char **argv)
 		}
 	}
 
+	if (argc > 0)
+		usage(syslogd_usage);
+
 	/* Store away localhost's name before the fork */
 	gethostname(LocalHostName, sizeof(LocalHostName));
 	if ((p = strchr(LocalHostName, '.'))) {