[asterisk-commits] twilson: branch 1.6.2 r323733 - in /branches/1.6.2: ./ main/features.c

[SVN commits to the Asterisk project]

Author: twilson
Date: Wed Jun 15 13:13:00 2011
New Revision: 323733

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=323733
Log:
Merged revisions 323732 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r323732 | twilson | 2011-06-15 13:06:24 -0500 (Wed, 15 Jun 2011) | 9 lines
  
  Fix DYNAMIC_FEATURES
  
  DYNAMIC_FEATURES were broken by a recent DTMF change. This patch makes
  sure that dynamic features are also checked when deciding whether or not
  to pass DTMF through or store it for interpreting.
  
  (closes issue ASTERISK-17914)
  Reported by: vrban
........

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

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

Modified: branches/1.6.2/main/features.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/main/features.c?view=diff&rev=323733&r1=323732&r2=323733
==============================================================================
--- branches/1.6.2/main/features.c (original)
+++ branches/1.6.2/main/features.c Wed Jun 15 13:13:00 2011
@@ -2381,8 +2381,12 @@
 
 /*! \brief Check if a feature exists */
 static int feature_check(struct ast_channel *chan, struct ast_flags *features, char *code) {
-
-	return feature_interpret_helper(chan, NULL, NULL, code, 0, NULL, features, FEATURE_INTERPRET_CHECK, NULL);
+	char *chan_dynamic_features;
+	ast_channel_lock(chan);
+	chan_dynamic_features = ast_strdupa(S_OR(pbx_builtin_getvar_helper(chan, "DYNAMIC_FEATURES"),""));
+	ast_channel_unlock(chan);
+
+	return feature_interpret_helper(chan, NULL, NULL, code, 0, chan_dynamic_features, features, FEATURE_INTERPRET_CHECK, NULL);
 }
 
 static void set_config_flags(struct ast_channel *chan, struct ast_channel *peer, struct ast_bridge_config *config)