May 2010 - sr-dev - kamailio.org

git:sr_3.0: core: fix bogus gcc-4.4 warning
by Andrei Pelinescu-Onciul 28 May '10

28 May '10

Module: sip-router Branch: sr_3.0 Commit: 720e50dea0528e55501fe41661941e4b40c903f9 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=720e50d… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Tue Feb 2 17:40:22 2010 +0100 core: fix bogus gcc-4.4 warning (cherry picked from commit 3a99415a7df46e2fd63698ee407c5075aac01846) --- ip_addr.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/ip_addr.h b/ip_addr.h index 2983e46..4f027ec 100644 --- a/ip_addr.h +++ b/ip_addr.h @@ -696,7 +696,7 @@ static inline char* su2a(union sockaddr_union* su, int su_len) if (unlikely(su_len<sizeof(su->sin6))) return "<addr. error>"; buf[0]='['; - offs=1+ip6tosbuf((unsigned char*)&su->sin6.sin6_addr, &buf[1], + offs=1+ip6tosbuf((unsigned char*)su->sin6.sin6_addr.s6_addr, &buf[1], sizeof(buf)-4); buf[offs]=']'; offs++;

1 0

git:refs/tags/sr_3.0.2: Tag sr_3.0.2: Mostly bugfixes & docs.
by Andrei Pelinescu-Onciul 28 May '10

28 May '10

Module: sip-router Branch: refs/tags/sr_3.0.2 Tag: 3a32408a6fab1c78a1cd14cbd78a5443c69cf2dc URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=tag;h=3a32408a6f… Tagger: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri May 28 21:14:06 2010 +0200 Mostly bugfixes & docs. Roughly synchronized with kamailio 3.0.2 (except packaging, xavps and the warning headers default).

1 0

New commits on branch sr_3.0
by Andrei Pelinescu-Onciul 28 May '10

28 May '10

URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=7c77719… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri May 28 21:11:50 2010 +0200 make: version number set to 3.0.2 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=facfe79… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu May 27 19:42:16 2010 +0200 tls: fixed usage of error label - introduced in previous commit (cherry picked from commit 81280e02fe83b534b37de1c156e272b9f7d64f99) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e0baf38… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed May 26 15:48:59 2010 +0200 tls: fix unregistered rpc commands In the process of merging the tls modules with some k tls PVs the module interface was changed, leaving the rpc functions un-registered (and hence unaccessible). (cherry picked from commit 577ccb71e7a1e113cacd74b19c8174ea2e9bacb6) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6964290… Merge: 63d59b6feac84ee0238411cf97e00fb796bb5c91 1c588c85a086aff42da5aa2a7da5ad663c896de1 Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri May 28 20:50:40 2010 +0200 Merge remote branch 'origin/andrei/pointer_alias_warnings' into sr_3.0 * origin/andrei/pointer_alias_warnings: rr(s): fix recently introduced avp_cookie name bug core: avp aliasing warning fixes group: fix pointer aliasing warnings rr: pointer aliasing warning fixes db_postgres: pointer aliasing warnings fixes core: pointer aliasing warnings fixed cfg: fixed pointer aliasing warnings Conflicts: route.c URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=63d59b6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Apr 7 22:56:46 2010 +0200 topoh: free nbuf - fixed leak due to no free of new buffer - reported by Libor Chocholaty (cherry picked from commit 198e468b7d83bb62d0761dd45e5c6820bc75a61a) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e23a006… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Apr 20 12:24:29 2010 +0200 perl: update the index of params in action struct - params start from 3rd position of val array (cherry picked from commit f624074a1680befed9328548864407c396ec196f) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b11596a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Apr 21 15:04:55 2010 +0200 dialplan: safety check for fetch rows before data init (cherry picked from commit 5040ff5cc54f673af80190916de9c1ac5d4c1af9) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=4563e73… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu May 13 11:41:06 2010 +0200 registrar(k): reset dst uri in lookup - upon lookup, dst uri is reset if no new dst uri is set from received or path, otherwise the coordinates of selected contact are not effective - reported by Andreas Granig (cherry picked from commit 53bb79cef5eb70e98472903ef84b5585cc798e65) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9259333… Author: Henning Westerholt <henning.westerholt(a)1und1.de> Date: Thu May 20 15:46:58 2010 +0200 fix memset call, set the variable to zero, not to the size with length 0.. (cherry picked from commit 355636358c07b8da9b1fcd6d04cdaf337c9f730c) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=c016360… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Tue May 4 13:10:03 2010 +0300 Modules:carrierroute: Fixed a bug in carrirroute module that might cause a crash if the routing prob was larger than the number of available routes. The problem was caused by a missing else (cherry picked from commit 16bdf9ea21b310d49d5d99dee43cfd57990eb1bb) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=df28756… Author: Ovidiu Sas <osas(a)voipembedded.com> Date: Fri May 14 15:05:17 2010 -0400 sdp parser: fix sdp payload attributes cloning - ptime attribute was not computed while allocating space for the cloned attributes (cherry picked from commit f1944ead66d854c349fca67da2b13d2be3c7fc33) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=114b282… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu May 6 21:09:47 2010 +0200 avposp: use state in searching first avp in avp_copy - state parameter was missing when searching first avp, keeping it uninitialized when searching the next, causing a crash when flag 'g' was used in avp_copy - reported by Jon Bonilla (Manwe) (cherry picked from commit 027d68e9cc7afcdb3ba8922215a28e33a5225097) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9e8d83e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu May 6 18:51:45 2010 +0200 dispatcher(k): allow functions from route block - they do not depend on sip message itself, but script variables (cherry picked from commit 25680d4bc14a4502e118951873c6da8edfc7e9a7) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a7f0bbc… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Mar 19 14:47:46 2010 +0100 core: cfg parser: allow negative numbers in modparams (cherry picked from commit d4b2eee2e4f0e5a1fe4b5ea17d78d51a6fab70c7) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3c22960… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Tue May 4 17:06:47 2010 +0300 core:sr_module.c increased number of pre-allocated static buffers in get_str_fparam() Increased the number from 3 to 6 as more are needed by modules like carrierroute when avp's are used as function parameters (Bug reported by Ole Kaas) (cherry picked from commit 4fd1e775f701d79d1d7f5e75bd7ff621739914b5) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=543eff1… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Tue Apr 27 15:40:10 2010 +0300 lib/srdb1 The RES_ROW(res) was not checked to be NULL before being used This caused a crash in the case that RES_ROW_N(res) was > 0, and RES_ROW was NULL (like a pkg_malloc going bad in a OOM scenario). (cherry picked from commit 79352b61841907f08bf7ca994acf814b2a5a6615) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=33d5439… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Apr 23 16:44:49 2010 +0200 core - tm: improved log messages - udp_send() prints outgoing buffer len as unsigned - build_uac_req() prints the size of attempted memory chunk allocation (cherry picked from commit 839c9509013abf77a446e1c35620a9fcd4223740) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=45f276a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Apr 22 10:35:33 2010 +0200 dialog(k): populate bind addr before any error - setting bind addr attribute in populate_leg_info() takes place before any parsing error may occur, to be sure it is set, avoiding extra check for null value when updating db - reported by Kelvin Chua - credits to Timo Reimann for troubleshooting (cherry picked from commit fd68980dc6f3de03a432f478c124852c23a1d1b2) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1b53e45… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Apr 20 22:06:46 2010 +0200 tm: kamailio flavour enables local-request event - will execute event_route[tm:local-request] - reported by Alex Balashov (cherry picked from commit e4bd74c1718ea2218f2ad426537641cc9ef71d7d) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6ef196b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Apr 15 11:02:06 2010 +0200 tm: added drop functionality for branch_route - missing part of K integration - credits to Andreas Granig for reporting and testing (cherry picked from commit d592824b6e1b0d31a0d4fff28d003cc4c3c878bc) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=74117b4… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Apr 14 10:39:47 2010 +0200 mysql: include dprint.h - several files were missing include of dprint.h - could be the cause for missing symbol reported by Pratab Ali for opensuse (cherry picked from commit 9c5da73226b3124c888e434e3df55cc1466b3b07) (cherry picked from commit 3a25f8327c39420d99d11ba4502d31617fa743cb) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a381294… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Fri Apr 2 15:03:09 2010 +0300 lib/kcore/km_crc.c Fixed wrong computation of crc32 The final result was not bitwise complemented, yeilding an incorrect result (cherry picked from commit 342d6eb44fe6f795d7d31fc42ec6f7942859503b) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=18361ac… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Fri Apr 9 12:09:33 2010 +0300 modules/carrierroute Improved usefulness of error messages in case of a incorrect AVP value (cherry picked from commit 6c51d903f5a314f4116df81121d1faa5ea8c95d3) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9405756… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Apr 8 23:21:53 2010 +0200 tm: notes about avp parameters format - they can be either PV style $avp(name) for K compat mode or just the name of avp for S compat mode (cherry picked from commit 2bbb798da2e332f5558646e6faa285e6377286bd) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=33cb93a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Apr 8 22:57:48 2010 +0200 tm: don't fix timeout AVP value for K compat mode - the avps for timeout still expect second values unlike module parameters that expect miliseconds - therefore do not adjust the value if it is too low - reported by Andreas Granig (cherry picked from commit 5aa3167b088bdf2e7a5d74fdbbd83c0e150f7741) (cherry picked from commit 4e5748d50b6a586fb45f312ade6aa572b4044d82) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=66c6953… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Thu Apr 8 16:54:53 2010 +0300 modules/carrierroute In case of incorrect avp value (unknown carrier id), print the name of the avp accordinly to its type (cherry picked from commit 5604e51eb8fd6502495a37c26f9c549bcd63ca8a) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=47a20a7… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Tue Apr 6 16:21:35 2010 +0300 utils/kamctl/kamctl.base added documentation of ps command (cherry picked from commit 5266c7f5ed1e2f6364f646ece3828e259491a6ae) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8d2fd62… Author: Ovidiu Sas <osas(a)voipembedded.com> Date: Wed Mar 31 10:31:05 2010 -0400 kamctl: fix 'kamctl cr addcarrier' command - 'COMMENT' field and 'MASK' field were mixed up causing 'kamctl cr addcarrier' to fail. (cherry picked from commit 7b8b6f647365655afa210f5e95baef06398ef612) (cherry picked from commit 3c4db3e110aae42f36e03e0d10792f4676063fa0) URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=57a0c6c… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Mar 26 10:48:41 2010 +0100 kamailo.cfg: fixed typo - typo in the hints of missed_calls table update for accouting (cherry picked from commit 6087434f2ddfbe07767d5a4eeb739b72b7c27a93)

1 0

git:andrei/tcp_tls_changes: tls: migrated to the runtime cfg framework
by Andrei Pelinescu-Onciul 28 May '10

28 May '10

Module: sip-router Branch: andrei/tcp_tls_changes Commit: e40e993ef9039ace73caae09e983abd328ee28fc URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e40e993… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri May 28 12:35:07 2010 +0200 tls: migrated to the runtime cfg framework - moved all the module params to runtime config. Most of the variables are read-only (for now) with the following exceptions: config - can be changed at runtime and if followed by a tls.realod, the configuration from the new file will be loaded. log connection_timeout low_mem_threshold1 low_mem_threshold2 - renamed tls_fix_cfg() to tls_fix_domains_cfg() to avoid confusion with the runtime cfg (named tls_cfg). --- modules/tls/tls_cfg.c | 222 +++++++++++++++++++++++++++++++++++++++++++++ modules/tls/tls_cfg.h | 102 +++++++++++++++++++++ modules/tls/tls_config.c | 1 + modules/tls/tls_domain.c | 32 +++++-- modules/tls/tls_domain.h | 2 +- modules/tls/tls_dump_vf.c | 4 + modules/tls/tls_init.c | 89 ++++++++++-------- modules/tls/tls_init.h | 5 - modules/tls/tls_mod.c | 195 ++++++++++++---------------------------- modules/tls/tls_mod.h | 8 -- modules/tls/tls_rpc.c | 6 +- modules/tls/tls_select.c | 5 +- modules/tls/tls_server.c | 18 +++- 13 files changed, 484 insertions(+), 205 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=e40…

2 2

[tracker] Task opened: debian changelog in Kamailio 3.0.2 tag
by sip-router 28 May '10

28 May '10

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Jon Bonilla (manwe) Attached to Project - sip-router Summary - debian changelog in Kamailio 3.0.2 tag Task Type - Bug Report Category - tools/scripts Status - Unconfirmed Assigned To - Operating System - Linux Severity - Very Low Priority - Normal Reported Version - Development Due in Version - Undecided Due Date - Undecided Details - The debian changelog files are not updated to Kamailio version 3.0.2 under pkg/kamailio/debian-etch and pkg/kamailio/debian-lenny. manwe@multivac:~/sipwise/build/kamailio-3.0.2/pkg/kamailio$ head -n1 debian-etch/changelog kamailio (1.5.0-svn1) unstable; urgency=low manwe@multivac:~/sipwise/build/kamailio-3.0.2/pkg/kamailio$ head -n1 debian-lenny/changelog kamailio (3.0.0-rc2) unstable; urgency=low manwe@multivac:~/sipwise/build/kamailio-3.0.2/pkg/kamailio$ head -n1 debian/changelog kamailio (3.0.2) unstable; urgency=low More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=75 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

[tracker] Task closed: allow rewriting request uri also in failure route
by sip-router 28 May '10

28 May '10

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#72 - allow rewriting request uri also in failure route User who did this - Juha Heinanen (jh) Reason for closing: Implemented More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=72 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

git:master: modules/lcr: Simpler implementation of next_gw() function
by Juha Heinanen 28 May '10

28 May '10

Module: sip-router Branch: master Commit: 5af7ae813ca63452c571ea7528754dd0b759c019 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=5af7ae8… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Fri May 28 15:54:51 2010 +0300 modules/lcr: Simpler implementation of next_gw() function - Based on recent changes related to Request-URI handling. --- modules/lcr/lcr_mod.c | 54 ++++++++++++------------------------------------ 1 files changed, 14 insertions(+), 40 deletions(-) diff --git a/modules/lcr/lcr_mod.c b/modules/lcr/lcr_mod.c index add81d9..95d6c12 100644 --- a/modules/lcr/lcr_mod.c +++ b/modules/lcr/lcr_mod.c @@ -79,7 +79,6 @@ #include "../../resolve.h" #include "../../mod_fix.h" #include "../../socket_info.h" -#include "../../modules/tm/tm_load.h" #include "../../pvar.h" #include "../../mod_fix.h" #include "hash.h" @@ -149,9 +148,6 @@ static void free_shared_memory(void); * Type definitions */ -/* TMB Structure */ -struct tm_binds tmb; - struct gw_grp { unsigned int grp_id; unsigned int first; /* index to first gw of group in gw table */ @@ -242,7 +238,6 @@ struct gw_info **gwtp = (struct gw_info **)NULL; /* Pointer to lcr hash table pointer table */ struct lcr_info ***lcrtp = (struct lcr_info ***)NULL; - /* * Functions that are defined later */ @@ -403,6 +398,7 @@ static int mod_init(void) unsigned short avp_flags; unsigned int i; + /* Register RPC commands */ if (rpc_register_array(lcr_rpc)!=0) { LM_ERR("failed to register RPC commands\n"); return -1; @@ -1869,22 +1865,20 @@ static int defunct_gw(struct sip_msg* _m, char *_defunct_period, char *_s2) /* - * When called first time in route block, rewrites scheme, host, port, and + * When called first time, rewrites scheme, host, port, and * transport parts of R-URI based on first gw_uri_avp value, which is then * destroyed. Saves R-URI user to ruri_user_avp for later use. * - * On other calls appends a new branch to request, where scheme, host, port, + * On other calls, rewrites R-URI, where scheme, host, port, * and transport of URI are taken from the first gw_uri_avp value, - * which is then destroyed. URI user is taken either from R-URI (first - * call in failure route block) or from ruri_user_avp value saved earlier. + * which is then destroyed. URI user is taken either from ruri_user_avp + * value saved earlier. * * Returns 1 upon success and -1 upon failure. */ static int next_gw(struct sip_msg* _m, char* _s1, char* _s2) { int_str ruri_user_val, val; - struct action act; - struct run_act_ctx ra_ctx; struct usr_avp *ru_avp; int rval; str uri_str; @@ -1899,6 +1893,7 @@ static int next_gw(struct sip_msg* _m, char* _s1, char* _s2) /* First invocation either in route or failure route block. * Take Request-URI user from Request-URI and generate Request * and Destination URIs. */ + if (parse_sip_msg_uri(_m) < 0) { LM_ERR("parsing of R-URI failed\n"); return -1; @@ -1910,48 +1905,27 @@ static int next_gw(struct sip_msg* _m, char* _s1, char* _s2) /* Save Request-URI user into uri_user_avp for use in subsequent * invocations. */ + val.s = _m->parsed_uri.user; add_avp(ruri_user_avp_type|AVP_VAL_STR, ruri_user_avp, val); LM_DBG("added ruri_user_avp <%.*s>\n", val.s.len, val.s.s); } else { - /* Subsequent invocation either in route or failure route block. */ - - /* Take Request-URI user from ruri_user_avp and generate Request + /* Subsequent invocation either in route or failure route block. + * Take Request-URI user from ruri_user_avp and generate Request * and Destination URIs. */ + if (generate_uris(r_uri, &(ruri_user_val.s), &r_uri_len, dst_uri, &dst_uri_len, &addr, &flags) != 1) { return -1; } } - if ((is_route_type(REQUEST_ROUTE)) && (ru_avp == NULL)) { - - /* First invocation in route block => Rewrite Request URI. */ - memset(&act, '\0', sizeof(act)); - act.type = SET_URI_T; - act.val[0].type = STRING_ST; - act.val[0].u.string = r_uri; - init_run_actions_ctx(&ra_ctx); - rval = do_action(&ra_ctx, &act, _m); - if (rval != 1) { - LM_ERR("do_action failed with return value <%d>\n", rval); - return -1; - } - - } else { - - /* Subsequent invocation in route block or any invocation in - * failure route block => append new branch. */ - uri_str.s = r_uri; - uri_str.len = r_uri_len; - LM_DBG("appending branch <%.*s>\n", uri_str.len, uri_str.s); - if (append_branch(_m, &uri_str, 0, 0, Q_UNSPECIFIED, 0, 0) == -1) { - LM_ERR("when appending branch <%.*s>\n", uri_str.len, uri_str.s); - return -1; - } - } + /* Rewrite Request URI */ + uri_str.s = r_uri; + uri_str.len = r_uri_len; + rewrite_uri(_m, &uri_str); /* Set Destination URI if not empty */ if (dst_uri_len > 0) {

1 0

[tracker] Task opened: acc report_ack doesn't work
by sip-router 28 May '10

28 May '10

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Savolainen Dmitri (snen) Attached to Project - sip-router Summary - acc report_ack doesn't work Task Type - Bug Report Category - Module Status - Unconfirmed Assigned To - Operating System - Linux Severity - Medium Priority - Normal Reported Version - Development Due in Version - Undecided Due Date - Undecided Details - in 3.0.2 kamailio revision. Trying to use modparam("acc", "report_ack", 1) and modparam("acc", "db_flag", 1) ACK records absent in acc table after dialog established... modules/tm/t_lookup.c:1464 unmatched_totag(t_ack, p_msg) return false; after changing defination of int unmatched_totag(struct cell *t, struct sip_msg *ack) from modules/tm/t_reply.c to old version (openser-1.2.0-notls for example) all work fine again More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=74 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

git:andrei/tcp_tls_changes: tls: fix empty files treatment
by Andrei Pelinescu-Onciul 28 May '10

28 May '10

Module: sip-router Branch: andrei/tcp_tls_changes Commit: 59f603ace872b2b4b05048b2e57b982b42f7aebe URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=59f603a… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri May 28 13:22:00 2010 +0200 tls: fix empty files treatment Empty filenames ("") are now treated as unset. --- modules/tls/tls_domain.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/modules/tls/tls_domain.c b/modules/tls/tls_domain.c index 793bf58..762991a 100644 --- a/modules/tls/tls_domain.c +++ b/modules/tls/tls_domain.c @@ -336,7 +336,7 @@ static int load_cert(tls_domain_t* d) int i; int procs_no; - if (!d->cert_file.s) { + if (!d->cert_file.s || !d->cert_file.len) { DBG("%s: No certificate configured\n", tls_domain_str(d)); return 0; } @@ -363,7 +363,7 @@ static int load_ca_list(tls_domain_t* d) int i; int procs_no; - if (!d->ca_file.s) { + if (!d->ca_file.s || !d->ca_file.len) { DBG("%s: No CA list configured\n", tls_domain_str(d)); return 0; } @@ -697,7 +697,7 @@ static int load_private_key(tls_domain_t* d) int idx, ret_pwd, i; int procs_no; - if (!d->pkey_file.s) { + if (!d->pkey_file.s || !d->pkey_file.len) { DBG("%s: No private key specified\n", tls_domain_str(d)); return 0; }

1 0

git:andrei/tcp_tls_changes: db_flatstore: updated get_abs_pathname use
by Andrei Pelinescu-Onciul 28 May '10

28 May '10

Module: sip-router Branch: andrei/tcp_tls_changes Commit: 8a5f9abd13a9339ea9e4011676a041b6fb0b5b90 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8a5f9ab… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri May 28 12:33:33 2010 +0200 db_flatstore: updated get_abs_pathname use Use pkg_free() now for freeing the result of get_abs_pathname(). --- modules/db_flatstore/flat_uri.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules/db_flatstore/flat_uri.c b/modules/db_flatstore/flat_uri.c index 05005f3..7c35f95 100644 --- a/modules/db_flatstore/flat_uri.c +++ b/modules/db_flatstore/flat_uri.c @@ -70,7 +70,7 @@ int flat_uri(db_uri_t* uri) error: if (furi) { - if (furi->path.s) free(furi->path.s); + if (furi->path.s) pkg_free(furi->path.s); db_drv_free(&furi->drv); pkg_free(furi); }

1 0