res_pjsip: When no global type the debug option defaults to "yes"
authorKevin Harwell <kharwell@digium.com>
Wed, 5 Feb 2014 19:42:51 +0000 (19:42 +0000)
committerKevin Harwell <kharwell@digium.com>
Wed, 5 Feb 2014 19:42:51 +0000 (19:42 +0000)
If the global section was not specified in pjsip.conf then the configuration
object does not exist in sorcery so when retrieving "debug" option it would
return NULL.  Then the NULL result was passed to ast_false utils function
which would return false because it wasn't set to some representation of
false, thus enabling sip debug logging.  Made it so if the global config object
does not exist then it will return a default of "no" for sip debugging.

(issue ASTERISK-23038)
Reported by: Rusty Newton
........

Merged revisions 407442 from http://svn.asterisk.org/svn/asterisk/branches/12

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@407443 65c4cc65-6c06-0410-ace0-fbb531ad65f3

res/res_pjsip/config_global.c

index 0f4350d..739a233 100644 (file)
@@ -105,7 +105,7 @@ char *ast_sip_get_debug(void)
        struct global_config *cfg = get_global_cfg();
 
        if (!cfg) {
-               return 0;
+               return ast_strdup("no");
        }
 
        res = ast_strdup(cfg->debug);