hello all
today testing kamailio 5.0.1 version we have seen that some rpc commands of benchmark module are not working fine.
when trying to change enable_global parameter, setting 1 for example it says that the parameter is invalid
/usr/local/kamailio-5.0/sbin/kamcmd -s udp:127.0.0.1:2046 benchmark.enable_global 1
error: 500 - Invalid Parameter Value
besides, it also permits to set a granularity value of 0 without giving error.
By doing that, if the kamailio has calls in progress, the kamailio get a crash
Core was generated by `/usr/local/kamailio-5.0/sbin/kamailio -P /var/run/kamailio/kamailio_talos.pid -'.
Program terminated with signal 8, Arithmetic exception.
#0 0x00007f4d55989757 in _bm_log_timer (id=0) at benchmark.c:320
320 if ((bm_mycfg->tindex[id]->calls % bm_mycfg->granularity) == 0)
Missing separate debuginfos, use: debuginfo-install glibc-2.12-1.132.el6.x86_64 libxml2-2.7.6-14.el6.x86_64 pcre-7.8-6.el6.x86_64 zlib-1.2.3-29.el6.x86_64
i understand that maybe is trying to divide by 0 to calcutate the benchmark stats.
checking the functions, we made a change to fix this problem, by chaning the type of the vars defined as "int", instead of "long int".
this way seems the commands work fine
/usr/local/kamailio-5.0/sbin/kamcmd -s udp:127.0.0.1:2046 benchmark.enable_global 1
/usr/local/kamailio-5.0/sbin/kamcmd -s udp:127.0.0.1:2046 benchmark.granularity 0
error: 500 - Invalid Parameter Value
this are the changes we made
index d40469b..5a353d5 100644
--- a/src/modules/benchmark/benchmark.c
+++ b/src/modules/benchmark/benchmark.c
@@ -479,7 +479,8 @@ static inline int fixup_bm_timer(void** param, int param_no)
*/
void bm_rpc_enable_global(rpc_t* rpc, void* ctx)
{
- long int v1;
+ int v1;
+ /*long int v1;*/
if(rpc->scan(ctx, "d", (int*)(&v1))<1) {
LM_WARN("no parameters\n");
rpc->fault(ctx, 500, "Invalid Parameters");
@@ -518,7 +519,8 @@ void bm_rpc_enable_timer(rpc_t* rpc, void* ctx)
void bm_rpc_granularity(rpc_t* rpc, void* ctx)
{
- long int v1;
+ int v1;
+ /*long int v1;*/
if(rpc->scan(ctx, "d", (int*)(&v1))<1) {
LM_WARN("no parameters\n");
rpc->fault(ctx, 500, "Invalid Parameters");
@@ -533,7 +535,8 @@ void bm_rpc_granularity(rpc_t* rpc, void* ctx)
void bm_rpc_loglevel(rpc_t* rpc, void* ctx)
{
- long int v1;
+ int v1;
+ /*long int v1;*/
if(rpc->scan(ctx, "d", (int*)(&v1))<1) {
LM_WARN("no parameters\n");
rpc->fault(ctx, 500, "Invalid Parameters");
could you please check if this fix is enough? and maybe there can be some other functions with this problem?
thanks alot and regards
david