]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
logger.c: Fix buffer overrun found by address sanitizer. 67/2067/1
authorRichard Mudgett <rmudgett@digium.com>
Fri, 22 Jan 2016 17:48:24 +0000 (11:48 -0600)
committerRichard Mudgett <rmudgett@digium.com>
Fri, 22 Jan 2016 17:50:41 +0000 (11:50 -0600)
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

index fb9e8ed4fca5db23677d17f48bea31527575caf7..46d9cbb5718b619a4a98ea7f6b6169e27a749508 100644 (file)
@@ -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;
                }