From 980a35030d45efece51d38fceb334b1ee94c5cea Mon Sep 17 00:00:00 2001 From: Donat Zenichev Date: Fri, 18 Oct 2024 14:15:57 +0200 Subject: [PATCH] MT#56128 Re-move misleading log line Re-move misleading log line to another place. In the `sdp_manipulate_remove()` function: - true - means remove - false - means don't remove Correct logging accordingly. Change-Id: I3e0c08552a0192f3985da990b5f0c96cc31ece94 (cherry picked from commit ac488e3e10dfc5942eb028ff77f056a4c07854af) (cherry picked from commit 0eec1b316f9f2bfdb86d98de662eb330a3773ac8) --- daemon/sdp.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/daemon/sdp.c b/daemon/sdp.c index 074d30b65..67405ee2b 100644 --- a/daemon/sdp.c +++ b/daemon/sdp.c @@ -307,13 +307,13 @@ static bool sdp_manipulate_remove(struct sdp_manipulations * sdp_manipulations, return false; str_case_ht ht = sdp_manipulations->rem_commands; - if (t_hash_table_is_set(ht) && t_hash_table_lookup(ht, attr_name)) - return true; - - ilog(LOG_DEBUG, "Cannot insert: '" STR_FORMAT "' because prevented by SDP manipulations (remove)", - STR_FMT(attr_name)); + if (t_hash_table_is_set(ht) && t_hash_table_lookup(ht, attr_name)) { + ilog(LOG_DEBUG, "Cannot insert: '" STR_FORMAT "' because prevented by SDP manipulations (remove)", + STR_FMT(attr_name)); + return true; /* means remove */ + } - return false; + return false; /* means don't remove */ } /**