From 2c6b41812febab045917c0be15ff8427191df0f4 Mon Sep 17 00:00:00 2001 From: Eric Andersen Date: Mon, 12 Sep 2005 19:16:11 +0000 Subject: [PATCH] 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. --- sysklogd/syslogd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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); -- 2.7.4