[sr-dev] git:master:8afbefda: textops: pkg free pmatch for regex_substring()

Module: kamailio Branch: master Commit: 8afbefda8b77db19f38c7f8984fe683c30ffc95d URL: https://github.com/kamailio/kamailio/commit/8afbefda8b77db19f38c7f8984fe683… Author: Daniel-Constantin Mierla &lt;miconda(a)gmail.com&gt; Committer: Daniel-Constantin Mierla &lt;miconda(a)gmail.com&gt; Date: 2018-12-24T14:04:16+01:00 textops: pkg free pmatch for regex_substring() --- Modified: src/modules/textops/textops.c --- Diff: https://github.com/kamailio/kamailio/commit/8afbefda8b77db19f38c7f8984fe683… Patch: https://github.com/kamailio/kamailio/commit/8afbefda8b77db19f38c7f8984fe683… --- diff --git a/src/modules/textops/textops.c b/src/modules/textops/textops.c index abf8e3e785..ab7d894bdd 100644 --- a/src/modules/textops/textops.c +++ b/src/modules/textops/textops.c @@ -833,6 +833,7 @@ static int ki_regex_substring(sip_msg_t* msg, str *input, str *regex, if (0 != rc) { LM_ERR("regular experession coudnt be compiled, Error code: (%d)\n", rc); + pkg_free(pmatch); regfree(&preg); return -1; } @@ -841,12 +842,14 @@ static int ki_regex_substring(sip_msg_t* msg, str *input, str *regex, regfree(&preg); if(rc!=0) { LM_DBG("no matches\n"); + pkg_free(pmatch); return -2; } /* matched */ if (pmatch[mindex].rm_so==-1) { LM_WARN("invalid offset for regular expression result\n"); + pkg_free(pmatch); return -1; } @@ -855,6 +858,7 @@ static int ki_regex_substring(sip_msg_t* msg, str *input, str *regex, if (pmatch[mindex].rm_so==pmatch[mindex].rm_eo) { LM_WARN("Matched string is empty\n"); + pkg_free(pmatch); return -1; } @@ -863,6 +867,7 @@ static int ki_regex_substring(sip_msg_t* msg, str *input, str *regex, if(tempstr.s== NULL || tempstr.len<=0) { LM_WARN("matched token is null\n"); + pkg_free(pmatch); return -1; } @@ -871,6 +876,7 @@ static int ki_regex_substring(sip_msg_t* msg, str *input, str *regex, valx.rs.len=tempstr.len; LM_DBG("result: %.*s\n", valx.rs.len, valx.rs.s); pvresult->setf(msg, &pvresult->pvp, (int)EQ_T, &valx); + pkg_free(pmatch); return 1; }