[asterisk-dev] [Code Review] Handle all possible responses to REFER requests.

Matthew Nicholson mnicholson at digium.com
Wed Aug 4 08:11:42 CDT 2010


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

(Updated 2010-08-04 08:11:42.928213)


Review request for Asterisk Developers.


Changes
-------

Restored unintentionally removed SUBSCRIBE response handling, and tweaked some formatting as suggested by dvossel.


Summary
-------

Currently asterisk does not handle certain responses to refer requests (e.g. 400).  This patch updates chan_sip to handle all possible responses to REFER requests.  Prior to this patch, asterisk can freeze in the Transfer app if asterisk receives an unhanded response to a REFER request.


This addresses bug 17486.
    https://issues.asterisk.org/view.php?id=17486


Diffs (updated)
-----

  /trunk/channels/chan_sip.c 280743 

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


Testing
-------

The reporter tested the patch.


Thanks,

Matthew




More information about the asterisk-dev mailing list