sr-dev

sr-dev@lists.kamailio.org

15 participants
44239 discussions

git:sr_3.0: parser: fix for solaris yacc compatibility

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: 835c266e5a5a421fb8b65077e053e06e43fb326a URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=835c266… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Dec 2 20:12:40 2009 -0500 parser: fix for solaris yacc compatibility %prec was used in a wrong way (not at the end of the rule), causing problems with some yacc implementations. --- cfg.y | 60 ++++++++++++++++++++++++++++++------------------------------ 1 files changed, 30 insertions(+), 30 deletions(-) diff --git a/cfg.y b/cfg.y index 585edf5..bfe45e4 100644 --- a/cfg.y +++ b/cfg.y @@ -540,7 +540,7 @@ extern char *finame; %left GT LT GTE LTE %left PLUS MINUS %left STAR SLASH MODULO -%right NOT +%right NOT UNARY %right DEFINED %right INTCAST STRCAST %left DOT @@ -730,7 +730,7 @@ id_lst: ; intno: NUMBER - | MINUS %prec NOT NUMBER { $$=-$2; } + | MINUS NUMBER %prec UNARY { $$=-$2; } ; flags_decl: FLAGS_DECL flag_list @@ -1840,41 +1840,41 @@ eip_op: SRCIP { $$=SRCIP_O; } exp_elem: - METHOD strop %prec EQUAL_T rval_expr + METHOD strop rval_expr %prec EQUAL_T {$$= mk_elem($2, METHOD_O, 0, RVE_ST, $3);} - | METHOD strop %prec EQUAL_T ID + | METHOD strop ID %prec EQUAL_T {$$ = mk_elem($2, METHOD_O, 0, STRING_ST,$3); } | METHOD strop error { $$=0; yyerror("string expected"); } | METHOD error { $$=0; yyerror("invalid operator,== , !=, or =~ expected"); } - | uri_type strop %prec EQUAL_T rval_expr + | uri_type strop rval_expr %prec EQUAL_T {$$ = mk_elem($2, $1, 0, RVE_ST, $3); } - | uri_type strop %prec EQUAL_T MYSELF + | uri_type strop MYSELF %prec EQUAL_T {$$=mk_elem($2, $1, 0, MYSELF_ST, 0); } - | uri_type strop %prec EQUAL_T error + | uri_type strop error %prec EQUAL_T { $$=0; yyerror("string or MYSELF expected"); } | uri_type error { $$=0; yyerror("invalid operator, == , != or =~ expected"); } - | eint_op cmpop %prec GT rval_expr { $$=mk_elem($2, $1, 0, RVE_ST, $3 ); } - | eint_op equalop %prec EQUAL_T rval_expr + | eint_op cmpop rval_expr %prec GT { $$=mk_elem($2, $1, 0, RVE_ST, $3 ); } + | eint_op equalop rval_expr %prec EQUAL_T { $$=mk_elem($2, $1, 0, RVE_ST, $3 ); } | eint_op cmpop error { $$=0; yyerror("number expected"); } | eint_op equalop error { $$=0; yyerror("number expected"); } | eint_op error { $$=0; yyerror("==, !=, <,>, >= or <= expected"); } - | PROTO equalop %prec EQUAL_T proto + | PROTO equalop proto %prec EQUAL_T { $$=mk_elem($2, PROTO_O, 0, NUMBER_ST, (void*)$3 ); } - | PROTO equalop %prec EQUAL_T rval_expr + | PROTO equalop rval_expr %prec EQUAL_T { $$=mk_elem($2, PROTO_O, 0, RVE_ST, $3 ); } | PROTO equalop error { $$=0; yyerror("protocol expected (udp, tcp, tls or sctp)"); } - | SNDPROTO equalop %prec EQUAL_T proto + | SNDPROTO equalop proto %prec EQUAL_T { $$=mk_elem($2, SNDPROTO_O, 0, NUMBER_ST, (void*)$3 ); } - | SNDPROTO equalop %prec EQUAL_T rval_expr + | SNDPROTO equalop rval_expr %prec EQUAL_T { $$=mk_elem($2, SNDPROTO_O, 0, RVE_ST, $3 ); } | SNDPROTO equalop error { $$=0; yyerror("protocol expected (udp, tcp, tls or sctp)"); } - | eip_op strop %prec EQUAL_T ipnet { $$=mk_elem($2, $1, 0, NET_ST, $3); } - | eip_op strop %prec EQUAL_T rval_expr { + | eip_op strop ipnet %prec EQUAL_T { $$=mk_elem($2, $1, 0, NET_ST, $3); } + | eip_op strop rval_expr %prec EQUAL_T { s_tmp.s=0; $$=0; if (rve_is_constant($3)){ @@ -1912,29 +1912,29 @@ exp_elem: $$=mk_elem($2, $1, 0, RVE_ST, $3); } } - | eip_op strop %prec EQUAL_T host + | eip_op strop host %prec EQUAL_T { $$=mk_elem($2, $1, 0, STRING_ST, $3); } - | eip_op strop %prec EQUAL_T MYSELF + | eip_op strop MYSELF %prec EQUAL_T { $$=mk_elem($2, $1, 0, MYSELF_ST, 0); } - | eip_op strop %prec EQUAL_T error + | eip_op strop error %prec EQUAL_T { $$=0; yyerror( "ip address or hostname expected" ); } | eip_op error { $$=0; yyerror("invalid operator, ==, != or =~ expected");} - | MYSELF equalop %prec EQUAL_T uri_type + | MYSELF equalop uri_type %prec EQUAL_T { $$=mk_elem($2, $3, 0, MYSELF_ST, 0); } - | MYSELF equalop %prec EQUAL_T eip_op + | MYSELF equalop eip_op %prec EQUAL_T { $$=mk_elem($2, $3, 0, MYSELF_ST, 0); } - | MYSELF equalop %prec EQUAL_T error + | MYSELF equalop error %prec EQUAL_T { $$=0; yyerror(" URI, SRCIP or DSTIP expected"); } | MYSELF error { $$=0; yyerror ("invalid operator, == or != expected"); } ; /* exp_elem2: - rval_expr cmpop %prec GT rval_expr + rval_expr cmpop rval_expr %prec GT { $$=mk_elem( $2, RVE_ST, $1, RVE_ST, $3);} | - rval_expr equalop %prec EQUAL_T rval_expr + rval_expr equalop rval_expr %prec EQUAL_T { $$=mk_elem( $2, RVE_ST, $1, RVE_ST, $3);} | rval_expr LOG_AND rval_expr { $$=mk_exp_rve(LOGAND_OP, $1, $3);} @@ -2527,7 +2527,7 @@ rval: intno {$$=mk_rve_rval(RV_INT, (void*)$1); } rve_un_op: NOT { $$=RVE_LNOT_OP; } - | MINUS %prec NOT { $$=RVE_UMINUS_OP; } + | MINUS %prec UNARY { $$=RVE_UMINUS_OP; } /* TODO: RVE_BOOL_OP, RVE_NOT_OP? */ ; @@ -2546,7 +2546,7 @@ rval_expr: rval { $$=$1; YYERROR; } } - | rve_un_op %prec NOT rval_expr {$$=mk_rve1($1, $2); } + | rve_un_op rval_expr %prec UNARY {$$=mk_rve1($1, $2); } | INTCAST rval_expr {$$=mk_rve1(RVE_INT_OP, $2); } | STRCAST rval_expr {$$=mk_rve1(RVE_STR_OP, $2); } | rval_expr PLUS rval_expr {$$=mk_rve2(RVE_PLUS_OP, $1, $3); } @@ -2556,8 +2556,8 @@ rval_expr: rval { $$=$1; | rval_expr MODULO rval_expr {$$=mk_rve2(RVE_MOD_OP, $1, $3); } | rval_expr BIN_OR rval_expr {$$=mk_rve2(RVE_BOR_OP, $1, $3); } | rval_expr BIN_AND rval_expr {$$=mk_rve2(RVE_BAND_OP, $1, $3);} - | rval_expr rve_cmpop %prec GT rval_expr { $$=mk_rve2( $2, $1, $3);} - | rval_expr rve_equalop %prec EQUAL_T rval_expr + | rval_expr rve_cmpop rval_expr %prec GT { $$=mk_rve2( $2, $1, $3);} + | rval_expr rve_equalop rval_expr %prec EQUAL_T { $$=mk_rve2( $2, $1, $3);} | rval_expr LOG_AND rval_expr { $$=mk_rve2(RVE_LAND_OP, $1, $3);} | rval_expr LOG_OR rval_expr { $$=mk_rve2(RVE_LOR_OP, $1, $3);} @@ -2565,7 +2565,7 @@ rval_expr: rval { $$=$1; | STRLEN LPAREN rval_expr RPAREN { $$=mk_rve1(RVE_STRLEN_OP, $3);} | STREMPTY LPAREN rval_expr RPAREN {$$=mk_rve1(RVE_STREMPTY_OP, $3);} | DEFINED rval_expr { $$=mk_rve1(RVE_DEFINED_OP, $2);} - | rve_un_op %prec NOT error { $$=0; yyerror("bad expression"); } + | rve_un_op error %prec UNARY { $$=0; yyerror("bad expression"); } | INTCAST error { $$=0; yyerror("bad expression"); } | STRCAST error { $$=0; yyerror("bad expression"); } | rval_expr PLUS error { $$=0; yyerror("bad expression"); } @@ -2575,9 +2575,9 @@ rval_expr: rval { $$=$1; | rval_expr MODULO error { $$=0; yyerror("bad expression"); } | rval_expr BIN_OR error { $$=0; yyerror("bad expression"); } | rval_expr BIN_AND error { $$=0; yyerror("bad expression"); } - | rval_expr rve_cmpop %prec GT error + | rval_expr rve_cmpop error %prec GT { $$=0; yyerror("bad expression"); } - | rval_expr rve_equalop %prec EQUAL_T error + | rval_expr rve_equalop error %prec EQUAL_T { $$=0; yyerror("bad expression"); } | rval_expr LOG_AND error { $$=0; yyerror("bad expression"); } | rval_expr LOG_OR error { $$=0; yyerror("bad expression"); }

15 years, 1 month

git:sr_3.0: core: pid_t warning fixes for solaris

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: ca351abe9bd52f24003a96ed4f9a420739728496 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ca351ab… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Dec 2 20:15:36 2009 -0500 core: pid_t warning fixes for solaris --- main.c | 12 ++++++------ tcp_main.c | 21 +++++++++++---------- 2 files changed, 17 insertions(+), 16 deletions(-) diff --git a/main.c b/main.c index fc25588..fd4b642 100644 --- a/main.c +++ b/main.c @@ -713,19 +713,19 @@ void handle_sigs() case SIGCHLD: while ((chld=waitpid( -1, &chld_status, WNOHANG ))>0) { if (WIFEXITED(chld_status)) - LOG(L_ALERT, "child process %d exited normally," - " status=%d\n", chld, + LOG(L_ALERT, "child process %ld exited normally," + " status=%d\n", (long)chld, WEXITSTATUS(chld_status)); else if (WIFSIGNALED(chld_status)) { - LOG(L_ALERT, "child process %d exited by a signal" - " %d\n", chld, WTERMSIG(chld_status)); + LOG(L_ALERT, "child process %ld exited by a signal" + " %d\n", (long)chld, WTERMSIG(chld_status)); #ifdef WCOREDUMP LOG(L_ALERT, "core was %sgenerated\n", WCOREDUMP(chld_status) ? "" : "not " ); #endif }else if (WIFSTOPPED(chld_status)) - LOG(L_ALERT, "child process %d stopped by a" - " signal %d\n", chld, + LOG(L_ALERT, "child process %ld stopped by a" + " signal %d\n", (long)chld, WSTOPSIG(chld_status)); } #ifndef STOP_JIRIS_CHANGES diff --git a/tcp_main.c b/tcp_main.c index 77ecdf5..4840ddc 100644 --- a/tcp_main.c +++ b/tcp_main.c @@ -2810,8 +2810,9 @@ inline static int handle_tcp_child(struct tcp_child* tcp_c, int fd_i) if (unlikely(tcp_c->unix_sock<=0)){ /* (we can't have a fd==0, 0 is never closed )*/ LOG(L_CRIT, "BUG: handle_tcp_child: fd %d for %d " - "(pid %d, ser no %d)\n", tcp_c->unix_sock, - (int)(tcp_c-&tcp_children[0]), tcp_c->pid, tcp_c->proc_no); + "(pid %ld, ser no %d)\n", tcp_c->unix_sock, + (int)(tcp_c-&tcp_children[0]), (long)tcp_c->pid, + tcp_c->proc_no); goto error; } /* read until sizeof(response) @@ -2820,9 +2821,9 @@ inline static int handle_tcp_child(struct tcp_child* tcp_c, int fd_i) if (unlikely(bytes<(int)sizeof(response))){ if (bytes==0){ /* EOF -> bad, child has died */ - DBG("DBG: handle_tcp_child: dead tcp child %d (pid %d, no %d)" + DBG("DBG: handle_tcp_child: dead tcp child %d (pid %ld, no %d)" " (shutting down?)\n", (int)(tcp_c-&tcp_children[0]), - tcp_c->pid, tcp_c->proc_no ); + (long)tcp_c->pid, tcp_c->proc_no ); /* don't listen on it any more */ io_watch_del(&io_h, tcp_c->unix_sock, fd_i, 0); goto error; /* eof. so no more io here, it's ok to return error */ @@ -2831,8 +2832,8 @@ inline static int handle_tcp_child(struct tcp_child* tcp_c, int fd_i) * e.g.: SIGIO_RT overflow mode or EPOLL ET */ if ((errno!=EAGAIN) && (errno!=EWOULDBLOCK)){ LOG(L_CRIT, "ERROR: handle_tcp_child: read from tcp child %ld " - " (pid %d, no %d) %s [%d]\n", - (long)(tcp_c-&tcp_children[0]), tcp_c->pid, + " (pid %ld, no %d) %s [%d]\n", + (long)(tcp_c-&tcp_children[0]), (long)tcp_c->pid, tcp_c->proc_no, strerror(errno), errno ); }else{ bytes=0; @@ -2857,8 +2858,8 @@ inline static int handle_tcp_child(struct tcp_child* tcp_c, int fd_i) if (unlikely(tcpconn==0)){ /* should never happen */ LOG(L_CRIT, "BUG: handle_tcp_child: null tcpconn pointer received" - " from tcp child %d (pid %d): %lx, %lx\n", - (int)(tcp_c-&tcp_children[0]), tcp_c->pid, + " from tcp child %d (pid %ld): %lx, %lx\n", + (int)(tcp_c-&tcp_children[0]), (long)tcp_c->pid, response[0], response[1]) ; goto end; } @@ -3330,9 +3331,9 @@ inline static int send2child(struct tcp_connection* tcpconn) " connection passed to the least busy one (%d)\n", min_busy); } - DBG("send2child: to tcp child %d %d(%d), %p\n", idx, + DBG("send2child: to tcp child %d %d(%ld), %p\n", idx, tcp_children[idx].proc_no, - tcp_children[idx].pid, tcpconn); + (long)tcp_children[idx].pid, tcpconn); /* first make sure this child doesn't have pending request for * tcp_main (to avoid a possible deadlock: e.g. child wants to * send a release command, but the master fills its socket buffer

15 years, 1 month

git:sr_3.0: ctl: fix #includes on solaris

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: 4920290b90947fa59f7e794fc92abd6796950c33 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=4920290… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Dec 2 20:14:54 2009 -0500 ctl: fix #includes on solaris --- modules/ctl/init_socks.h | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/modules/ctl/init_socks.h b/modules/ctl/init_socks.h index 3eddea9..a2be3a4 100644 --- a/modules/ctl/init_socks.h +++ b/modules/ctl/init_socks.h @@ -31,6 +31,7 @@ #ifndef _init_socks_h #define _init_socks_h +#include <sys/types.h> #include <sys/un.h> #include "../../ip_addr.h"

15 years, 1 month

git:sr_3.0: makefile: fix assembler detection on solaris

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: 0f7ec046ce88202e508ced7c1b1d544f05529758 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=0f7ec04… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Dec 2 20:07:13 2009 -0500 makefile: fix assembler detection on solaris Assembler test waited forever for input (missing /dev/null input redirection for as -V). --- Makefile.defs | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/Makefile.defs b/Makefile.defs index 0684e21..7f4dd97 100644 --- a/Makefile.defs +++ b/Makefile.defs @@ -671,7 +671,8 @@ ifeq ($(CC_NAME), gcc) fi) ASPATH:=$(shell if [ -z "$(ASGCC)" ] ; then echo "as" ;\ else \ - if $(ASGCC) -V 2>/dev/null 1>/dev/null; then \ + if $(ASGCC) -V 2>/dev/null 1>/dev/null </dev/null; \ + then \ echo $(ASGCC); \ else echo "as" ; \ fi\

15 years, 1 month

rfc: t_drop_replies()

by Daniel-Constantin Mierla

Hello, based on the discussion from last weeks on sr-dev related to selection of reply code for failure route, I did some updates in kamailio_3.0 branch. Main goal was to have the option of Kamailio 1.5 compatibility mode. Here is the patch: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3ed6685… Description of the changes is included in the commit message. Andrei, can you double check if there is any possible side effect considering the new TM architecture inherited from ser? Patch is not that complex. Miklos, does it meet what we tried to cover in the discussion? Anything missing? Very much appreciated if anyone can test and send feedback. Cheers, Daniel -- Daniel-Constantin Mierla * http://www.asipto.com/

15 years, 1 month

git:kamailio_3.0: tm: drop replies behaviour based on cfg compat mode

by Daniel-Constantin Mierla

Module: sip-router Branch: kamailio_3.0 Commit: 3ed668500c9a0f0d2f864f5a56cf7fa886f8457c URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3ed6685… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Dec 2 19:01:09 2009 +0100 tm: drop replies behaviour based on cfg compat mode - default behaviour for #!KAMAILIO is to drop replies after failure route to avoid selecting same reply during the next failure event - for the rest of cases all replies are kept and used to select the code for new failure route (SER compat mode) - t_drop_replies() can take one parameter - 'n[one]' - do not drop any reply (good for #!KAMAILIO mode) - 'l[ast]' - drop replies received during last serial forking step - 'a[ll]' - drop all replies - if the parameter is missing, then will drop all replies --- modules/tm/t_reply.c | 20 ++++++++++++++++---- modules/tm/t_reply.h | 2 +- modules/tm/tm.c | 11 ++++++++++- 3 files changed, 27 insertions(+), 6 deletions(-) diff --git a/modules/tm/t_reply.c b/modules/tm/t_reply.c index a250f63..1573e4d 100644 --- a/modules/tm/t_reply.c +++ b/modules/tm/t_reply.c @@ -138,6 +138,7 @@ #include "t_lookup.h" #include "t_fwd.h" #include "../../fix_lumps.h" +#include "../../sr_compat.h" #include "t_stats.h" #include "uac.h" @@ -1124,7 +1125,10 @@ static enum rps t_should_relay_response( struct cell *Trans , int new_code, Trans->flags&=~T_6xx; /* clear the 6xx flag , we want to allow new branches from the failure route */ - drop_replies = 0; + if(sr_cfg_compat==SR_COMPAT_KAMAILIO) + drop_replies = 1; + else + drop_replies = 0; /* run ON_FAILURE handlers ( route and callbacks) */ if (unlikely(has_tran_tmcbs( Trans, TMCB_ON_FAILURE_RO|TMCB_ON_FAILURE) || Trans->on_negative )) { @@ -1137,7 +1141,15 @@ static enum rps t_should_relay_response( struct cell *Trans , int new_code, picked_code, extra_flags); if (unlikely(drop_replies)) { /* drop all the replies that we have already saved */ - for (i=0; i<branch_cnt; i++) { + i = 0; + if(drop_replies==2) + { + for(i=branch_cnt-1; i>=0; i--) + if(Trans->uac[i].flags&TM_UAC_FLAG_FB) + break; + if(i<0) i=0; + } + for (; i<branch_cnt; i++) { if (Trans->uac[i].reply && (Trans->uac[i].reply != FAKED_REPLY) && (Trans->uac[i].reply->msg_flags & FL_SHM_CLONE)) @@ -2237,14 +2249,14 @@ error: /* drops all the replies to make sure * that none of them is picked up again */ -void t_drop_replies(void) +void t_drop_replies(int v) { /* It is too risky to free the replies that are in shm mem at the middle of failure_route block, because other functions might need them as well. And it can also happen that the current reply is not yet in shm mem, we are just going to clone it. So better to set a flag and check it after failure_route has ended. (Miklos) */ - drop_replies = 1; + drop_replies = v; } #if 0 diff --git a/modules/tm/t_reply.h b/modules/tm/t_reply.h index c385697..0a97e6a 100644 --- a/modules/tm/t_reply.h +++ b/modules/tm/t_reply.h @@ -153,7 +153,7 @@ int t_pick_branch_blind(struct cell *t, int *res_code); /* drops all the replies to make sure * that none of them is picked up again */ -void t_drop_replies(void); +void t_drop_replies(int v); extern const char* rpc_reply_doc[2]; void rpc_reply(rpc_t* rpc, void* c); diff --git a/modules/tm/tm.c b/modules/tm/tm.c index 09f08c6..51a31c0 100644 --- a/modules/tm/tm.c +++ b/modules/tm/tm.c @@ -381,6 +381,8 @@ static cmd_export_t cmds[]={ REQUEST_ROUTE|ONREPLY_ROUTE|FAILURE_ROUTE|BRANCH_ROUTE }, {"t_drop_replies", w_t_drop_replies, 0, 0, FAILURE_ROUTE}, + {"t_drop_replies", w_t_drop_replies, 1, 0, + FAILURE_ROUTE}, {"t_save_lumps", w_t_save_lumps, 0, 0, REQUEST_ROUTE}, {"t_check_trans", t_check_trans, 0, 0, @@ -1725,7 +1727,14 @@ int t_grep_status(struct sip_msg* msg, char* status, char* bar) * that none of them is picked up again */ static int w_t_drop_replies(struct sip_msg* msg, char* foo, char* bar) { - t_drop_replies(); + if(foo==NULL) + t_drop_replies(1); + else if(*foo=='n') + t_drop_replies(0); + else if(*foo=='l') + t_drop_replies(2); + else + t_drop_replies(1); return 1; }

15 years, 1 month

sr and kamailio autogenerated tarballs

by Andrei Pelinescu-Onciul

Daily snapshots of sr_3.0, kamailio_3.0 and sr master branch can be downloaded from: http://sip-router.org/tarballs/sr/ . Note that new tarballs are generated only if there are changes for the corresponding branch (so if you don't see a tarball with today's date it means there was no change from the previous one). The tarballs are generated via make tar, which makes sure the correct repository version will be included in the compiled binary (make tar generates first autover.h before creating the archive). Andrei P.S.: we should really choose a short name

15 years, 1 month

git:kamailio_3.0: presence(k): fix off-by-one memory allocation

by Daniel-Constantin Mierla

Module: sip-router Branch: kamailio_3.0 Commit: 4ef016026292c71d0ad731a8895c6b89efe1c769 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=4ef0160… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Mon Nov 30 12:29:20 2009 +0100 presence(k): fix off-by-one memory allocation Due to some missing '()' ( (...)?resource.len:event.len + 1 instead of ((...)?resource.len:event.len) + 1 ) sometimes the memory allocated was less with one byte then what was needed. The fixed version also replaces ?: with MAX_unsigned() (does the same thing and it's easier to read). Reported-by: Andres Moya andres.moya.i gmail com (cherry picked from commit 229bd1f517cb9177dc731929c31153b2b9513215) --- modules_k/presence/notify.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/modules_k/presence/notify.c b/modules_k/presence/notify.c index 92c2f55..6d16033 100644 --- a/modules_k/presence/notify.c +++ b/modules_k/presence/notify.c @@ -1793,8 +1793,7 @@ str* create_winfo_xml(watcher_t* watchers, char* version, LM_ERR("while adding child\n"); goto error; } - res= (char*)pkg_malloc((resource.len>event.len)?resource.len:event.len - + 1); + res= (char*)pkg_malloc(MAX_unsigned(resource.len, event.len) + 1); if(res== NULL) { ERR_MEM(PKG_MEM_STR);

15 years, 1 month

Memory leak issue

by Piyush.Bansal＠relianceada.com

Hi All, I'm using SER version 2.0.0-rc1 as a SIP server and in that facing lot of memory leaks issue. I'm not able to enable the memory logs in SER. Can anybody help me identifying memory leaks or tell me how to enable memlogs so that I myself will try removing all the leaks... Plz help me ASAP, its urgent... Thanx n regards, --Piyush The information contained in this e-mail message is intended only for the use of the individual or entity to which it is addressed. If you are not the intended recipient, you should return it to the sender immediately. Please note that while we scan all e-mails for viruses we cannot guarantee that any e-mail is virus-free and accept no liability for any damage caused by any virus transmitted by this email.

15 years, 1 month

deamonize.c and the option DSYSLOG_ASYNC Kamailio 1.5

by Torben Friese

Hi all, I have a problem with the DSYSLOG_ASYNC option in the Makefile.defs If I have set this option I could not longer receive log messages in the "normal" running mode (forking, debug level=2 stderr = no). If I set the debug to 9 I receive logging messages, then via $kamctl debug 2 is everything fine. If the debug is set to 2 and I run the proxy in foreground I also receive log messages. Any way, I have tried to find out what is going wrong here and found something in : daemonize.c At the end of the daemonize function is a for loop int daemonize(int * own_pgid) . . /* close any open file descriptors 32 is the maximum number of inherited open file descriptors */ for (r=3; r < 32; r++){ close(r); } . . The variable "r" is used for the return value of fprintf (Upon successful return, these functions return the number of characters printed ..). At the moment I have no idea why this triggering this error and only for the DSYSLOG_ASYNC option. Furthermore why or for what is this for-loop good ? Maybe I am totally wrong or off the track but I really want this async logging. best regards Torben

15 years, 1 month

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev