[asterisk-commits] branch oej/sdpcleanup r32568 - /team/oej/sdpcleanup/channels/chan_sip.c

[asterisk-commits at lists.digium.com]

Author: oej
Date: Tue Jun  6 10:52:30 2006
New Revision: 32568

URL: http://svn.digium.com/view/asterisk?rev=32568&view=rev
Log:
Don't build SDP media string before we actually know the video port number...  Thanks, John.

Modified:
    team/oej/sdpcleanup/channels/chan_sip.c

Modified: team/oej/sdpcleanup/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/team/oej/sdpcleanup/channels/chan_sip.c?rev=32568&r1=32567&r2=32568&view=diff
==============================================================================
--- team/oej/sdpcleanup/channels/chan_sip.c (original)
+++ team/oej/sdpcleanup/channels/chan_sip.c Tue Jun  6 10:52:30 2006
@@ -5157,7 +5157,6 @@
 	/* Ok, we need video. Let's add what we need for video and set codecs.
 	   Video is handled differently than audio since we can not transcode. */
 	if (needvideo) {
-		ast_build_string(&m_video_next, &m_video_left, "m=video %d RTP/AVP", ntohs(vdest.sin_port));
 
 		/* Determine video destination */
 		if (p->vredirip.sin_addr.s_addr) {
@@ -5167,6 +5166,7 @@
 			vdest.sin_addr = p->ourip;
 			vdest.sin_port = vsin.sin_port;
 		}
+		ast_build_string(&m_video_next, &m_video_left, "m=video %d RTP/AVP", ntohs(vdest.sin_port));
 
 		/* Build max bitrate string */
 		if (p->maxcallbitrate)