[sr-dev] git:4.3:0aaa2dd0: tsilo: unref the transaction after appending new branches

Module: kamailio Branch: 4.3 Commit: 0aaa2dd0afab46cb35e88d4091a9118066682059 URL: https://github.com/kamailio/kamailio/commit/0aaa2dd0afab46cb35e88d4091a9118… Author: Federico Cabiddu &lt;federico.cabiddu(a)gmail.com&gt; Committer: Federico Cabiddu &lt;federico.cabiddu(a)gmail.com&gt; Date: 2015-10-08T08:19:39+02:00 tsilo: unref the transaction after appending new branches (cherry picked from commit 3c086a85b99d59dabc48f77893d4c68199af0577) --- Modified: modules/tsilo/ts_append.c --- Diff: https://github.com/kamailio/kamailio/commit/0aaa2dd0afab46cb35e88d4091a9118… Patch: https://github.com/kamailio/kamailio/commit/0aaa2dd0afab46cb35e88d4091a9118… --- diff --git a/modules/tsilo/ts_append.c b/modules/tsilo/ts_append.c index 100fcec..99cf89e 100644 --- a/modules/tsilo/ts_append.c +++ b/modules/tsilo/ts_append.c @@ -49,9 +49,9 @@ int ts_append(struct sip_msg* msg, str *ruri, char *table) { if (res != 0) { LM_ERR("failed to retrieve record for %.*s\n", ruri->len, ruri->s); - unlock_entry_by_ruri(ruri); - return -1; - } + unlock_entry_by_ruri(ruri); + return -1; + } ptr = _r->transactions; @@ -68,24 +68,32 @@ int ts_append(struct sip_msg* msg, str *ruri, char *table) { int ts_append_to(struct sip_msg* msg, int tindex, int tlabel, char *table) { struct cell *t; + struct cell *orig_t; struct sip_msg *orig_msg; int ret; + orig_t = _tmb.t_gett(); + if(_tmb.t_lookup_ident(&t, tindex, tlabel) < 0) { LM_ERR("transaction [%u:%u] not found\n", tindex, tlabel); - return -1; + ret = -1; + goto done; } + if (t->flags & T_CANCELED) { LM_DBG("trasaction [%u:%u] was cancelled\n", tindex, tlabel); - return -2; + ret = -2; + goto done; } + if (t->uas.status >= 200) { LM_DBG("trasaction [%u:%u] sent out a final response already - %d\n", tindex, tlabel, t->uas.status); - return -3; + ret = -3; + goto done; } orig_msg = t->uas.request; @@ -93,8 +101,17 @@ int ts_append_to(struct sip_msg* msg, int tindex, int tlabel, char *table) { ret = _regapi.lookup_to_dset(orig_msg, table, NULL); if(ret != 1) { LM_DBG("transaction %u:%u: error updating dset (%d)\n", tindex, tlabel, ret); - return -4; + ret = -4; + goto done; } - - return _tmb.t_append_branches(); + + ret = _tmb.t_append_branches(); + +done: + /* unref the transaction which had been referred by t_lookup_ident() call. + * Restore the original transaction (if any) */ + _tmb.unref_cell(t); + _tmb.t_sett(orig_t, T_BR_UNDEFINED); + + return ret; }