[svn-commits] file: branch file/gulp_fax r394438 - /team/file/gulp_fax/res/res_sip_session.c

[SVN commits to the Digium repositories]

Author: file
Date: Tue Jul 16 09:25:47 2013
New Revision: 394438

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=394438
Log:
Remove an assumption that during negotiation the remote SDP will contain the same number of media streams.

Fun fact: Offering an audio and T.38 image stream to devices in SDP brings about interesting answers.

Modified:
    team/file/gulp_fax/res/res_sip_session.c

Modified: team/file/gulp_fax/res/res_sip_session.c
URL: http://svnview.digium.com/svn/asterisk/team/file/gulp_fax/res/res_sip_session.c?view=diff&rev=394438&r1=394437&r2=394438
==============================================================================
--- team/file/gulp_fax/res/res_sip_session.c (original)
+++ team/file/gulp_fax/res/res_sip_session.c Tue Jul 16 09:25:47 2013
@@ -411,6 +411,10 @@
 		char media[20];
 		struct ast_sip_session_sdp_handler *handler;
 		RAII_VAR(struct sdp_handler_list *, handler_list, NULL, ao2_cleanup);
+
+		if (!remote->media[i]) {
+			continue;
+		}
 
 		/* We need a null-terminated version of the media string */
 		ast_copy_pj_str(media, &local->media[i]->desc.media, sizeof(media));