From: Richard Mudgett <rmudgett@digium.com>
Date: Fri, 22 Jan 2016 17:48:24 +0000 (-0600)
Subject: logger.c: Fix buffer overrun found by address sanitizer.
X-Git-Tag: 13.8.0-rc1~112
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1c95b211a0416c0d9ee0a732fe5615a356e7c12f;p=thirdparty%2Fasterisk.git

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
---

diff --git a/main/logger.c b/main/logger.c
index fb9e8ed4fc..46d9cbb571 100644
--- a/main/logger.c
+++ b/main/logger.c
@@ -464,7 +464,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;
 		}