From: Mark Michelson <mmichelson@digium.com>
Date: Wed, 20 Aug 2014 20:02:56 +0000 (+0000)
Subject: Move evaluation of set_var options in pjsip to the end of channel initialization.
X-Git-Tag: 12.6.0-rc1~68
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ac1ff37254f6ee3cafd8371b12c7167a8bb5d478;p=thirdparty%2Fasterisk.git

Move evaluation of set_var options in pjsip to the end of channel initialization.

This allows for set_var to override certain defaults such as caller ID and codec
values. This also fixes a test suite regression. The "set_var" test suite test attempted
to use set_var to override caller ID, but a recent change caused that to no longer work.



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/12@421565 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/channels/chan_pjsip.c b/channels/chan_pjsip.c
index 1640ca0c79..c8a0e0bdca 100644
--- a/channels/chan_pjsip.c
+++ b/channels/chan_pjsip.c
@@ -393,12 +393,6 @@ static struct ast_channel *chan_pjsip_new(struct ast_sip_session *session, int s
 		return NULL;
 	}
 
-	for (var = session->endpoint->channel_vars; var; var = var->next) {
-		char buf[512];
-		pbx_builtin_setvar_helper(chan, var->name, ast_get_encoded_str(
-						  var->value, buf, sizeof(buf)));
-	}
-
 	ast_channel_stage_snapshot(chan);
 
 	ast_channel_tech_pvt_set(chan, channel);
@@ -446,6 +440,12 @@ static struct ast_channel *chan_pjsip_new(struct ast_sip_session *session, int s
 		ast_channel_zone_set(chan, zone);
 	}
 
+	for (var = session->endpoint->channel_vars; var; var = var->next) {
+		char buf[512];
+		pbx_builtin_setvar_helper(chan, var->name, ast_get_encoded_str(
+						  var->value, buf, sizeof(buf)));
+	}
+
 	ast_channel_stage_snapshot_done(chan);
 	ast_channel_unlock(chan);