From c9f03fb2229783fd242af389d5ee14b658a57028 Mon Sep 17 00:00:00 2001 From: Miroslav Lichvar Date: Wed, 24 Jun 2020 11:27:41 +0200 Subject: [PATCH] logging: handle too many file logs Don't rely on an assert to catch insufficient maximum number of file logs (e.g. after introducing a new file log). --- logging.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/logging.c b/logging.c index d5440a1..57f6945 100644 --- a/logging.c +++ b/logging.c @@ -243,7 +243,10 @@ LOG_CloseParentFd() LOG_FileID LOG_FileOpen(const char *name, const char *banner) { - assert(n_filelogs < MAX_FILELOGS); + if (n_filelogs >= MAX_FILELOGS) { + assert(0); + return -1; + } logfiles[n_filelogs].name = name; logfiles[n_filelogs].banner = banner;