[Asterisk-code-review] res pjsip endpoint identifier ip: Don't output error if no h... (asterisk[13])

Joshua Colp asteriskteam at digium.com
Wed Mar 15 08:46:27 CDT 2017


Joshua Colp has uploaded a new change for review. ( https://gerrit.asterisk.org/5213 )

Change subject: res_pjsip_endpoint_identifier_ip: Don't output error if no header_match.
......................................................................

res_pjsip_endpoint_identifier_ip: Don't output error if no header_match.

This change ensures that if no header_match option is set on an
identify an error message is not output stating the option is set
to an invalid value.

ASTERISK-26863

Change-Id: I239bc6d2319dd3da24ba96a38d4d6e9b5526d62a
---
M res/res_pjsip_endpoint_identifier_ip.c
1 file changed, 6 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/13/5213/1

diff --git a/res/res_pjsip_endpoint_identifier_ip.c b/res/res_pjsip_endpoint_identifier_ip.c
index 1bd7e89..f935882 100644
--- a/res/res_pjsip_endpoint_identifier_ip.c
+++ b/res/res_pjsip_endpoint_identifier_ip.c
@@ -140,9 +140,14 @@
 	pjsip_generic_string_hdr *header;
 	pj_str_t pj_header_name;
 	pj_str_t pj_header_value;
-	char *c_header = ast_strdupa(identify->match_header);
+	char *c_header;
 	char *c_value;
 
+	if (ast_strlen_zero(identify->match_header)) {
+		return 0;
+	}
+
+	c_header = ast_strdupa(identify->match_header);
 	c_value = strchr(c_header, ':');
 	if (!c_value) {
 		ast_log(LOG_WARNING, "Identify '%s' has invalid header_match: No ':' separator found!\n",

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I239bc6d2319dd3da24ba96a38d4d6e9b5526d62a
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list