[Home]

Summary:ASTERISK-12466: AST_CONFIG() doesn't handle correctly, if variable isn't found
Reporter:pj (pj)Labels:
Date Opened:2008-07-27 15:48:01Date Closed:2008-07-27 16:04:14
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Functions/func_config
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) func_config.c.patch
Description:simple dialplan
 '900' =>          1. Verbose(${AST_CONFIG(uri.txt,uri,test)})     [pbx_ael]
                   2. Congestion()                               [pbx_ael]


if variable 'test' doesn't exist in uri.txt file in [uri] section, asterisk generates weird ERROR in 'utils.c:585 lock_info_destroy'
generated error (or warning) message is dependent from which channel is called,
cli log below showing call from skinny and sip phone



****** ADDITIONAL INFORMATION ******

[Jul 27 22:50:33] ERROR[30536]: utils.c:585 lock_info_destroy: Thread 'skinny_ss            started at [ 4653] chan_skinny.c handle_stimulus_message()' still has a lock! - '&(&configs)->lock' (0xb744b1c8) from 'config_function_read' in func_config.c:91!
[Jul 27 22:50:33] WARNING[30321]: chan_skinny.c:1582 find_subchannel_by_reference: Could not find any lines that contained a subchannel with reference '79' on device 'PJ'


[Jul 27 22:50:41] ERROR[30537]: utils.c:585 lock_info_destroy: Thread 'pbx_thread           started at [ 3997] pbx.c ast_pbx_start()' still has a lock! - '&(&configs)->lock' (0xb744b1c8) from 'config_function_read' in func_config.c:91!
Comments:By: Digium Subversion (svnbot) 2008-07-27 16:04:08

Repository: asterisk
Revision: 134005

_U  trunk/
U   trunk/funcs/func_config.c

------------------------------------------------------------------------
r134005 | russell | 2008-07-27 16:04:06 -0500 (Sun, 27 Jul 2008) | 4 lines

Add a missing unlock within error handling
(closes issue ASTERISK-12466)
Reported by: pj

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=134005