From 24ef2a922e9ccc0bee477bcd383d840ad48d5f41 Mon Sep 17 00:00:00 2001 From: "Dwayne M. Hubbard" Date: Tue, 17 Feb 2009 22:09:16 +0000 Subject: [PATCH] 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. ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@176709 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- channels/chan_sip.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 1d571f3a48..937e7024ec 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -4039,6 +4039,10 @@ static int create_addr_from_peer(struct sip_pvt *dialog, struct sip_peer *peer) } 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;