From: Eric Andersen <andersen@codepoet.org>
Date: Mon, 12 Sep 2005 19:16:11 +0000 (-0000)
Subject: Kunihiko IMAI writes:
X-Git-Tag: 1_1_0~747
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2c6b41812febab045917c0be15ff8427191df0f4;p=platform%2Fupstream%2Fbusybox.git

Kunihiko IMAI writes:


We're using busybox and thank the authors.

I found a short buffer allocation at busybox/sysklod/syslod.c.  It
mis-declares length of filenames, seems to forget for '\0' or two
column numbered suffix of backup files (ex. messages.10).

Here is a patch for the problem.

Thanks.
---

diff --git a/sysklogd/syslogd.c b/sysklogd/syslogd.c
index aecd35d..8f9d75c 100644
--- a/sysklogd/syslogd.c
+++ b/sysklogd/syslogd.c
@@ -333,7 +333,7 @@ static void message(char *fmt, ...)
 				&& (lseek(fd,0,SEEK_END) > logFileSize) ) {
 				if(logFileRotate > 0) {
 					int i;
-					char oldFile[(strlen(logFilePath)+3)], newFile[(strlen(logFilePath)+3)];
+					char oldFile[(strlen(logFilePath)+4)], newFile[(strlen(logFilePath)+4)];
 					for(i=logFileRotate-1;i>0;i--) {
 						sprintf(oldFile, "%s.%d", logFilePath, i-1);
 						sprintf(newFile, "%s.%d", logFilePath, i);