retronetworking
/
yate
12
0
Fork 2
Code Issues Pull Requests Projects Releases Wiki Activity

Consistently use the branch from the last Via header in matching transactions. 

Fixes bug #237.


git-svn-id: http://yate.null.ro/svn/yate/trunk@4080 acf43c95-373e-0410-b603-e72c3f656dc1
This commit is contained in:
paulc 2011-01-28 09:49:55 +00:00
parent 313d1bb82a
commit 3400667350
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
libs/ysip/transaction.cpp
View File

@ -39,7 +39,7 @@ SIPTransaction::SIPTransaction(SIPMessage* message, SIPEngine* engine, bool outg
if (m_firstMessage) {
m_firstMessage->ref();
const NamedString* ns = message->getParam("Via","branch");
const NamedString* ns = message->getParam("Via","branch",true);
if (ns)
m_branch = *ns;
if (!m_branch.startsWith("z9hG4bK"))
@ -87,7 +87,7 @@ SIPTransaction::SIPTransaction(SIPTransaction& original, SIPMessage* answer)
m_firstMessage->setAutoAuth();
msg->complete(m_engine);
msg->addHeader(auth);
const NamedString* ns = msg->getParam("Via","branch");
const NamedString* ns = msg->getParam("Via","branch",true);
if (ns)
original.m_branch = *ns;
else
@ -439,7 +439,7 @@ SIPTransaction::Processed SIPTransaction::processMessage(SIPMessage* message, co
return NoMatch;
// allow braindamaged UAs that send answers with no Via line
if (m_firstMessage->getHeader("Via") && message->getHeader("Via") &&
(m_firstMessage->getHeaderValue("Via") != message->getHeaderValue("Via")))
(m_firstMessage->getHeaderValue("Via",true) != message->getHeaderValue("Via",true)))
return NoMatch;
// extra checks are to be made for ACK only
if (message->isACK()) {