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

[Christof Lauber]

Christof Lauber has uploaded a new change for review.

  https://gerrit.asterisk.org/1621

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/1

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 52d26f5..3e4b97f 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -16125,7 +16125,8 @@
 {
 	struct ast_channel *owner;
 	owner = pvt->owner;
-	int ret=FALSE;
+	int ret;
+	ret=FALSE;
 	if (owner) {
 		const char *rp = NULL, *rh = NULL;
 		ast_channel_hangupcause_set(owner, 0);

-- 
To view, visit https://gerrit.asterisk.org/1621
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id24a20310a5edf821a7d6963a2f02146f0d78529
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Christof Lauber <christof.lauber at annax.ch>