From: Matthew Jordan <mjordan@digium.com>
Date: Wed, 4 Jun 2014 14:12:41 +0000 (+0000)
Subject: chan_pjsip: Add debug in RTP Engine glue callback
X-Git-Tag: 12.4.0-rc1~82
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a9533248d3452a641b42055cc8c2ca8944d806fc;p=thirdparty%2Fasterisk.git

chan_pjsip: Add debug in RTP Engine glue callback

This patch adds some debug statements that aid with determining why a direct
media request may or may not be initiated.


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

diff --git a/channels/chan_pjsip.c b/channels/chan_pjsip.c
index 4876f02a4b..26dda4ae2e 100644
--- a/channels/chan_pjsip.c
+++ b/channels/chan_pjsip.c
@@ -303,10 +303,12 @@ static int chan_pjsip_set_rtp_peer(struct ast_channel *chan,
 
 	/* Don't try to do any direct media shenanigans on early bridges */
 	if ((rtp || vrtp || tpeer) && !ast_channel_is_bridged(chan)) {
+		ast_debug(4, "Disregarding setting RTP on %s: channel is not bridged\n", ast_channel_name(chan));
 		return 0;
 	}
 
 	if (nat_active && session->endpoint->media.direct_media.disable_on_nat) {
+		ast_debug(4, "Disregarding setting RTP on %s: NAT is active\n", ast_channel_name(chan));
 		return 0;
 	}
 
@@ -318,6 +320,7 @@ static int chan_pjsip_set_rtp_peer(struct ast_channel *chan,
 	}
 
 	if (direct_media_mitigate_glare(session)) {
+		ast_debug(4, "Disregarding setting RTP on %s: mitigating re-INVITE glare\n", ast_channel_name(chan));
 		return 0;
 	}
 
@@ -329,7 +332,7 @@ static int chan_pjsip_set_rtp_peer(struct ast_channel *chan,
 	if (changed) {
 		ao2_ref(session, +1);
 
-
+		ast_debug(4, "RTP changed on %s; initiating direct media update\n", ast_channel_name(chan));
 		if (ast_sip_push_task(session->serializer, send_direct_media_request, session)) {
 			ao2_cleanup(session);
 		}