[sr-dev] git:master:0f479b53: sanity: adjusted log messages for check_proxy_require()

[Daniel-Constantin Mierla]

Module: kamailio
Branch: master
Commit: 0f479b5366fae4c5ad0c78e4c91661b7a421d5e8
URL: https://github.com/kamailio/kamailio/commit/0f479b5366fae4c5ad0c78e4c91661b7a421d5e8

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date: 2015-08-24T09:28:29+02:00

sanity: adjusted log messages for check_proxy_require()

---

Modified: modules/sanity/sanity.c

---

Diff:  https://github.com/kamailio/kamailio/commit/0f479b5366fae4c5ad0c78e4c91661b7a421d5e8.diff
Patch: https://github.com/kamailio/kamailio/commit/0f479b5366fae4c5ad0c78e4c91661b7a421d5e8.patch

---

diff --git a/modules/sanity/sanity.c b/modules/sanity/sanity.c
index 0e2cb45..8140c07 100644
--- a/modules/sanity/sanity.c
+++ b/modules/sanity/sanity.c
@@ -652,19 +652,18 @@ int check_proxy_require(struct sip_msg* _msg) {
 	int u_len;
 
 #ifdef EXTRA_DEBUG
-	DBG("check_proxy_require entered\n");
+	LM_DBG("checking proxy require\n");
 #endif
 
 	if (parse_headers(_msg, HDR_PROXYREQUIRE_F, 0) != 0) {
-		LOG(L_WARN, "sanity_check(): check_proxy_require():"
-				" failed to parse proxy require header\n");
+		LM_WARN("failed to parse proxy require header\n");
 		return SANITY_CHECK_FAILED;
 	}
 	if (_msg->proxy_require != NULL) {
 		//dump_hdr_field(_msg->proxy_require);
 		if (_msg->proxy_require->parsed == NULL &&
 				parse_proxyrequire(_msg->proxy_require) < 0) {
-			LOG(L_WARN, "sanity_check(): check_proxy_require(): parse_proxy_require failed\n");
+			LM_WARN("parse_proxy_require failed\n");
 			return SANITY_CHECK_FAILED;
 		}
 		r_pr = _msg->proxy_require->parsed;
@@ -672,7 +671,7 @@ int check_proxy_require(struct sip_msg* _msg) {
 			l_pr = proxyrequire_list;
 			while (l_pr != NULL) {
 #ifdef EXTRA_DEBUG
-				DBG("check_proxy_require(): comparing r='%.*s' l='%.*s'\n",
+				LM_DBG("comparing r='%.*s' l='%.*s'\n",
 						r_pr->string.len, r_pr->string.s, l_pr->string.len,
 						l_pr->string.s);
 #endif
@@ -685,14 +684,12 @@ int check_proxy_require(struct sip_msg* _msg) {
 				l_pr = l_pr->next;
 			}
 			if (l_pr == NULL) {
-				DBG("sanit_check(): check_proxy_require():"
-						" request contains unsupported extension: %.*s\n",
+				LM_DBG("request contains unsupported extension: %.*s\n",
 						r_pr->string.len, r_pr->string.s);
 				u_len = UNSUPPORTED_HEADER_LEN + 2 + r_pr->string.len;
 				u = pkg_malloc(u_len);
 				if (u == NULL) {
-					LOG(L_ERR, "sanity_check(): check_proxy_require():"
-							" failed to allocate memory for"
+					LM_ERR("failed to allocate memory for"
 							" Unsupported header\n");
 				}
 				else {
@@ -706,12 +703,11 @@ int check_proxy_require(struct sip_msg* _msg) {
 
 				if (_msg->REQ_METHOD != METHOD_ACK) {
 					if (sanity_reply(_msg, 420, "Bad Extension") < 0) {
-						LOG(L_WARN, "sanity_check(): check_proxy_require():"
-								" failed to send 420 via sl reply\n");
+						LM_WARN("failed to send 420 via sl reply\n");
 					}
 				}
 #ifdef EXTRA_DEBUG
-				DBG("check_proxy_require failed\n");
+				LM_DBG("checking proxy require failed\n");
 #endif
 				if (u) pkg_free(u);
 				return SANITY_CHECK_FAILED;
@@ -721,7 +717,7 @@ int check_proxy_require(struct sip_msg* _msg) {
 			}
 		}
 #ifdef EXTRA_DEBUG
-		DBG("check_proxy_require passed\n");
+		LM_DBG("checking proxy require passed\n");
 #endif
 		if (_msg->proxy_require->parsed) {
 			/* TODO we have to free it here, because it is not automatically
@@ -732,7 +728,7 @@ int check_proxy_require(struct sip_msg* _msg) {
 	}
 #ifdef EXTRA_DEBUG
 	else {
-		DBG("check_proxy_require(): no proxy-require header found\n");
+		LM_DBG("no proxy-require header found\n");
 	}
 #endif