[asterisk-commits] jrose: branch 12 r403587 - /branches/12/channels/chan_pjsip.c

[SVN commits to the Asterisk project]

Author: jrose
Date: Mon Dec  9 16:47:06 2013
New Revision: 403587

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=403587
Log:
chan_pjsip: Fix a sticking channel lock caused by channel masquerades

(closes issue ASTERISK-22936)
Reported by: Jonathan Rose
Review: https://reviewboard.asterisk.org/r/3042/

Modified:
    branches/12/channels/chan_pjsip.c

Modified: branches/12/channels/chan_pjsip.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/channels/chan_pjsip.c?view=diff&rev=403587&r1=403586&r2=403587
==============================================================================
--- branches/12/channels/chan_pjsip.c (original)
+++ branches/12/channels/chan_pjsip.c Mon Dec  9 16:47:06 2013
@@ -841,7 +841,16 @@
 struct fixup_data {
 	struct ast_sip_session *session;
 	struct ast_channel *chan;
+	struct ast_channel *oldchan;
 };
+
+static void fixup_data_destroy(struct fixup_data *fix_data)
+{
+	ao2_cleanup(fix_data->session);
+	ast_channel_cleanup(fix_data->chan);
+	ast_channel_cleanup(fix_data->oldchan);
+	ast_free(fix_data);
+}
 
 static int fixup(void *data)
 {
@@ -849,6 +858,11 @@
 	struct ast_sip_channel_pvt *channel = ast_channel_tech_pvt(fix_data->chan);
 	struct chan_pjsip_pvt *pvt = channel->pvt;
 
+	if (channel->session->channel != fix_data->oldchan) {
+		fixup_data_destroy(fix_data);
+		return -1;
+	}
+
 	channel->session->channel = fix_data->chan;
 	if (pvt->media[SIP_MEDIA_AUDIO] && pvt->media[SIP_MEDIA_AUDIO]->rtp) {
 		ast_rtp_instance_set_channel_id(pvt->media[SIP_MEDIA_AUDIO]->rtp, ast_channel_uniqueid(fix_data->chan));
@@ -857,6 +871,8 @@
 		ast_rtp_instance_set_channel_id(pvt->media[SIP_MEDIA_VIDEO]->rtp, ast_channel_uniqueid(fix_data->chan));
 	}
 
+	fixup_data_destroy(fix_data);
+
 	return 0;
 }
 
@@ -864,16 +880,23 @@
 static int chan_pjsip_fixup(struct ast_channel *oldchan, struct ast_channel *newchan)
 {
 	struct ast_sip_channel_pvt *channel = ast_channel_tech_pvt(newchan);
-	struct fixup_data fix_data;
-
-	fix_data.session = channel->session;
-	fix_data.chan = newchan;
-
-	if (channel->session->channel != oldchan) {
+	struct fixup_data *fix_data = ast_calloc(1, sizeof(*fix_data));
+
+	if (!fix_data) {
 		return -1;
 	}
 
-	if (ast_sip_push_task_synchronous(channel->session->serializer, fixup, &fix_data)) {
+	fix_data->session = channel->session;
+	ao2_ref(fix_data->session, +1);
+
+	fix_data->chan = newchan;
+	ast_channel_ref(fix_data->chan);
+
+	fix_data->oldchan = oldchan;
+	ast_channel_ref(fix_data->oldchan);
+
+	if (ast_sip_push_task(channel->session->serializer, fixup, fix_data)) {
+		fixup_data_destroy(fix_data);
 		ast_log(LOG_WARNING, "Unable to perform channel fixup\n");
 		return -1;
 	}