fix small interoperability issues if remote endpoint send progress twice

diff --git a/src/mod/endpoints/mod_h323/changes.txt b/src/mod/endpoints/mod_h323/changes.txt
index ff99e50bd6c8648902c78994200a8e4e56e937a0..0076291621187b33b4c103e36d4c811dcfd0b475 100644 (file)
--- a/src/mod/endpoints/mod_h323/changes.txt
+++ b/src/mod/endpoints/mod_h323/changes.txt
@@ -1,3 +1,4 @@
+fix small interoperability issues if remote endpoint send progress twice
 make sure dtmfinband gets initialized
 make gk-identifier and gk-interface settings optional (documentation sayed that about gk-identifier already)
 fix race condition on destroying signaling thread in h323 library

diff --git a/src/mod/endpoints/mod_h323/mod_h323.cpp b/src/mod/endpoints/mod_h323/mod_h323.cpp
index 985f6bb3757115bb849b5a6be842bcb4780f5542..72846b07dc2bea184c3b2cffffa84dfee5fce1d5 100644 (file)
--- a/src/mod/endpoints/mod_h323/mod_h323.cpp
+++ b/src/mod/endpoints/mod_h323/mod_h323.cpp
@@ -1138,8 +1138,7 @@ bool FSH323Connection::OnAlerting(const H323SignalPDU &alertingPDU, const PStrin
 {
        switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG,"======>PFSH323Connection::OnAlerting user = %s [%p]\n",(const char *)user,this);
        unsigned pi;
-       switch_status_t status = switch_channel_mark_ring_ready(m_fsChannel);
-       switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG,"----------->OnAlerting return = %d\n",status);
+       switch_channel_mark_ring_ready(m_fsChannel);
        
        if (!alertingPDU.GetQ931().GetProgressIndicator(pi))
                pi = 0;
@@ -1151,7 +1150,7 @@ bool FSH323Connection::OnAlerting(const H323SignalPDU &alertingPDU, const PStrin
                        m_ChannelProgress = true;
                }
        }
-       return ( status == SWITCH_STATUS_SUCCESS);
+       return H323Connection::OnAlerting(alertingPDU,user);
 }
 
 void FSH323Connection::AnsweringCall(AnswerCallResponse response)