[asterisk-dev] [Code Review] SIP: peer matching TCP/TLS 1.6.0

David Vossel dvossel at digium.com
Fri Sep 4 17:05:07 CDT 2009


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

Review request for Asterisk Developers.


Summary
-------

Using both the ip address and port number to peer match a connection coming in over tcp does not work.  A work around for this issue was committed a few months ago which used the contact header to peer match tcp connections. This is incorrect.  Instead, peer matching for tcp connections should be done by ip address only.

This patch removes the contact header matching logic and adds logic to match all tcp/tls connections by ip only.


Diffs
-----

  /branches/1.6.0/channels/chan_sip.c 216600 

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


Testing
-------

made calls via tcp and udp, verified peer matching worked as expected.


Thanks,

David




More information about the asterisk-dev mailing list