From 954e3a52787013c360c11e8de7740b0480cbf840 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) --- daemon/sdp.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/daemon/sdp.c b/daemon/sdp.c index cf37f7bd5..c2456cb89 100644 --- a/daemon/sdp.c +++ b/daemon/sdp.c @@ -322,13 +322,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 */ } /**