[asterisk-dev] [Code Review] Don't allow a REFER with replaces to try to replace the dialog it is a part of

Terry Wilson reviewboard at asterisk.org
Thu Feb 3 13:27:31 CST 2011

This is an automatically generated e-mail. To reply, visit:

Review request for Asterisk Developers, Olle E Johansson and David Vossel.


Asterisk currently accepts a REFER with a Refer-To with an embedded Replaces header that matches the dialog of the REFER. This would be a situation like A calls B, A calls C, A transfers B to A, which is just silly. This patch makes the transfer fail instead of making Asterisk freak out and forget to hang other channels up.


  /branches/1.4/channels/chan_sip.c 306052 

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


Used sipp to test that the transfer failed properly in the aforementioned case. Used polycom phones to ensure that normal transfers still worked.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110203/15c75526/attachment.htm>

More information about the asterisk-dev mailing list