[sr-dev] git:master:dddd530e: core: cfg.lex - do not destroy action if pre-fixup fails

[Daniel-Constantin Mierla]

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

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date: 2021-02-11T10:27:20+01:00

core: cfg.lex - do not destroy action if pre-fixup fails

- params might be in inconsistent state, shutdown is triggered which
cleans the pkg anyhow
- GH #2630

---

Modified: src/core/cfg.y

---

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

---

diff --git a/src/core/cfg.y b/src/core/cfg.y
index 0d991ee98b..f659aad087 100644
--- a/src/core/cfg.y
+++ b/src/core/cfg.y
@@ -3542,7 +3542,7 @@ cmd:
 		}else{
 			if (mod_func_action && mod_f_params_pre_fixup(mod_func_action)<0) {
 				/* error messages are printed inside the function */
-				free_mod_func_action(mod_func_action);
+				/* free_mod_func_action(mod_func_action); */
 				mod_func_action = 0;
 				YYERROR;
 			}