Browse code

tm: some more cleanups regarding logging (remove double log level or function)

Henning Westerholt authored on 01/12/2020 08:28:51
Showing 4 changed files
... ...
@@ -80,8 +80,7 @@ again:
80 80
 			lock_set_dealloc(reply_semaphore);
81 81
 		}
82 82
 		if(i == 0) {
83
-			LOG(L_CRIT, "lock_initialize: could not allocate semaphore"
84
-						" sets\n");
83
+			LOG(L_CRIT, "could not allocate semaphore sets\n");
85 84
 			goto error;
86 85
 		}
87 86
 
... ...
@@ -1067,8 +1067,7 @@ int t_check_msg( struct sip_msg* p_msg , int *param_branch )
1067 1067
 			if (cfg_get(tm, tm_cfg, tm_aggregate_auth) &&
1068 1068
 					(p_msg->REPLY_STATUS==401 || p_msg->REPLY_STATUS==407)){
1069 1069
 				if (parse_headers(p_msg, HDR_EOH_F,0)==-1){
1070
-					LM_WARN("WARNING: the reply cannot be "
1071
-							"completely parsed\n");
1070
+					LM_WARN("the reply cannot be completely parsed\n");
1072 1071
 					/* try to continue, via1 & cseq are checked below */
1073 1072
 				}
1074 1073
 			}else if ( parse_headers(p_msg, HDR_VIA1_F|HDR_CSEQ_F|HDR_CALLID_F,
... ...
@@ -484,7 +484,7 @@ char *build_local_reparse(tm_cell_t *Trans,unsigned int branch,
484 484
 				/* final (end-of-headers) CRLF */
485 485
 				append_str(d, CRLF, CRLF_LEN);
486 486
 				*len = d - cancel_buf;
487
-				/* LOG(L_DBG, "DBG: build_local: %.*s\n", *len, cancel_buf); */
487
+				/* LOG(L_DBG, "%.*s\n", *len, cancel_buf); */
488 488
 				return cancel_buf;
489 489
 
490 490
 			default:
... ...
@@ -1666,8 +1666,7 @@ int t_retransmit_reply( struct cell *t )
1666 1666
 	 * the chances for this increase a lot.
1667 1667
 	 */
1668 1668
 	if (!t->uas.response.dst.send_sock) {
1669
-		LOG(L_WARN, "WARNING: t_retransmit_reply: "
1670
-			"no resolved dst to retransmit\n");
1669
+		LOG(L_WARN, "no resolved dst to retransmit\n");
1671 1670
 		return -1;
1672 1671
 	}
1673 1672
 
... ...
@@ -1677,14 +1676,13 @@ int t_retransmit_reply( struct cell *t )
1677 1676
 	LOCK_REPLIES( t );
1678 1677
 
1679 1678
 	if (!t->uas.response.buffer) {
1680
-		DBG("DBG: t_retransmit_reply: nothing to retransmit\n");
1679
+		DBG("nothing to retransmit\n");
1681 1680
 		goto error;
1682 1681
 	}
1683 1682
 
1684 1683
 	len=t->uas.response.buffer_len;
1685 1684
 	if ( len==0 || len>BUF_SIZE )  {
1686
-		DBG("DBG: t_retransmit_reply: "
1687
-			"zero length or too big to retransmit: %d\n", len);
1685
+		DBG("zero length or too big to retransmit: %d\n", len);
1688 1686
 		goto error;
1689 1687
 	}
1690 1688
 	memcpy( b, t->uas.response.buffer, len );