Merge "res_pjsip_rfc3326.c: Account for more than one 'Reason' header" into 13

certified/13.21
Joshua Colp 7 years ago committed by Gerrit Code Review
commit c78f4b8e61

@ -36,20 +36,21 @@
static void rfc3326_use_reason_header(struct ast_sip_session *session, struct pjsip_rx_data *rdata) static void rfc3326_use_reason_header(struct ast_sip_session *session, struct pjsip_rx_data *rdata)
{ {
const pj_str_t str_reason = { "Reason", 6 }; static const pj_str_t str_reason = { "Reason", 6 };
pjsip_generic_string_hdr *header = pjsip_msg_find_hdr_by_name(rdata->msg_info.msg, &str_reason, NULL); pjsip_generic_string_hdr *header;
char buf[20], *cause, *text; char buf[20];
char *cause;
char *text;
int code; int code;
if (!header) { header = pjsip_msg_find_hdr_by_name(rdata->msg_info.msg, &str_reason, NULL);
return; for (; header;
} header = pjsip_msg_find_hdr_by_name(rdata->msg_info.msg, &str_reason, header->next)) {
ast_copy_pj_str(buf, &header->hvalue, sizeof(buf)); ast_copy_pj_str(buf, &header->hvalue, sizeof(buf));
cause = ast_skip_blanks(buf); cause = ast_skip_blanks(buf);
if (strncasecmp(cause, "Q.850", 5) || !(cause = strstr(cause, "cause="))) { if (strncasecmp(cause, "Q.850", 5) || !(cause = strstr(cause, "cause="))) {
return; continue;
} }
/* If text is present get rid of it */ /* If text is present get rid of it */
@ -58,10 +59,12 @@ static void rfc3326_use_reason_header(struct ast_sip_session *session, struct pj
} }
if (sscanf(cause, "cause=%30d", &code) != 1) { if (sscanf(cause, "cause=%30d", &code) != 1) {
return; continue;
} }
ast_channel_hangupcause_set(session->channel, code & 0x7f); ast_channel_hangupcause_set(session->channel, code & 0x7f);
break;
}
} }
static int rfc3326_incoming_request(struct ast_sip_session *session, struct pjsip_rx_data *rdata) static int rfc3326_incoming_request(struct ast_sip_session *session, struct pjsip_rx_data *rdata)

Loading…
Cancel
Save