From ee0c503615dd0834bed2b569570de003ded3a91f 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 e736530c34c45019343a3e3cb9ef9be3e7dd3a4b) --- daemon/sdp.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/daemon/sdp.c b/daemon/sdp.c index 06ce5e48c..84aa15e69 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 */ } /**