[asterisk-commits] dhubbard: branch 1.6.1 r176731 - in /branches/1.6.1: ./ channels/chan_sip.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Feb 17 16:22:00 CST 2009


Author: dhubbard
Date: Tue Feb 17 16:21:59 2009
New Revision: 176731

URL: http://svn.digium.com/svn-view/asterisk?view=rev&rev=176731
Log:
Merged revisions 176705 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r176705 | dhubbard | 2009-02-17 15:59:38 -0600 (Tue, 17 Feb 2009) | 11 lines
  
  create a UDPTL structure in create_addr_from_peer() if it does not already exist for T38
  
  This is required to create a UDPTL structure in create_addr_from_peer() to handle the
  scenario where 't38pt_udptl=yes' is not defined in the [general] section of sip.conf but 
  is defined the peer's context.  I tested this patch by enabling t38pt_udptl in the 
  [general] section on one system and only enabling t38pt_udptl in a peer's context on
  the system sending a fax.  Without the patch, the sending system will fail to initiate
  T38 negotiation with the warning message, "No way to add SDP without an UDPTL structure".
  When this patch is applied the sending side will successfully initiate T38 negotiation.
........

Modified:
    branches/1.6.1/   (props changed)
    branches/1.6.1/channels/chan_sip.c

Propchange: branches/1.6.1/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.1/channels/chan_sip.c
URL: http://svn.digium.com/svn-view/asterisk/branches/1.6.1/channels/chan_sip.c?view=diff&rev=176731&r1=176730&r2=176731
==============================================================================
--- branches/1.6.1/channels/chan_sip.c (original)
+++ branches/1.6.1/channels/chan_sip.c Tue Feb 17 16:21:59 2009
@@ -4304,6 +4304,10 @@
 	}
 	dialog->prefs = peer->prefs;
 	if (ast_test_flag(&dialog->flags[1], SIP_PAGE2_T38SUPPORT)) {
+		if (!dialog->udptl) {
+			/* t38pt_udptl was enabled in the peer and not in [general] */
+			dialog->udptl = ast_udptl_new_with_bindaddr(sched, io, 0, bindaddr.sin_addr);
+		}
 		ast_copy_flags(&dialog->t38.t38support, &peer->flags[1], SIP_PAGE2_T38SUPPORT);
 		set_t38_capabilities(dialog);
 		dialog->t38.jointcapability = dialog->t38.capability;




More information about the asterisk-commits mailing list