[asterisk-commits] kharwell: trunk r407443 - in /trunk: ./ res/res_pjsip/config_global.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Feb 5 13:42:53 CST 2014


Author: kharwell
Date: Wed Feb  5 13:42:51 2014
New Revision: 407443

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=407443
Log:
res_pjsip: When no global type the debug option defaults to "yes"

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

Modified:
    trunk/   (props changed)
    trunk/res/res_pjsip/config_global.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-12-merged' - no diff available.

Modified: trunk/res/res_pjsip/config_global.c
URL: http://svnview.digium.com/svn/asterisk/trunk/res/res_pjsip/config_global.c?view=diff&rev=407443&r1=407442&r2=407443
==============================================================================
--- trunk/res/res_pjsip/config_global.c (original)
+++ trunk/res/res_pjsip/config_global.c Wed Feb  5 13:42:51 2014
@@ -105,7 +105,7 @@
 	struct global_config *cfg = get_global_cfg();
 
 	if (!cfg) {
-		return 0;
+		return ast_strdup("no");
 	}
 
 	res = ast_strdup(cfg->debug);




More information about the asterisk-commits mailing list