Update of /usr/cvsroot/asterisk/channels/h323
In directory mongoose.digium.com:/tmp/cvs-serv25682

Modified Files:
        ast_h323.cpp 
Log Message:
Request a sane amount of bandwdith from GK, and fix one-way auido problems with 
broken H.323 stacks *cough* CCM.

Index: ast_h323.cpp
===================================================================
RCS file: /usr/cvsroot/asterisk/channels/h323/ast_h323.cpp,v
retrieving revision 1.79
retrieving revision 1.80
diff -u -d -r1.79 -r1.80
--- ast_h323.cpp        1 Jun 2005 16:00:13 -0000       1.79
+++ ast_h323.cpp        3 Nov 2005 01:41:32 -0000       1.80
@@ -155,6 +155,11 @@
 {
        cout << "  == Creating H.323 Endpoint" << endl;
        endPoint = new MyH323EndPoint();
+       /* Due to a bug in the H.323 recomendation/stack we should request a 
sane
+          amount of bandwidth from the GK - this function is ignored if not 
using a GK 
+          We are requesting 128 (64k in each direction), which is the worst 
case codec. */
+        endPoint->SetInitialBandwidth(1280);
+
        PTrace::Initialise(0, NULL, PTrace::Timestamp | PTrace::Thread | 
PTrace::FileAndLine);
        PTrace::SetStream(logstream);
 }
@@ -360,7 +365,7 @@
        PINDEX as=SupportedPrefixes.GetSize();
        ((H225_VoiceCaps &)protocol).m_supportedPrefixes.SetSize(as);
        for (PINDEX p=0; p<as; p++) {
-               H323SetAliasAddress(SupportedPrefixes[p], ((H225_VoiceCaps 
&)protocol).m_supportedPrefixes[p].m_prefix);
+               H323SetAliasAddress(SupportedPrefixes[p], ((H225_VoiceCaps 
&)protocol).m_supportedPrefixes[p].m_prefix, H225_AliasAddress::e_dialedDigits);
        }
 }
 
@@ -1085,22 +1090,18 @@
                cout << "\t\tRTP channel id " << sessionID << " parameters:" << 
endl;
        }
 
-       /* Update RTP parameters by outgoing voice path only */
-       if (GetDirection() == IsTransmitter) {
-               /* Collect the remote information */
-               H323_ExternalRTPChannel::GetRemoteAddress(remoteIpAddr, 
remotePort);
-
-               if (h323debug) {
-                       cout << "\t\t-- remoteIpAddress: " << remoteIpAddr << 
endl;
-                       cout << "\t\t-- remotePort: " << remotePort << endl;
-                       cout << "\t\t-- ExternalIpAddress: " <<  localIpAddr << 
endl;
-                       cout << "\t\t-- ExternalPort: " << localPort << endl;
-               }
+       /* Collect the remote information */
+       H323_ExternalRTPChannel::GetRemoteAddress(remoteIpAddr, remotePort);
 
-               /* Notify Asterisk of remote RTP information */
-               on_start_rtp_channel(connection.GetCallReference(), (const char 
*)remoteIpAddr.AsString(), remotePort, 
-                       (const char *)connection.GetCallToken(), 
(int)payloadCode);
+       if (h323debug) {
+               cout << "\t\t-- remoteIpAddress: " << remoteIpAddr << endl;
+               cout << "\t\t-- remotePort: " << remotePort << endl;
+               cout << "\t\t-- ExternalIpAddress: " <<  localIpAddr << endl;
+               cout << "\t\t-- ExternalPort: " << localPort << endl;
        }
+       /* Notify Asterisk of remote RTP information */
+       on_start_rtp_channel(connection.GetCallReference(), (const char 
*)remoteIpAddr.AsString(), remotePort, 
+               (const char *)connection.GetCallToken(), (int)payloadCode);
        return TRUE;
 }
 

_______________________________________________
Asterisk-Cvs mailing list
[email protected]
http://lists.digium.com/mailman/listinfo/asterisk-cvs

Reply via email to