[asterisk-commits] mjordan: trunk r415118 - in /trunk: ./ channels/chan_pjsip.c

[SVN commits to the Asterisk project]

Author: mjordan
Date: Wed Jun  4 09:13:07 2014
New Revision: 415118

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=415118
Log:
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.
........

Merged revisions 415117 from http://svn.asterisk.org/svn/asterisk/branches/12

Modified:
    trunk/   (props changed)
    trunk/channels/chan_pjsip.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-12-merged' - no diff available.

Modified: trunk/channels/chan_pjsip.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_pjsip.c?view=diff&rev=415118&r1=415117&r2=415118
==============================================================================
--- trunk/channels/chan_pjsip.c (original)
+++ trunk/channels/chan_pjsip.c Wed Jun  4 09:13:07 2014
@@ -303,10 +303,12 @@
 
 	/* 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 @@
 	}
 
 	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 @@
 	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);
 		}