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

Author: Federico Cabiddu <federico.cabi...@gmail.com>
Committer: Federico Cabiddu <federico.cabi...@gmail.com>
Date: 2024-11-25T08:32:48+01:00

ldap: add fixup_free* to exported functions

---

Modified: src/modules/ldap/ldap_mod.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/adffa9e2b31bf0ad7070420ac821a914fc2bcd6b.diff
Patch: 
https://github.com/kamailio/kamailio/commit/adffa9e2b31bf0ad7070420ac821a914fc2bcd6b.patch

---

diff --git a/src/modules/ldap/ldap_mod.c b/src/modules/ldap/ldap_mod.c
index 27d44b9d88b..8303885f6ba 100644
--- a/src/modules/ldap/ldap_mod.c
+++ b/src/modules/ldap/ldap_mod.c
@@ -59,6 +59,8 @@ static int child_init(int rank);
 static int ldap_result_fixup(void **param, int param_no);
 static int ldap_filter_url_encode_fixup(void **param, int param_no);
 static int ldap_result_check_fixup(void **param, int param_no);
+static int ldap_fixup_free(void **param, int param_no);
+static int ldap_filter_url_encode_fixup_free(void **param, int param_no);
 
 /*
 * exported functions
@@ -94,25 +96,25 @@ static dictionary *config_vals = NULL;
 /* clang-format off */
 static cmd_export_t cmds[] = {
        {"ldap_search",            (cmd_function)w_ldap_search,            1,
-               fixup_spve_null, 0,
+               fixup_spve_null, fixup_free_spve_null,
                
REQUEST_ROUTE|FAILURE_ROUTE|BRANCH_ROUTE|ONREPLY_ROUTE|LOCAL_ROUTE},
        {"ldap_result",            (cmd_function)w_ldap_result1,           1,
-               ldap_result_fixup, 0,
+               ldap_result_fixup, ldap_fixup_free,
                
REQUEST_ROUTE|FAILURE_ROUTE|BRANCH_ROUTE|ONREPLY_ROUTE|LOCAL_ROUTE},
        {"ldap_result",            (cmd_function)w_ldap_result2,           2,
-               ldap_result_fixup, 0,
+               ldap_result_fixup, ldap_fixup_free,
                
REQUEST_ROUTE|FAILURE_ROUTE|BRANCH_ROUTE|ONREPLY_ROUTE|LOCAL_ROUTE},
        {"ldap_result_next",       (cmd_function)w_ldap_result_next,       0,
                0, 0,
                
REQUEST_ROUTE|FAILURE_ROUTE|BRANCH_ROUTE|ONREPLY_ROUTE|LOCAL_ROUTE},
        {"ldap_result_check",      (cmd_function)w_ldap_result_check_1,    1,
-               ldap_result_check_fixup, 0,
+               ldap_result_check_fixup, ldap_fixup_free,
                
REQUEST_ROUTE|FAILURE_ROUTE|BRANCH_ROUTE|ONREPLY_ROUTE|LOCAL_ROUTE},
        {"ldap_result_check",      (cmd_function)w_ldap_result_check_2,    2,
-               ldap_result_check_fixup, 0,
+               ldap_result_check_fixup, ldap_fixup_free,
                
REQUEST_ROUTE|FAILURE_ROUTE|BRANCH_ROUTE|ONREPLY_ROUTE|LOCAL_ROUTE},
        {"ldap_filter_url_encode", (cmd_function)w_ldap_filter_url_encode, 2,
-               ldap_filter_url_encode_fixup, 0,
+               ldap_filter_url_encode_fixup, ldap_filter_url_encode_fixup_free,
                
REQUEST_ROUTE|FAILURE_ROUTE|BRANCH_ROUTE|ONREPLY_ROUTE|LOCAL_ROUTE},
        {"load_ldap",              (cmd_function)load_ldap,  0,
                0, 0, 0},
@@ -374,6 +376,14 @@ static int ldap_result_fixup(void **param, int param_no)
        return 0;
 }
 
+static int ldap_fixup_free(void **param, int param_no)
+{
+       if(param_no == 1) {
+               pkg_free(*param);
+       }
+       return 0;
+}
+
 static int ldap_result_check_fixup(void **param, int param_no)
 {
        struct ldap_result_check_params *lp;
@@ -475,6 +485,14 @@ static int ldap_filter_url_encode_fixup(void **param, int 
param_no)
        return 0;
 }
 
+static int ldap_filter_url_encode_fixup_free(void **param, int param_no)
+{
+       if(param_no == 2) {
+               pkg_free(*param);
+       }
+       return 0;
+}
+
 /**
  *
  */

_______________________________________________
Kamailio - Development Mailing List -- sr-dev@lists.kamailio.org
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
Important: keep the mailing list in the recipients, do not reply only to the 
sender!

Reply via email to