]> git.ipfire.org Git - thirdparty/chrony.git/commitdiff
logging: make banner printing safer
authorMiroslav Lichvar <mlichvar@redhat.com>
Tue, 22 Oct 2019 13:25:03 +0000 (15:25 +0200)
committerMiroslav Lichvar <mlichvar@redhat.com>
Thu, 24 Oct 2019 09:03:41 +0000 (11:03 +0200)
Don't rely on the buffer filled with '=' characters to be always at
least as long as the log-specific banner string.

logging.c

index e12bad7f1b6bd094f0f2f2ca7f0ee817a2b2a9cf..1993ab724c16f3b7a2a73219c15eec3c10b392ea 100644 (file)
--- a/logging.c
+++ b/logging.c
@@ -289,7 +289,7 @@ LOG_FileWrite(LOG_FileID id, const char *format, ...)
     char bannerline[256];
     int i, bannerlen;
 
-    bannerlen = strlen(logfiles[id].banner);
+    bannerlen = MIN(strlen(logfiles[id].banner), sizeof (bannerline) - 1);
 
     for (i = 0; i < bannerlen; i++)
       bannerline[i] = '=';