[core] do not use LM_xxx if log_level is NULL

This may never happen, but as LM_xxx is testing the log_level value, it is not wise to use it to report that log_level is NULL Reported by coverity CID 207925
OpenSIPS · Jul 15, 2020 · e43b6e5 · e43b6e5
1 parent f564faa
commit e43b6e5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/core_cmds.c b/core_cmds.c
@@ -1290,7 +1290,8 @@ static int w_script_trace(struct sip_msg *msg, int *log_level,
 		use_script_trace = 0;
 		return 1;
 	} else if (!log_level) {
-		LM_ERR("Missing 'log_level' parameter\n");
+		// this should not happen, it's just a super precaution
+		// and no LM_xxx here, as it assumes a valid log_level
 		return E_CFG;
 	} else if (!fmt_string) {
 		LM_ERR("Missing 'pv_format_string' parameter\n");