Hello,
can you make a pull request with your fixes on github project? It makes it easier to review and merge if all ok.
Cheers, Daniel
On 31.05.17 18:06, David Escartín Almudévar wrote:
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
Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev