From 391112d89ac6e0df012beff3b6e59c158e898c42 Mon Sep 17 00:00:00 2001 From: Chris-Savinovich Date: Thu, 4 Apr 2019 16:02:42 -0500 Subject: [PATCH] config.c: Fix a crash in extconfig parsing When extconfig.conf file is parsed, the code previously searched for character comma without verifying if error (null or blank). This caused a segmentation error. Change-Id: Id76b452d8f330d11c2742c37232761ad71472a8b --- main/config.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/main/config.c b/main/config.c index f694c2bd8c..5660fbe963 100644 --- a/main/config.c +++ b/main/config.c @@ -2904,7 +2904,10 @@ static int reload_module(void) ast_copy_string(buf, v->value, sizeof(buf)); stringp = buf; driver = strsep(&stringp, ","); - + if (!stringp) { + ast_log(LOG_WARNING, "extconfig.conf: value '%s' ignored due to wrong format\n", v->value); + continue; + } if ((tmp = strchr(stringp, '\"'))) stringp = tmp;