sr-dev July 2009

sr-dev@lists.kamailio.org

35 participants
328 discussions

git:master: xmlrpc(s) doc: added the autoconversion parameter
by Andrei Pelinescu-Onciul 16 Jul '09

16 Jul '09

Module: sip-router Branch: master Commit: d6e8fdb3a06aab44c01ce5e5a2ca4f2d43117676 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d6e8fdb… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu Jul 16 10:38:14 2009 +0200 xmlrpc(s) doc: added the autoconversion parameter --- modules_s/xmlrpc/doc/params.xml | 29 ++++++++++++++++++++++++++++- 1 files changed, 28 insertions(+), 1 deletions(-) diff --git a/modules_s/xmlrpc/doc/params.xml b/modules_s/xmlrpc/doc/params.xml index 324495d..be7fd63 100644 --- a/modules_s/xmlrpc/doc/params.xml +++ b/modules_s/xmlrpc/doc/params.xml @@ -41,8 +41,35 @@ modparam("xmlrpc", "route", "route_for_xmlrpcs") </example> </section> + <section id="autoconversion"> + <title><varname>autoconversion</varname> (string)</title> + <para> + Enable or disable automatic parameter type conversion globally, + for all the methods parameters. + If on, a type mismatch in a method parameter + will not cause a fault if it is possible to automatically + convert it to the type expected by the method. + </para> + <para> + Default: off. + </para> + <para> + It is recommended to leave this parameter to its default off value + and fix instead the client application (which should use the + proper types) or to modify the target rpc to accept any type + (see the rpc scan '.' modifier). + </para> + <example> + <title>Set the <varname>autoconversion</varname> parameter</title> + <programlisting> +modparam("xmlrpc", "autoconversion", 1) + </programlisting> + </example> + </section> + <!-- - Seems to be obsolete (on always) -andrei + Obsolete (hardwired on in the rpc core functions, cannot be turned off) + -andrei <section id="enable_introspection"> <title><varname>enable_introspection</varname> (integer)</title> <para>

1 0

git:master: doc: rpc api documentation updated: the '.' modifier
by Andrei Pelinescu-Onciul 16 Jul '09

16 Jul '09

Module: sip-router Branch: master Commit: 46fb4917c2163fd5c8bd01fdd866813211cd6ec7 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=46fb491… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu Jul 16 09:36:22 2009 +0200 doc: rpc api documentation updated: the '.' modifier --- doc/rpc/ser_rpc.txt | 59 ++++++++++++++++++++++++++++++++++++++------------ doc/rpc/ser_rpc.xml | 60 +++++++++++++++++++++++++++++++++++++++++++++------ 2 files changed, 98 insertions(+), 21 deletions(-) diff --git a/doc/rpc/ser_rpc.txt b/doc/rpc/ser_rpc.txt index 778c03f..b2c5337 100644 --- a/doc/rpc/ser_rpc.txt +++ b/doc/rpc/ser_rpc.txt @@ -249,12 +249,13 @@ add("sd", string_param, int_param); ints and a string) are optional. Table 1. Data Type Overview -Name Formating String Char C-Style Variable -Integer d int -Float f double -String s char* -String S str -Optional modifier * marks all further parameters as optional + Name Formating String Char C-Style Variable + Integer d int + Float f double + String s char* + String S str + Optional modifier * marks all further parameters as optional + Autoconvert modifier . requires auto-conversion for the next parameter 1.2.3. Getting Parameters @@ -278,17 +279,27 @@ Note parameters to be retrieved. Each parameter is represented by exactly one parameter type character in the string. The variable part of parameters must contain as many pointers to C variables as there are - formatting non-modifiers characters in the formatting string. The - formatting string can also contain a "*" modifier that does not have a - correspondent in the variable part of the parameters. The meaning of - "*" is that any further parameters (defined by other type characters in - the formatting string) are optional (they can be missing in the input - and no rpc fault will automatically be generated). + formatting non-modifiers characters in the formatting string. Warning The function will crash if you fail to provide enough parameters. + Besides characters representing parameter types, the formatting string + can contain two special modifiers: "*" and ".". The modifiers do not + have a correspondent in the variable part of the parameters. + + The meaning of "*" modifier is that any further parameters (defined by + other type characters in the formatting string) are optional (they can + be missing in the input and no rpc fault will automatically be + generated). + + The '.' modifiers turns on type autoconversion for the next parameter. + This means that if the type of the next parameter differs from the type + specified in the formatting string, the parameter will be automatically + converted to the formatting string type (if possible) and if the + automatic conversion succeeds, no fault will be generated. + The function returns the number of parameters read on success (a number greater or equal 0) and - (minus) the number of parameters read on error (for example for an error after reading 2 parameters it will @@ -297,8 +308,9 @@ Warning number (- number of parameters read so far) and it will also automatically change the reply that will be sent to the caller to indicate that a failure has occurred on the server (unless the "*" is - used and the error is lack of more parameters). The prototype of the - function is: + used and the error is lack of more parameters). + + The prototype of the function is: int scan(char* fmt, ...) It is possible to either call the function once to scan all the @@ -368,6 +380,25 @@ static void rpc_delete_contact(rpc_t* rpc) /* Process retrieved parameters here */ } +/* variable number of parameters: + echo back all the parameters, string type required */ +static void core_prints(rpc_t* rpc, void* c) +{ + char* string = 0; + while((rpc->scan(c, "*s", &string)>0)) + rpc->add(c, "s", string); +} + +/* variable number of parameters and auto conversion: + echo back all the parameters, works with any type (everything is + internally converted to string, notice the '.' modifier) */ +static void core_echo(rpc_t* rpc, void* c) +{ + char* string = 0; + while((rpc->scan(c, "*.s", &string)>0)) + rpc->add(c, "s", string); +} + 1.2.4. Building Reply The RPC API contains several functions that can be used to modify diff --git a/doc/rpc/ser_rpc.xml b/doc/rpc/ser_rpc.xml index 5a20d8a..698bc9c 100644 --- a/doc/rpc/ser_rpc.xml +++ b/doc/rpc/ser_rpc.xml @@ -363,6 +363,11 @@ add("sd", string_param, int_param); <entry>*</entry> <entry>marks all further parameters as optional</entry> </row> + <row> + <entry>Autoconvert modifier</entry> + <entry>.</entry> + <entry>requires auto-conversion for the next parameter</entry> + </row> </tbody> </tgroup> </table> @@ -398,18 +403,34 @@ add("sd", string_param, int_param); The variable part of parameters must contain as many pointers to C variables as there are formatting non-modifiers characters in the formatting string. - The formatting string can also contain a "*" modifier that does not - have a correspondent in the variable part of the parameters. The - meaning of "*" is that any further parameters (defined by other - type characters in the formatting string) are optional (they - can be missing in the input and no rpc fault will automatically - be generated). <warning> <para> The function will crash if you fail to provide enough parameters. </para> </warning> + </para> + <para> + Besides characters representing parameter types, the formatting + string can contain two special modifiers: "*" and ".". The + modifiers do not have a correspondent in the variable part of the + parameters. + </para> + <para> + The meaning of "*" modifier is that any further parameters + (defined by other type characters in the formatting string) are + optional (they can be missing in the input and no rpc fault will + automatically be generated). + </para> + <para> + The '.' modifiers turns on type autoconversion for the next + parameter. This means that if the type of the next parameter + differs from the type specified in the formatting string, the + parameter will be automatically converted to the formatting string + type (if possible) and if the automatic conversion succeeds, no + fault will be generated. + </para> + <para> The function returns the number of parameters read on success (a number greater or equal 0) and - (minus) the number of parameters read on error (for example for an error after @@ -420,7 +441,10 @@ add("sd", string_param, int_param); and it will also automatically change the reply that will be sent to the caller to indicate that a failure has occurred on the server (unless the "*" is used and the error is lack - of more parameters). The prototype of the function is: + of more parameters). + </para> + <para> + The prototype of the function is: <programlisting> int scan(char* fmt, ...) </programlisting> @@ -437,6 +461,8 @@ rpc->scan("d", &int_val); rpc->scan("f", &double_val); </programlisting> </para> + <para> + </para> </section> <section> <title><function>struct_scan</function></title> @@ -506,6 +532,26 @@ static void rpc_delete_contact(rpc_t* rpc) /* Process retrieved parameters here */ } + +/* variable number of parameters: + echo back all the parameters, string type required */ +static void core_prints(rpc_t* rpc, void* c) +{ + char* string = 0; + while((rpc->scan(c, "*s", &string)>0)) + rpc->add(c, "s", string); +} + +/* variable number of parameters and auto conversion: + echo back all the parameters, works with any type (everything is + internally converted to string, notice the '.' modifier) */ +static void core_echo(rpc_t* rpc, void* c) +{ + char* string = 0; + while((rpc->scan(c, "*.s", &string)>0)) + rpc->add(c, "s", string); +} + ]]> </programlisting> </example>

1 0

CVS:commitlog: sip_router/modules/iptrtpproxy/examples clean_rtpproxy_attrs
by Tomas Mandys 15 Jul '09

15 Jul '09

tma0 2009/07/16 01:02:17 CEST SER CVS Repository Modified files: modules/iptrtpproxy/examples clean_rtpproxy_attrs Log: - socket support Revision Changes Path 1.3 +17 -3 sip_router/modules/iptrtpproxy/examples/clean_rtpproxy_attrs http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/sip_router/modules/iptrtpprox…

1 0

mi set_gflag fails
by jh＠tutpro.com 15 Jul '09

15 Jul '09

next i tried 'mi set_gflag' and it failed like this: srctl> mi set_gflag 1024 error: 400 - error at parameter 2: expected string type but record doesn't match type did i do something wrong or is also this mi command buggy? -- juha

2 3

git:master: ctl(s): support for '.' and type autoconversion
by Andrei Pelinescu-Onciul 15 Jul '09

15 Jul '09

Module: sip-router Branch: master Commit: 07276eb9a1173b4e58d14aa82a7b81e5f2dca824 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=07276eb… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Jul 15 22:39:20 2009 +0200 ctl(s): support for '.' and type autoconversion - support for the new '.' rpc->scan modifier (when used it turns auto-conversion on for the next input parameter, e.g.: rpc->scan(".s", &str)) - new module "autoconversion" module parameter that enables auto-conversion globally - fixed minor parameter number bug in binrpc faults, when the '*' modifier was used in rpc->scan() - added binrpc context garbage collection supports (needed to track memory allocation used in type conversions) --- modules_s/ctl/binrpc_run.c | 189 +++++++++++++++++++++++++++++++++++++++++--- modules_s/ctl/ctl.c | 5 + 2 files changed, 181 insertions(+), 13 deletions(-) diff --git a/modules_s/ctl/binrpc_run.c b/modules_s/ctl/binrpc_run.c index 0fac504..aa50e8a 100644 --- a/modules_s/ctl/binrpc_run.c +++ b/modules_s/ctl/binrpc_run.c @@ -39,12 +39,20 @@ #include "io_listener.h" #include <stdio.h> /* vsnprintf */ +#include <stdlib.h> /* strtod */ #include <stdarg.h> +/* if set try to automatically convert values to the requested type in + rpc->scan (default: not set) */ +int autoconvert=0; + + #define BINRPC_MAX_BODY 4096 /* maximum body for send */ #define STRUCT_MAX_BODY 1024 #define MAX_MSG_CHUNKS 96 +#define BINRPC_GC_IBSIZE 4 /* initial gc block size (pointers no.) */ + struct rpc_struct_head{ struct rpc_struct_l* next; struct rpc_struct_l* prev; @@ -72,11 +80,21 @@ struct binrpc_recv_ctx{ int in_struct; }; + +struct binrpc_gc_block{ + unsigned short p_no; /**< array size */ + unsigned short idx; /**< current/last used pos. in the array */ + struct binrpc_gc_block* next; + void* p[1]; /**< array of pointers that will be free'd */ +}; + + struct binrpc_ctx{ struct binrpc_recv_ctx in; struct binrpc_send_ctx out; void* send_h; /* send handle */ char* method; + struct binrpc_gc_block* gc; /**< garbage collection */ int replied; }; @@ -113,6 +131,51 @@ static rpc_t binrpc_callbacks={ +/** mark a pointer for freeing when the ctx is destroyed. + * @return 0 on success, -1 on error + */ +inline static int binrpc_gc_track(struct binrpc_ctx* ctx, void* p) +{ + struct binrpc_gc_block* b; + int n; + + b=ctx->gc; + if (b==0 || (b->idx>=b->p_no)){ + n=(b==0)?BINRPC_GC_IBSIZE:b->p_no*2; + b=pkg_malloc(sizeof(*b)+n*sizeof(void*)-sizeof(b->p)); + if (b==0) + return -1; + b->p_no=n; + b->idx=0; + /* link in front */ + b->next=ctx->gc; + ctx->gc=b; + } + b->p[b->idx]=p; + b->idx++; + return 0; +} + + + +/** free all the tracked pointer from ctx->gc. + */ +inline static void binrpc_gc_collect(struct binrpc_ctx* ctx) +{ + struct binrpc_gc_block* b; + struct binrpc_gc_block* next; + int i; + + for(b=ctx->gc; b; b=next){ + next=b->next; + for (i=0; i<b->idx; i++) + pkg_free(b->p[i]); + pkg_free(b); + } + ctx->gc=0; +} + + static struct rpc_struct_l* new_rpc_struct() { struct rpc_struct_l* rs; @@ -322,6 +385,7 @@ inline void destroy_binrpc_ctx(struct binrpc_ctx* ctx) pkg_free(ctx->out.pkt.body); ctx->out.pkt.body=0; } + binrpc_gc_collect(ctx); } @@ -583,6 +647,88 @@ static char* rpc_type_name(int type) +/** converts a binrpc_val to int. + *@return int val on success, 0 and sets err on error (E_BINRPC_TYPE) */ +inline static int binrpc_val_conv_int( struct binrpc_val* v, int* err) +{ + int ret; + + *err=0; + switch(v->type){ + case BINRPC_T_INT: + return v->u.intval; + case BINRPC_T_DOUBLE: + return (int) v->u.fval; + case BINRPC_T_STR: + if (str2sint(&v->u.strval, &ret)==0) + return ret; + } + *err=E_BINRPC_TYPE; + return 0; +} + + + +/** converts a binrpc_val to double. + *@return double val on success, 0 and sets err on error (E_BINRPC_TYPE) */ +inline static double binrpc_val_conv_double( struct binrpc_val* v, int* err) +{ + double ret; + char* end; + + *err=0; + switch(v->type){ + case BINRPC_T_DOUBLE: + return v->u.fval; + case BINRPC_T_INT: + return (double)v->u.intval; + case BINRPC_T_STR: + ret=strtod(v->u.strval.s, &end); + if (end!=v->u.strval.s) + return ret; + } + *err=E_BINRPC_TYPE; + return 0; +} + + + +/** converts a binrpc_val to str. + *@return str val pointer on success, 0 and sets err on error (E_BINRPC_TYPE)*/ +inline static str* binrpc_val_conv_str(struct binrpc_ctx* ctx, + struct binrpc_val* v, int* err) +{ + str* ret; + char* s; + int len; + + *err=0; + switch(v->type){ + case BINRPC_T_STR: + return &v->u.strval; + case BINRPC_T_INT: + s=int2str(v->u.intval, &len); + ret=pkg_malloc(sizeof(*ret)+len+1); + if (ret==0 || binrpc_gc_track(ctx, ret)!=0){ + *err=E_BINRPC_OVERFLOW; + return 0; + } + ret->s=(char*)ret+sizeof(*ret); + ret->len=len; + memcpy(ret->s, s, len); + ret->s[len]=0; + return ret; + case BINRPC_T_DOUBLE: + /* for now the double to string conversion is not supported*/ + *err=E_BINRPC_BUG; + return 0; + } + *err=E_BINRPC_TYPE; + return 0; +} + + + /* rpc interface functions */ /* returns the number of parameters read @@ -595,45 +741,61 @@ static int rpc_scan(struct binrpc_ctx* ctx, char* fmt, ...) char* orig_fmt; int nofault; int modifiers; + int autoconv; + int i; + double d; + str* s; va_start(ap, fmt); orig_fmt=fmt; nofault = 0; modifiers=0; + autoconv=autoconvert; for (;*fmt; fmt++){ switch(*fmt){ case '*': /* start of optional parameters */ nofault = 1; modifiers++; - break; + continue; + case '.': /* autoconv. on for the next parameter */ + modifiers++; + autoconv=1; + continue; case 'b': /* bool */ case 't': /* time */ case 'd': /* int */ - v.type=BINRPC_T_INT; + v.type=autoconv?BINRPC_T_ALL:BINRPC_T_INT; ctx->in.s=binrpc_read_record(&ctx->in.ctx, ctx->in.s, ctx->in.end, &v, &err); - if (err<0) goto error_read; - *(va_arg(ap, int*))=v.u.intval; + if (err<0 || ((i=binrpc_val_conv_int(&v, &err))==0 && err<0)) + goto error_read; + *(va_arg(ap, int*))=i; break; case 'f': - v.type=BINRPC_T_DOUBLE; + v.type=autoconv?BINRPC_T_ALL:BINRPC_T_DOUBLE; ctx->in.s=binrpc_read_record(&ctx->in.ctx, ctx->in.s, ctx->in.end, &v, &err); - if (err<0) goto error_read; - *(va_arg(ap, double*))=v.u.fval; + if (err<0 || ((d=binrpc_val_conv_double(&v, &err))==0 && + err<0)) + goto error_read; + *(va_arg(ap, double*))=d; break; case 's': /* asciiz */ case 'S': /* str */ - v.type=BINRPC_T_STR; - v.u.strval.s="if you get this string, you don't" - "check rpc_scan return code !!! (very bad)"; - v.u.strval.len=strlen(v.u.strval.s); + v.type=autoconv?BINRPC_T_ALL:BINRPC_T_STR; ctx->in.s=binrpc_read_record(&ctx->in.ctx, ctx->in.s, ctx->in.end, &v,&err); + if (err<0 || ((s=binrpc_val_conv_str(ctx, &v, &err))==0 && + err<0)){ + v.u.strval.s="if you get this string, you don't" + "check rpc_scan return code !!! (very bad)"; + v.u.strval.len=strlen(v.u.strval.s); + s=&v.u.strval; + } if (*fmt=='s'){ - *(va_arg(ap, char**))=v.u.strval.s; /* 0 term by proto*/ + *(va_arg(ap, char**))=s->s; /* 0 term by proto*/ }else{ - *(va_arg(ap, str*))=v.u.strval; + *(va_arg(ap, str*))=*s; } if (err<0) goto error_read; break; @@ -652,6 +814,7 @@ static int rpc_scan(struct binrpc_ctx* ctx, char* fmt, ...) default: goto error_inv_param; } + autoconv=autoconvert; /* reset autoconv*/ ctx->in.record_no++; } va_end(ap); diff --git a/modules_s/ctl/ctl.c b/modules_s/ctl/ctl.c index 071c2e2..730de47 100644 --- a/modules_s/ctl/ctl.c +++ b/modules_s/ctl/ctl.c @@ -66,6 +66,10 @@ static int usock_mode=0600; /* permissions, default rw-------*/ static int usock_uid=-1; /* username and group for the unix sockets*/ static int usock_gid=-1; +/* if set try to automatically convert values to the requested type in + rpc->scan (default: not set) */ +extern int autoconvert; + static int add_binrpc_socket(modparam_t type, void * val); #ifdef USE_FIFO static int add_fifo_socket(modparam_t type, void * val); @@ -98,6 +102,7 @@ static param_export_t params[]={ {"mode", PARAM_INT, &usock_mode }, {"user", PARAM_STRING|PARAM_USE_FUNC, fix_user }, {"group", PARAM_STRING|PARAM_USE_FUNC, fix_group }, + {"autoconversion", PARAM_INT, &autoconvert }, {0,0,0} }; /* no params */

1 0

git:master: core: new debug core rpcs
by Andrei Pelinescu-Onciul 15 Jul '09

15 Jul '09

Module: sip-router Branch: master Commit: aafc8ea72a36be3ac9bd6115cd492a68e4166e85 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=aafc8ea… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Jul 15 22:46:07 2009 +0200 core: new debug core rpcs - added core.printi that expects integer parameters and prints them back in the reply (works with variable numbers of params.) - added core.echo that echoes back its parameters (works with variable number of parameters and with any type, it auto-converts all of them to string). - changed core.prints to work with variable number of parameters. --- core_cmd.c | 41 +++++++++++++++++++++++++++++++++++++---- 1 files changed, 37 insertions(+), 4 deletions(-) diff --git a/core_cmd.c b/core_cmd.c index f2edc3d..bc2bd13 100644 --- a/core_cmd.c +++ b/core_cmd.c @@ -279,15 +279,43 @@ static void system_methodHelp(rpc_t* rpc, void* c) static const char* core_prints_doc[] = { - "Returns the string given as parameter.", /* Documentation string */ - 0 /* Method signature(s) */ + "Returns the strings given as parameters.", /* Documentation string */ + 0 /* Method signature(s) */ }; static void core_prints(rpc_t* rpc, void* c) { char* string = 0; - if (rpc->scan(c, "s", &string)>0) + while((rpc->scan(c, "*s", &string)>0)) + rpc->add(c, "s", string); +} + + +static const char* core_printi_doc[] = { + "Returns the integers given as parameters.", /* Documentation string */ + 0 /* Method signature(s) */ +}; + + +static void core_printi(rpc_t* rpc, void* c) +{ + int i; + while((rpc->scan(c, "*d", &i)>0)) + rpc->add(c, "d", i); +} + + +static const char* core_echo_doc[] = { + "Returns back its parameters.", /* Documentation string */ + 0 /* Method signature(s) */ +}; + + +static void core_echo(rpc_t* rpc, void* c) +{ + char* string = 0; + while((rpc->scan(c, "*.s", &string)>0)) rpc->add(c, "s", string); } @@ -699,7 +727,12 @@ static rpc_export_t core_rpc_methods[] = { {"system.listMethods", system_listMethods, system_listMethods_doc, RET_ARRAY}, {"system.methodSignature", system_methodSignature, system_methodSignature_doc, 0 }, {"system.methodHelp", system_methodHelp, system_methodHelp_doc, 0 }, - {"core.prints", core_prints, core_prints_doc, 0 }, + {"core.prints", core_prints, core_prints_doc, + RET_ARRAY}, + {"core.printi", core_printi, core_printi_doc, + RET_ARRAY}, + {"core.echo", core_echo, core_echo_doc, + RET_ARRAY}, {"core.version", core_version, core_version_doc, 0 }, {"core.uptime", core_uptime, core_uptime_doc, 0 }, {"core.ps", core_ps, core_ps_doc, RET_ARRAY},

1 0

git:master: mi_rpc: use autoconversion to string
by Andrei Pelinescu-Onciul 15 Jul '09

15 Jul '09

Module: sip-router Branch: master Commit: 47cca1121e45d6c99375cfe80b63cbf634e05bd0 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=47cca11… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Jul 15 22:55:28 2009 +0200 mi_rpc: use autoconversion to string Because mi doesn't understand anything else besides strings, use the new auto-convert modifier when reading the mi commands parameters. --- modules/mi_rpc/mi_rpc_mod.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/mi_rpc/mi_rpc_mod.c b/modules/mi_rpc/mi_rpc_mod.c index 007c575..9c68839 100644 --- a/modules/mi_rpc/mi_rpc_mod.c +++ b/modules/mi_rpc/mi_rpc_mod.c @@ -89,7 +89,7 @@ struct mi_root *mi_rpc_read_params(rpc_t *rpc, void *ctx) } node = &root->node; - while (rpc->scan(ctx, "*S", &value) == 1) + while (rpc->scan(ctx, "*.S", &value) == 1) { name.s = 0; name.len = 0; @@ -104,7 +104,7 @@ struct mi_root *mi_rpc_read_params(rpc_t *rpc, void *ctx) } /* value */ - if(rpc->scan(ctx, "*S", &value) != 1) + if(rpc->scan(ctx, "*.S", &value) != 1) { LM_ERR("value expected\n"); goto error;

1 0

git:master: core: rpc more verbose when duplicates found
by Andrei Pelinescu-Onciul 15 Jul '09

15 Jul '09

Module: sip-router Branch: master Commit: 1f7b7e075d35d0ca00fb943795f74af99fce06a1 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1f7b7e0… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Jul 15 22:49:23 2009 +0200 core: rpc more verbose when duplicates found When duplicate rpc names are detected, print them (WARN(..)). --- rpc_lookup.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/rpc_lookup.c b/rpc_lookup.c index c1e1a6d..2bc286f 100644 --- a/rpc_lookup.c +++ b/rpc_lookup.c @@ -27,6 +27,7 @@ #include "rpc.h" #include "str_hash.h" #include "ut.h" +#include "dprint.h" #define RPC_HASH_SIZE 32 #define RPC_SARRAY_SIZE 32 /* initial size */ @@ -186,8 +187,10 @@ int rpc_register(rpc_export_t* rpc) { /* check if the entry is already registered */ - if (rpc_lookup(rpc->name, strlen(rpc->name))) + if (rpc_lookup(rpc->name, strlen(rpc->name))){ + WARN("duplicate rpc \"%s\"\n", rpc->name); return 1; + } if (rpc_hash_add(rpc)!=0) return -1; return 0; }

1 0

git:master: xmlrpc(s): type autoconversion support
by Andrei Pelinescu-Onciul 15 Jul '09

15 Jul '09

Module: sip-router Branch: master Commit: 6f38c000cc3b5c76c99904fa7d5ab244aaa774f4 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6f38c00… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Jul 15 20:32:53 2009 +0200 xmlrpc(s): type autoconversion support - added a new module parameter ("autoconversion") that when enabled will try to autoconvert the xmlrpc parameters types into what the rpc method expects (e.g. if the method expects a string, but receives an int, convert the int to string). By default it's off. - support for a new modifier: '.' for rpc_scan. When used it turns auto-conversion on for the next input. E.g. rpc->scan(".s", &string) will work even if the next input is an int (it will convert it to string). More complex example: while((rpc->scan(c, "*.s", &p)>0)) rpc->add(c, "s", p); (work with any number of inputs/parameters, convert all of them to string and echo them back in the reply) --- modules_s/xmlrpc/xmlrpc.c | 304 ++++++++++++++++++++++++++++++++++++++------- 1 files changed, 256 insertions(+), 48 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=6f3…

1 0

git:master: fix error in cr dump command, reported from Edson 4lists at gmail dot com
by Henning Westerholt 15 Jul '09

15 Jul '09

Module: sip-router Branch: master Commit: 7a1676e337f40c286735060394963436854e999e URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=7a1676e… Author: Henning Westerholt <henning.westerholt(a)1und1.de> Committer: Henning Westerholt <henning.westerholt(a)1und1.de> Date: Wed Jul 15 15:06:25 2009 +0200 fix error in cr dump command, reported from Edson 4lists at gmail dot com --- tools/kamctl | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/kamctl b/tools/kamctl index 0835b8c..cbbe72b 100755 --- a/tools/kamctl +++ b/tools/kamctl @@ -1264,7 +1264,7 @@ cr() { ;; dump) - $CTLCMD cr_dump + $CTLCMD cr_dump_routes ;; addcn)

1 0

← Newer
1
...
9
10
11
12
13
14
15
...
33
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev July 2009