[sr-dev] git:master:1d8c6e43: tm: skip empty lists when listing transactions via rpc tm.list

Module: kamailio Branch: master Commit: 1d8c6e4330fc069ed6606e0b966ece76053909b8 URL: https://github.com/kamailio/kamailio/commit/1d8c6e4330fc069ed6606e0b966ece7… Author: Daniel-Constantin Mierla &lt;miconda(a)gmail.com&gt; Committer: Daniel-Constantin Mierla &lt;miconda(a)gmail.com&gt; Date: 2017-08-31T11:20:17+02:00 tm: skip empty lists when listing transactions via rpc tm.list --- Modified: src/modules/tm/t_stats.c --- Diff: https://github.com/kamailio/kamailio/commit/1d8c6e4330fc069ed6606e0b966ece7… Patch: https://github.com/kamailio/kamailio/commit/1d8c6e4330fc069ed6606e0b966ece7… --- diff --git a/src/modules/tm/t_stats.c b/src/modules/tm/t_stats.c index 5b1cc5fbbc..d6ae5f52e4 100644 --- a/src/modules/tm/t_stats.c +++ b/src/modules/tm/t_stats.c @@ -260,34 +260,37 @@ void tm_rpc_list(rpc_t* rpc, void* c) tm_cell_t *tcell; for (r=0; r<TABLE_ENTRIES; r++) { - if(_tm_table->entries[r].next_c!=NULL) { - if (rpc->add(c, "{", &h) < 0) { - LM_ERR("failed to add transaction structure\n"); - return; - } - lock_hash(r); - clist_foreach(&_tm_table->entries[r], tcell, next_c) - { - rpc->struct_add(h, "ddSSSSSsdddd", - "tindex", (unsigned)tcell->hash_index, - "tlabel", (unsigned)tcell->label, - "method", &tcell->method, - "from", &tcell->from, - "to", &tcell->to, - "callid", &tcell->callid, - "cseq", &tcell->cseq_n, - "uas_request", (tcell->uas.request)?"yes":"no", - "tflags", (unsigned)tcell->flags, - "outgoings", (unsigned)tcell->nr_of_outgoings, + lock_hash(r); + if(clist_empty(&_tm_table->entries[r], next_c)) { + unlock_hash(r); + continue; + } + if (rpc->add(c, "{", &h) < 0) { + LM_ERR("failed to add transaction structure\n"); + unlock_hash(r); + return; + } + clist_foreach(&_tm_table->entries[r], tcell, next_c) + { + rpc->struct_add(h, "ddSSSSSsdddd", + "tindex", (unsigned)tcell->hash_index, + "tlabel", (unsigned)tcell->label, + "method", &tcell->method, + "from", &tcell->from, + "to", &tcell->to, + "callid", &tcell->callid, + "cseq", &tcell->cseq_n, + "uas_request", (tcell->uas.request)?"yes":"no", + "tflags", (unsigned)tcell->flags, + "outgoings", (unsigned)tcell->nr_of_outgoings, #ifdef TM_DEL_UNREF - "ref_count", (unsigned)atomic_get(&tcell->ref_count), + "ref_count", (unsigned)atomic_get(&tcell->ref_count), #else - "ref_count", tcell->ref_count, + "ref_count", tcell->ref_count, #endif - "lifetime", (unsigned)TICKS_TO_S(tcell->end_of_life) - ); - } - unlock_hash(r); + "lifetime", (unsigned)TICKS_TO_S(tcell->end_of_life) + ); } + unlock_hash(r); } }