[sipxtapi] r1419 committed - Applied patch from Saab:...

2 views
Skip to first unread message

codesite...@google.com

unread,
Feb 7, 2011, 5:11:28 PM2/7/11
to sipxtapi...@googlegroups.com
Revision: 1419
Author: jaroslavl1
Date: Mon Feb 7 14:10:46 2011
Log: Applied patch from Saab:
- Possible issue in handleResentEvent line 1349 (mTransactionState ==
TRANSACTION_CALLING) || relationship == MESSAGE_CANCEL)): Changed to
(mTransactionState == TRANSACTION_CALLING || relationship ==
MESSAGE_CANCEL)))) - notice brackets. The previous implementation did not
seem to work for CANCEL messages.

http://code.google.com/p/sipxtapi/source/detail?r=1419

Modified:
/trunk/sipXtackLib/src/net/SipTransaction.cpp

=======================================
--- /trunk/sipXtackLib/src/net/SipTransaction.cpp Sun Jan 24 03:43:15 2010
+++ /trunk/sipXtackLib/src/net/SipTransaction.cpp Mon Feb 7 14:10:46 2011
@@ -1346,7 +1346,8 @@
else if(!mIsCanceled &&
((mpLastFinalResponse == NULL &&
mpLastProvisionalResponse == NULL &&
- mTransactionState == TRANSACTION_CALLING) || relationship ==
MESSAGE_CANCEL))
+ (mTransactionState == TRANSACTION_CALLING || relationship ==
MESSAGE_CANCEL))))
+ // This does not seem to work for CANCEL messages, they keep
being resend: mTransactionState == TRANSACTION_CALLING) || relationship ==
MESSAGE_CANCEL))
{
UtlString method;
outgoingMessage.getRequestMethod(&method);

Reply all
Reply to author
Forward
0 new messages