[asterisk-commits] russell: branch 1.4 r46358 - /branches/1.4/channels/chan_zap.c

[asterisk-commits at lists.digium.com]

Author: russell
Date: Fri Oct 27 10:32:40 2006
New Revision: 46358

URL: http://svn.digium.com/view/asterisk?rev=46358&view=rev
Log:
Instead of iterating all of the options once to look for jitterbuffer options,
and then again for everything else, move the processing of jitterbuffer
options into the main loop so that there are no erroneous messages about
ignoring unknown options.  (issue #8226)

Modified:
    branches/1.4/channels/chan_zap.c

Modified: branches/1.4/channels/chan_zap.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/channels/chan_zap.c?rev=46358&r1=46357&r2=46358&view=diff
==============================================================================
--- branches/1.4/channels/chan_zap.c (original)
+++ branches/1.4/channels/chan_zap.c Fri Oct 27 10:32:40 2006
@@ -10419,7 +10419,13 @@
 	int y;
 	int found_pseudo = 0;
 
+	/* Copy the default jb config over global_jbconf */
+	memcpy(&global_jbconf, &default_jbconf, sizeof(struct ast_jb_conf));
+
 	while(v) {
+		if (!ast_jb_read_conf(&global_jbconf, v->name, v->value))
+			continue;
+
 		/* Create the interface list */
 		if (!strcasecmp(v->name, "channel")
 #ifdef HAVE_PRI
@@ -11064,7 +11070,6 @@
 {
 	struct ast_config *cfg;
 	struct ast_variable *v;
-	struct ast_variable *vjb;
 	int res;
 
 #ifdef HAVE_PRI
@@ -11147,11 +11152,6 @@
 	}
 #endif
 	v = ast_variable_browse(cfg, "channels");
-	/* Copy the default jb config over global_jbconf */
-	memcpy(&global_jbconf, &default_jbconf, sizeof(struct ast_jb_conf));
-	/* Traverse all variables to handle jb conf */
-	for (vjb = v; vjb; vjb = vjb->next)
-		ast_jb_read_conf(&global_jbconf, vjb->name, vjb->value);
 	res = process_zap(v, reload, 0);
 	ast_mutex_unlock(&iflock);
 	ast_config_destroy(cfg);