summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenis Vlasenko2007-11-06 11:34:03 +0000
committerDenis Vlasenko2007-11-06 11:34:03 +0000
commitcb12cb240714f2599addd4ec61ef336ab87482cd (patch)
treeeb218e0c85ceb81753cb3a27535b93069d4ebb7a
parent52816302299854ba1644fce98b5d19db526e6c29 (diff)
downloadbusybox-cb12cb240714f2599addd4ec61ef336ab87482cd.zip
busybox-cb12cb240714f2599addd4ec61ef336ab87482cd.tar.gz
modprobe: fix a bug where we were entering endless loop
syslogd: strip trailing '\n' too, not only NULs
-rw-r--r--modutils/modprobe.c2
-rw-r--r--sysklogd/syslogd.c12
2 files changed, 12 insertions, 2 deletions
diff --git a/modutils/modprobe.c b/modutils/modprobe.c
index f7d193a..dafbb4e 100644
--- a/modutils/modprobe.c
+++ b/modutils/modprobe.c
@@ -791,7 +791,7 @@ static void check_dep(char *mod, struct mod_list_t **head, struct mod_list_t **t
if (*tail)
(*tail)->m_next = find;
find->m_prev = *tail;
- /*find->m_next = NULL; - xzalloc did it */
+ find->m_next = NULL; /* possibly NOT done by xzalloc! */
if (!*head)
*head = find;
diff --git a/sysklogd/syslogd.c b/sysklogd/syslogd.c
index 2bf5b5c..da63ced 100644
--- a/sysklogd/syslogd.c
+++ b/sysklogd/syslogd.c
@@ -536,10 +536,20 @@ static void do_syslogd(void)
while (1) {
if (sz == 0)
goto read_again;
- if (G.recvbuf[sz-1])
+ /* man 3 syslog says: "A trailing newline is added when needed".
+ * However, neither glibc nor uclibc do this:
+ * syslog(prio, "test") sends "test\0" to /dev/log,
+ * syslog(prio, "test\n") sends "test\n\0",
+ * IOW: newline is passed verbatim!
+ * I take it to mean that it's syslogd's job
+ * to make those look identical in the log files */
+ if (G.recvbuf[sz-1] && G.recvbuf[sz-1] != '\n')
break;
sz--;
}
+ /* Maybe we need to add '\n' here, not later?
+ * It looks like stock syslogd does send '\n' over network,
+ * but we do not (see sendto below) */
G.recvbuf[sz] = '\0'; /* make sure it *is* NUL terminated */
/* TODO: maybe suppress duplicates? */