< Previous by Date Date Index Next by Date >
  Thread Index Next in Thread >

[reSIProcate] InviteSession::getRemoteSdp() does not always return latest sdp


200 Ok after a re-invite does not update mCurrentRemoteSdp of InviteSession, see patch.

best regards
Björn Andersson


--
This communication is confidential and intended solely for the addressee(s). 
Any unauthorized review, use, disclosure or distribution is prohibited. If you 
believe this message has been sent to you in error, please notify the sender by 
replying to this transmission and delete the message without disclosing it. 
Thank you.
E-mail including attachments is susceptible to data corruption, interruption, 
unauthorized amendment, tampering and viruses, and we only send and receive 
e-mails on the basis that we are not liable for any such corruption, 
interception, amendment, tampering or viruses or any consequences thereof.

--- InviteSession.cxx.orig	2008-06-11 15:04:12.000000000 +0200
+++ InviteSession.cxx	2008-06-11 15:04:41.000000000 +0200
@@ -1449,20 +1449,21 @@
             bool changed = currentRemote.getHex() != newRemote.getHex();
 
             if (changed)
             {
                mCurrentRemoteSdp = sdp; 
                handler->onRemoteSdpChanged(getSessionHandle(), msg, *mCurrentRemoteSdp);
             }
          }
          else
          {
+            mCurrentRemoteSdp = sdp; 
             handler->onAnswer(getSessionHandle(), msg, *sdp);
          }
          
          // !jf! do I need to allow a reINVITE overlapping the retransmission of
          // the ACK when a 200I is received? If yes, then I need to store all
          // ACK messages for 64*T1
          break;
       }
       case On2xx:
          mStaleReInviteTimerSeq++;