sr-dev

sr-dev@lists.kamailio.org

12 participants
44199 discussions

[ openser-Bugs-2791122 ] fr_inv_timer_avp on amd64

by SourceForge.net

Bugs item #2791122, was opened at 2009-05-13 14:07 Message generated for change (Comment added) made by miconda You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2791122&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Tobias Lindgren (effeks) Assigned to: Nobody/Anonymous (nobody) Summary: fr_inv_timer_avp on amd64 Initial Comment: Hi! When we compile kamailio 1.5.1 trunk on Debian Linux amd64 the fr_inv_time_avp timer cannot altered (the value of the AVP-variable has no effect on the timer) and seem stuck on something like 15 seconds. Compiling the same code in a 32bit chroot works fine, aswell as on any other standard 32bit system, where the timer is set to the value of the AVP as expected. /effeks ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2009-06-10 14:23 Message: I did several tests and the timeout was ok. Here are my parameters: modparam("tm", "fr_timer", 15) modparam("tm", "fr_inv_timer", 60) modparam("tm", "wt_timer", 5) modparam("tm", "fr_inv_timer_avp", "$avp(i:120)") $avp(i:120) = 30; Can you get a ngrep trace and send it to me at: <miconda [at] gmail.com>? Thanks. ---------------------------------------------------------------------- Comment By: Tobias Lindgren (effeks) Date: 2009-06-05 14:23 Message: Any news? ---------------------------------------------------------------------- Comment By: Tobias Lindgren (effeks) Date: 2009-05-25 14:50 Message: The avp-value was a string, but changing it into an integer makes no difference. On 32bit it works regardless of int or str. ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2009-05-25 12:49 Message: Did you check my last message? Is the timer avp integer or string value? ---------------------------------------------------------------------- Comment By: Nobody/Anonymous (nobody) Date: 2009-05-25 12:47 Message: Any progress on this? ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2009-05-22 20:12 Message: Just saw in your last comment that you assign a string to the avp: $avp(i:10) = "25"; Should be: $avp(i:10) = 25; Can you doublecheck? ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2009-05-19 10:25 Message: OK, i am going to check the log. thanks. ---------------------------------------------------------------------- Comment By: Tobias Lindgren (effeks) Date: 2009-05-15 14:23 Message: Done, the config snippet looks something like this: modparam("tm", "fr_inv_timer_avp", "$avp(i:10)") $avp(i:10) = "25"; t_relay(); ---------------------------------------------------------------------- Comment By: Daniel-Constantin Mierla (miconda) Date: 2009-05-15 13:07 Message: Can you run with debug=5 and attach the syslog output here? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2791122&group_…

15 years, 7 months

m_dump un-register check fails

by jh＠tutpro.com

while trying find out, why sip-router produces mysql "out of sync" errors, i discovered that msilo m_dump only recognizes that register request is un-register request, if it has expires header with body 0. it is not checking if contact header has expires=0 parameter like this: Contact: <sip:jh_test_fi@192.98.101.10:5074;transport=udp>;expires=0 -- juha

15 years, 7 months

git:master: xcap_client: proper return code in child init function when all is ok

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 2e51695a7e26ad0e2dabd6dc44787b41862164fc URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=2e51695… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jun 9 18:34:35 2009 +0300 xcap_client: proper return code in child init function when all is ok - introduced by previous commit, reported by Pascal Maugeri --- modules_k/xcap_client/xcap_client.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules_k/xcap_client/xcap_client.c b/modules_k/xcap_client/xcap_client.c index 55084d8..dab7ef3 100644 --- a/modules_k/xcap_client/xcap_client.c +++ b/modules_k/xcap_client/xcap_client.c @@ -174,7 +174,7 @@ static int child_init(int rank) LM_ERR("cannot connect to db\n"); return -1; } - return -1; + return 0; } void destroy(void)

15 years, 7 months

SF.net SVN: openser:[5872] branches/1.5/modules/xcap_client/xcap_client.c

by Daniel-Constantin Mierla

Revision: 5872 http://openser.svn.sourceforge.net/openser/?rev=5872&view=rev Author: miconda Date: 2009-06-09 15:26:03 +0000 (Tue, 09 Jun 2009) Log Message: ----------- - proper return code in child init function when all is ok - reported by Pascal Maugeri Modified Paths: -------------- branches/1.5/modules/xcap_client/xcap_client.c This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

15 years, 7 months

git:master: * Closed database connection after two error case. Thanks to Jan for

by Juha Heinanen

Module: sip-router Branch: master Commit: f3e18e3885eff64d5808ede28a12210a4ad4ae19 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f3e18e3… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Tue Jun 9 15:54:42 2009 +0300 * Closed database connection after two error case. Thanks to Jan for pointing out the bug. --- modules/lcr/lcr_mod.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/modules/lcr/lcr_mod.c b/modules/lcr/lcr_mod.c index 52e2cf5..6743a21 100644 --- a/modules/lcr/lcr_mod.c +++ b/modules/lcr/lcr_mod.c @@ -1052,6 +1052,7 @@ int reload_gws_and_lcrs(void) } if (lcr_dbf.use_table(dbh, &gw_table) < 0) { + lcr_dbf.close(dbh); LM_ERR("error while trying to use gw table\n"); return -1; } @@ -1273,6 +1274,7 @@ int reload_gws_and_lcrs(void) } if (lcr_dbf.use_table(dbh, &lcr_table) < 0) { + lcr_dbf.close(dbh); LM_ERR("error while trying to use lcr table\n"); return -1; }

15 years, 7 months

git:master: xcap_client: init db con per process

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 7e2bf0f1c1a7a84e2b4fb33f2175d7bc0d057c10 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=7e2bf0f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jun 9 14:47:15 2009 +0300 xcap_client: init db con per process - introduced child_init function to init db connection per process - db connection closed in destroy function - reported and credits to Pascal Maugeri and Inaki Baz Castillo --- modules_k/presence/subscribe.c | 2 +- modules_k/xcap_client/xcap_client.c | 20 ++++++++++++++++++-- 2 files changed, 19 insertions(+), 3 deletions(-) diff --git a/modules_k/presence/subscribe.c b/modules_k/presence/subscribe.c index f05300a..3067919 100644 --- a/modules_k/presence/subscribe.c +++ b/modules_k/presence/subscribe.c @@ -1610,7 +1610,7 @@ void update_db_subs(db1_con_t *db,db_func_t dbf, shtable_t hash_table, lock_release(&hash_table[i].lock); } - update_vals[0].val.int_val= (int)time(NULL)- 10; + update_vals[0].val.int_val= (int)time(NULL) - expires_offset; update_ops[0]= OP_LT; if(dbf.delete(db, update_cols, update_ops, update_vals, 1) < 0) { diff --git a/modules_k/xcap_client/xcap_client.c b/modules_k/xcap_client/xcap_client.c index 8fde618..55084d8 100644 --- a/modules_k/xcap_client/xcap_client.c +++ b/modules_k/xcap_client/xcap_client.c @@ -53,6 +53,7 @@ MODULE_VERSION #define XCAP_TABLE_VERSION 3 static int mod_init(void); +static int child_init(int rank); void destroy(void); struct mi_root* refreshXcapDoc(struct mi_root* cmd, void* param); int get_auid_flag(str auid); @@ -112,7 +113,7 @@ struct module_exports exports= { mod_init, /* module initialization function */ 0, /* response handling function */ (destroy_function) destroy, /* destroy function */ - 0 /* per-child init function */ + child_init /* per-child init function */ }; /** @@ -149,10 +150,13 @@ static int mod_init(void) return -1; } - if(db_check_table_version(&xcap_dbf, xcap_db, &xcap_db_table, XCAP_TABLE_VERSION) < 0) { + if(db_check_table_version(&xcap_dbf, xcap_db, &xcap_db_table, + XCAP_TABLE_VERSION) < 0) { LM_ERR("error during table version check.\n"); return -1; } + xcap_dbf.close(xcap_db); + xcap_db = NULL; curl_global_init(CURL_GLOBAL_ALL); @@ -163,9 +167,21 @@ static int mod_init(void) return 0; } +static int child_init(int rank) +{ + if((xcap_db = xcap_dbf.init(&xcap_db_url))==NULL) + { + LM_ERR("cannot connect to db\n"); + return -1; + } + return -1; +} + void destroy(void) { curl_global_cleanup(); + if(xcap_db != NULL) + xcap_dbf.close(xcap_db); } void query_xcap_update(unsigned int ticks, void* param)

15 years, 7 months

SF.net SVN: openser:[5871] branches/1.5/modules

by Daniel-Constantin Mierla

Revision: 5871 http://openser.svn.sourceforge.net/openser/?rev=5871&view=rev Author: miconda Date: 2009-06-09 11:37:46 +0000 (Tue, 09 Jun 2009) Log Message: ----------- - introduced child_init function to init db connection per process - db connection closed in destroy function - reported and credits to Pascal Maugeri and Inaki Baz Castillo Modified Paths: -------------- branches/1.5/modules/presence/subscribe.c branches/1.5/modules/xcap_client/xcap_client.c This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

15 years, 7 months

driver error on query: Commands out of sync

by jh＠tutpro.com

it turned out the hang up problems (now fixed) were separate from mysql "driver error on query: Commands out of sync" problems. here is one example on that error from process 609 right after sr restart: Jun 6 18:22:41 localhost /usr/sbin/sip-router[609]: INFO: db_mysql [km_dbase.c:97]: submit_query(): select username,contact,expires,q,callid,cseq,flags,cflags,user_agent,received,path,socket,methods,last_modified,domain from location Jun 6 18:22:44 localhost /usr/sbin/sip-router[609]: INFO: db_mysql [km_dbase.c:97]: submit_query(): insert into location (username,contact,expires,q,callid,cseq,flags,cflags,user_agent,received,path,socket,methods,last_modified,domain ) values ('test','sip:test@192.98.101.10:5074;transport=udp','2009-06-06 19:22:44',0.50 ,'ikquzsrapccwsci@localhost',219,0,0,'Twinkle/1.4.2',NULL,NULL,'',6111,'2009-06-06 18:22:44','test.fi') Jun 6 18:22:44 localhost /usr/sbin/sip-router[609]: INFO: db_mysql [km_dbase.c:97]: submit_query(): select id,src_addr,dst_addr,body,ctype,inc_time from silo where username='test' AND domain='test.fi' AND snd_time=0 order by id Jun 6 18:22:44 localhost /usr/sbin/sip-router[609]: ERROR: db_mysql [km_dbase.c:118]: driver error on query: Commands out of sync; you can't run this command now Jun 6 18:22:44 localhost /usr/sbin/sip-router[609]: ERROR: <core> [db_query.c:101]: error while submitting query Jun 6 18:22:44 localhost /usr/sbin/sip-router[609]: ERROR: msilo [msilo.c:1007]: failed to query database as you see, there is first select on location, then insert on location (save) and finally select on msilo (m_dump) that causes the error. any idea how it is possible that insert (which does not return result) could cause following select to return this kind of error? i have not experienced these kind of errors in k with the same config and modules. could it be that there is a bug in srdb1 lib? -- juha

15 years, 7 months

git:master: core: fixed pv_get_spec_name()

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: a93f14f36e007741ba2c531ae4d15170a449deca URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a93f14f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jun 9 11:59:53 2009 +0300 core: fixed pv_get_spec_name() - check for PV_NAME_PVAR was wrongly used with PV_NAME_INTSTR - reported and credits to Christian Koch --- pvapi.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/pvapi.c b/pvapi.c index 87de64a..72fb48d 100644 --- a/pvapi.c +++ b/pvapi.c @@ -800,7 +800,7 @@ int pv_get_spec_name(struct sip_msg* msg, pv_param_p ip, pv_value_t *name) name->flags = PV_VAL_INT|PV_TYPE_INT; } return 0; - } else if(ip->pvn.type==PV_NAME_INTSTR) { + } else if(ip->pvn.type==PV_NAME_PVAR) { /* pvar */ if(pv_get_spec_value(msg, (pv_spec_p)(ip->pvn.u.dname), name)!=0) {

15 years, 7 months

git:master: htable: replace PV_NAME_PVAR with PV_NAME_OTHER

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 33e4b20ffeb6a26f49080f701dc03e35498ac221 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=33e4b20… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jun 9 12:02:22 2009 +0300 htable: replace PV_NAME_PVAR with PV_NAME_OTHER - forgotten to be updated when PV_NAME_OTHER was introduced to get rid of PVAR type hack - name of $sht is a custom structure, not a PV - reported by Christian Koch --- modules_k/htable/ht_var.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules_k/htable/ht_var.c b/modules_k/htable/ht_var.c index 3f7ecba..4cc0808 100644 --- a/modules_k/htable/ht_var.c +++ b/modules_k/htable/ht_var.c @@ -163,7 +163,7 @@ int pv_parse_ht_name(pv_spec_p sp, str *in) } hpv->ht = ht_get_table(&hpv->htname); sp->pvp.pvn.u.dname = (void*)hpv; - sp->pvp.pvn.type = PV_NAME_PVAR; + sp->pvp.pvn.type = PV_NAME_OTHER; return 0; error:

15 years, 7 months

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev