From f656c1e62a2a8139ebf64705a8f0e63e6a563fcf Mon Sep 17 00:00:00 2001 From: Richard Mudgett Date: Fri, 22 Jan 2016 11:48:24 -0600 Subject: [PATCH] logger.c: Fix buffer overrun found by address sanitizer. The null terminator of the tail struct member was not being allocated when no logger.conf config file is installed. ASTERISK-25714 #close Reported by: Badalian Vyacheslav Change-Id: I45770fdd08af39506a3bc33ba279c4f16e047a30 --- main/logger.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/main/logger.c b/main/logger.c index 795c85d97b..aeb07325d6 100644 --- a/main/logger.c +++ b/main/logger.c @@ -408,7 +408,7 @@ static int init_logger_chain(int locked, const char *altconf) /* If no config file, we're fine, set default options. */ if (!cfg) { - if (!(chan = ast_calloc(1, sizeof(*chan)))) { + if (!(chan = ast_calloc(1, sizeof(*chan) + 1))) { fprintf(stderr, "Failed to initialize default logging\n"); return -1; } -- 2.47.2