Module: kamailio Branch: master Commit: df8e38958a3fa0a2e356bec727bb8d7ce19bec43 URL: https://github.com/kamailio/kamailio/commit/df8e38958a3fa0a2e356bec727bb8d7c...

Author: Daniel-Constantin Mierla miconda@gmail.com Committer: Daniel-Constantin Mierla miconda@gmail.com Date: 2025-05-27T08:58:17+02:00

secfilter: remove unnecessary check on clean up

---

Modified: src/modules/secfilter/secfilter_db.c

---

Diff: https://github.com/kamailio/kamailio/commit/df8e38958a3fa0a2e356bec727bb8d7c... Patch: https://github.com/kamailio/kamailio/commit/df8e38958a3fa0a2e356bec727bb8d7c...

---

diff --git a/src/modules/secfilter/secfilter_db.c b/src/modules/secfilter/secfilter_db.c index 1c0138b0193..24884b231c9 100644 --- a/src/modules/secfilter/secfilter_db.c +++ b/src/modules/secfilter/secfilter_db.c @@ -185,9 +185,9 @@ int secf_append_rule(int action, int type, str *value) /** * @brief Removes entries from a whitelist or blacklist based on action, type, and value. * - * The function validates the action and type, locates the relevant list, and iterates - * through its nodes to find and remove all matches for the given value. If matches are found, - * the corresponding memory is freed, and the list pointers are updated. Logs are generated + * The function validates the action and type, locates the relevant list, and iterates + * through its nodes to find and remove all matches for the given value. If matches are found, + * the corresponding memory is freed, and the list pointers are updated. Logs are generated * for each removal and at the end of the process to summarize the result. * @param action: Specifies the target list (0 = blacklist, 1 = whitelist, 2 = destination blacklist). * @param type: Indicates the type of rule to be removed (e.g., domain, IP, user, etc.). @@ -358,10 +358,8 @@ int secf_load_db(void) lock_release(&(*secf_data)->lock);

clean: - if(db_res) { - if(db_funcs.free_result(db_handle, db_res) < 0) { - LM_DBG("Failed to free the result\n"); - } + if(db_funcs.free_result(db_handle, db_res) < 0) { + LM_DBG("Failed to free the result\n"); } db_funcs.close(db_handle); return res;