[asterisk-commits] oej: branch oej/pinefrog-rtcp-1.8 r382722 - /team/oej/pinefrog-rtcp-1.8/chann...

[SVN commits to the Asterisk project]

Author: oej
Date: Fri Mar  8 10:26:26 2013
New Revision: 382722

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=382722
Log:
<room for a comment, contact management for sponsorships and banner ads in commit messages>

Modified:
    team/oej/pinefrog-rtcp-1.8/channels/sip/rtcp.c

Modified: team/oej/pinefrog-rtcp-1.8/channels/sip/rtcp.c
URL: http://svnview.digium.com/svn/asterisk/team/oej/pinefrog-rtcp-1.8/channels/sip/rtcp.c?view=diff&rev=382722&r1=382721&r2=382722
==============================================================================
--- team/oej/pinefrog-rtcp-1.8/channels/sip/rtcp.c (original)
+++ team/oej/pinefrog-rtcp-1.8/channels/sip/rtcp.c Fri Mar  8 10:26:26 2013
@@ -91,9 +91,12 @@
 			}
 		}
 
+	} else {
+ 		ast_debug(1, "######## Not setting rtcp media data. Dialog %s Dialog owner %s \n", dialog ? "set" : "unset",  dialog->owner ? "set" : "unset");
 	}
 
 	if (ast_rtp_instance_get_stats(instance, &qual, AST_RTP_INSTANCE_STAT_ALL)) {
+ 		ast_debug(1, "######## Did not get any statistics... bad, bad, RTP instance\n");
 		/* Houston, we got a problem */
 		return;
 	}
@@ -300,10 +303,13 @@
 	struct sip_pvt *dialog = (struct sip_pvt *) data;
 	ast_log(LOG_DEBUG, "***** SENDING RTCP EVENT \n");
 
-	if (dialog->rtp && ast_rtp_instance_isactive(dialog->rtp)) {
+	if (dialog->rtp && !ast_rtp_instance_isactive(dialog->rtp)) {
+		ast_log(LOG_DEBUG, "          ***** Activating RTCP report \n");
 		sip_rtcp_report(dialog, dialog->rtp, SDP_AUDIO, FALSE);
-	}
-	if (dialog->vrtp && ast_rtp_instance_isactive(dialog->vrtp)) {
+	} else {
+		ast_log(LOG_DEBUG, "          ***** NOT Activating RTCP report \n");
+	}
+	if (dialog->vrtp && !ast_rtp_instance_isactive(dialog->vrtp)) {
 		sip_rtcp_report(dialog, dialog->vrtp, SDP_VIDEO, FALSE);
 	}
 	return (dialog->sip_cfg ? dialog->sip_cfg->rtcptimer : 0);