[asterisk-commits] file: trunk r44487 - in /trunk: ./ channels/chan_sip.c

[asterisk-commits at lists.digium.com]

Author: file
Date: Thu Oct  5 13:03:34 2006
New Revision: 44487

URL: http://svn.digium.com/view/asterisk?rev=44487&view=rev
Log:
Merged revisions 44486 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r44486 | file | 2006-10-05 14:01:51 -0400 (Thu, 05 Oct 2006) | 2 lines

One more T.38 fix! Don't leave a reinvite hanging by a thread if the other side is already setup with T.38

........

Modified:
    trunk/   (props changed)
    trunk/channels/chan_sip.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_sip.c?rev=44487&r1=44486&r2=44487&view=diff
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Thu Oct  5 13:03:34 2006
@@ -13133,6 +13133,12 @@
 									transmit_response_reliable(p, "488 Not acceptable here", req);
 								sip_scheddestroy(p, DEFAULT_TRANS_TIMEOUT);
 							}
+						} else {
+							/* The other side is already setup for T.38 most likely so we need to acknowledge this too */
+							transmit_response_with_t38_sdp(p, "200 OK", req, XMIT_CRITICAL);
+							p->t38.state = T38_ENABLED;
+							if (option_debug)
+								ast_log(LOG_DEBUG, "T38 state changed to %d on channel %s\n", p->t38.state, p->owner ? p->owner->name : "<none>");
 						}
 					} else {
 						/* Other side is not a SIP channel */