[Asterisk-code-review] chan sip: Support parsing of Q.850 reason header in SIP BYE ... (asterisk[master])

Christof Lauber asteriskteam at digium.com
Tue Nov 17 08:59:50 CST 2015


Hello Matt Jordan,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/1621

to look at the new patch set (#2).

Change subject: chan_sip: Support parsing of Q.850 reason header in SIP BYE and CANCEL requests
......................................................................

chan_sip: Support parsing of Q.850 reason header in SIP BYE and CANCEL requests

Current support for reason header did work only in SIP responses.
According to RFC3336 the reason header might appear in any SIP request.
But it seems to make most sence in BYE and CANCEL so parasing is done
there too. (if use_q850_reason=yes)

* Fixed ISO C90 warning in parse_reason_header

Change-Id: Id24a20310a5edf821a7d6963a2f02146f0d78529
---
M channels/chan_sip.c
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/21/1621/2
-- 
To view, visit https://gerrit.asterisk.org/1621
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id24a20310a5edf821a7d6963a2f02146f0d78529
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Christof Lauber <christof.lauber at annax.ch>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list