[asterisk-commits] mvanbaak: branch 1.6.2 r215647 - in /branches/1.6.2: ./ main/features.c

[SVN commits to the Asterisk project]

Author: mvanbaak
Date: Wed Sep  2 15:55:02 2009
New Revision: 215647

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=215647
Log:
Merged revisions 215622 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r215622 | mvanbaak | 2009-09-02 22:21:51 +0200 (Wed, 02 Sep 2009) | 4 lines
  
  - lock channel before looking for a channel variable
  - Init the parkings list member of struct parkinglot.
  Thanks Sean for the explanation why this should be here.
........

Modified:
    branches/1.6.2/   (props changed)
    branches/1.6.2/main/features.c

Propchange: branches/1.6.2/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.2/main/features.c
URL: http://svn.asterisk.org/svn-view/asterisk/branches/1.6.2/main/features.c?view=diff&rev=215647&r1=215646&r2=215647
==============================================================================
--- branches/1.6.2/main/features.c (original)
+++ branches/1.6.2/main/features.c Wed Sep  2 15:55:02 2009
@@ -593,7 +593,9 @@
 	/* Lock parking list */
 	AST_LIST_LOCK(&parkinglot->parkings);
 	/* Check for channel variable PARKINGEXTEN */
+	ast_channel_lock(chan);
 	parkingexten = pbx_builtin_getvar_helper(chan, "PARKINGEXTEN");
+	ast_channel_unlock(chan);
 	if (!ast_strlen_zero(parkingexten)) {
 		/*!\note The API forces us to specify a numeric parking slot, even
 		 * though the architecture would tend to support non-numeric extensions
@@ -3506,6 +3508,7 @@
 		return NULL;
 	
 	ast_copy_string(newlot->name, name, sizeof(newlot->name));
+	AST_LIST_HEAD_INIT(&newlot->parkings);
 
 	return newlot;
 }