-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4146/
-----------------------------------------------------------

Review request for Asterisk Developers.


Bugs: AST-1459
    https://issues.asterisk.org/jira/browse/AST-1459


Repository: Asterisk


Description
-------

Currently when distributing a message the code finds the dialog related to it 
and if present uses the information to send the message to the right serializer 
and to place the endpoint on the message. For the endpoint its presence is 
checked in the endpoint lookup step and placed into the expected endpoint spot. 
This causes problems as the dialog handling will occur before the endpoint 
lookup step, causing the endpoint to get lost and never placed on the message.

This change removes the endpoint lookup step requirement and places the 
endpoint in the expected spot in the distribution step. If the endpoint lookup 
step is actually invoked it just immediately returns if an endpoint is already 
present and does nothing.


Diffs
-----

  /branches/12/res/res_pjsip/pjsip_distributor.c 427112 

Diff: https://reviewboard.asterisk.org/r/4146/diff/


Testing
-------

Added some code to determine if endpoint is present in the NAT handling code. 
Without the patch no endpoint would be present on the 200 OK response to an 
outgoing re-INVITE. With the patch an endpoint is present on the 200 OK 
response.


Thanks,

Joshua Colp

-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Reply via email to