sr-dev

sr-dev@lists.kamailio.org

18 participants
44296 discussions

by Klaus Darilion

Module: sip-router Branch: sr_3.0 Commit: 1ab44045e92c397c801a9dccd82215125b098895 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1ab4404… Author: root <root(a)relay.labs.nic.at> Committer: Klaus Darilion <klaus.mailinglists(a)pernau.at> Date: Wed Dec 30 13:25:01 2009 +0000 - remove unused code - correct indention and make code more readable (cherry picked from commit 9eacaacb45fe700615ac94cc3944824f1b22fdc4) --- modules_k/siputils/contact_ops.c | 46 +++++++++++++++++-------------------- 1 files changed, 21 insertions(+), 25 deletions(-) diff --git a/modules_k/siputils/contact_ops.c b/modules_k/siputils/contact_ops.c index a796610..f0d0dbb 100644 --- a/modules_k/siputils/contact_ops.c +++ b/modules_k/siputils/contact_ops.c @@ -160,40 +160,36 @@ decode_contact (struct sip_msg *msg,char *unused1,char *unused2) separator = contact_flds_separator[0]; if ((msg->new_uri.s == NULL) || (msg->new_uri.len == 0)) - { + { uri = msg->first_line.u.request.uri; - if (uri.s == NULL) return -1; - } + if (uri.s == NULL) + return -1; + } - res = decode_uri (uri, separator, &newUri); + res = decode_uri (uri, separator, &newUri); #ifdef DEBUG - if (res == 0) fprintf (stdout, "newuri.s=[%.*s]\n", newUri.len, newUri.s); + if (res == 0) + fprintf (stdout, "newuri.s=[%.*s]\n", newUri.len, newUri.s); #endif - if (res != 0) - { - LM_ERR("failed decoding contact.Code %d\n", res); + if (res != 0) + { + LM_ERR("failed decoding contact.Code %d\n", res); #ifdef STRICT_CHECK - return res; + return res; #endif - } + } + else + { + /* we do not modify the original first line */ + if ((msg->new_uri.s == NULL) || (msg->new_uri.len == 0)) + msg->new_uri = newUri; else - /* we do not modify the original first line */ - if ((msg->new_uri.s == NULL) || (msg->new_uri.len == 0)) msg->new_uri = newUri; - else - { - pkg_free(msg->new_uri.s); - msg->new_uri = newUri; - } - - - /* - if (patch (msg, uri.s, uri.len, newUri.s, newUri.len) < 0) { - LM_ERR("lumping failed in mangling port \n"); - return -2; - } - */ + pkg_free(msg->new_uri.s); + msg->new_uri = newUri; + } + } return 1; }

15 years

git:kamailio_3.0: - fix bug in encode_contact()

by Klaus Darilion

Module: sip-router Branch: kamailio_3.0 Commit: 8929913f5edb519b7ed280a12728376ed872ec53 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8929913… Author: Klaus Darilion <klaus.mailinglists(a)pernau.at> Committer: Klaus Darilion <klaus.mailinglists(a)pernau.at> Date: Wed Dec 30 15:53:12 2009 +0000 - fix bug in encode_contact() - fix and extend documentation about contact encoding/decoding --- modules_k/siputils/README | 392 ++++++++++++++++------------- modules_k/siputils/contact_ops.c | 5 +- modules_k/siputils/doc/siputils_admin.xml | 42 ++-- 3 files changed, 252 insertions(+), 187 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=892…

15 years

git:kamailio_3.0: - remove unused code

by Klaus Darilion

Module: sip-router Branch: kamailio_3.0 Commit: 9eacaacb45fe700615ac94cc3944824f1b22fdc4 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9eacaac… Author: root <root(a)relay.labs.nic.at> Committer: Klaus Darilion <klaus.mailinglists(a)pernau.at> Date: Wed Dec 30 13:25:01 2009 +0000 - remove unused code - correct indention and make code more readable --- modules_k/siputils/contact_ops.c | 46 +++++++++++++++++-------------------- 1 files changed, 21 insertions(+), 25 deletions(-) diff --git a/modules_k/siputils/contact_ops.c b/modules_k/siputils/contact_ops.c index a796610..f0d0dbb 100644 --- a/modules_k/siputils/contact_ops.c +++ b/modules_k/siputils/contact_ops.c @@ -160,40 +160,36 @@ decode_contact (struct sip_msg *msg,char *unused1,char *unused2) separator = contact_flds_separator[0]; if ((msg->new_uri.s == NULL) || (msg->new_uri.len == 0)) - { + { uri = msg->first_line.u.request.uri; - if (uri.s == NULL) return -1; - } + if (uri.s == NULL) + return -1; + } - res = decode_uri (uri, separator, &newUri); + res = decode_uri (uri, separator, &newUri); #ifdef DEBUG - if (res == 0) fprintf (stdout, "newuri.s=[%.*s]\n", newUri.len, newUri.s); + if (res == 0) + fprintf (stdout, "newuri.s=[%.*s]\n", newUri.len, newUri.s); #endif - if (res != 0) - { - LM_ERR("failed decoding contact.Code %d\n", res); + if (res != 0) + { + LM_ERR("failed decoding contact.Code %d\n", res); #ifdef STRICT_CHECK - return res; + return res; #endif - } + } + else + { + /* we do not modify the original first line */ + if ((msg->new_uri.s == NULL) || (msg->new_uri.len == 0)) + msg->new_uri = newUri; else - /* we do not modify the original first line */ - if ((msg->new_uri.s == NULL) || (msg->new_uri.len == 0)) msg->new_uri = newUri; - else - { - pkg_free(msg->new_uri.s); - msg->new_uri = newUri; - } - - - /* - if (patch (msg, uri.s, uri.len, newUri.s, newUri.len) < 0) { - LM_ERR("lumping failed in mangling port \n"); - return -2; - } - */ + pkg_free(msg->new_uri.s); + msg->new_uri = newUri; + } + } return 1; }

15 years

git:kamailio_3.0: - fix bug when RURI was changed before and newuri will be used

by Klaus Darilion

Module: sip-router Branch: kamailio_3.0 Commit: 1e663ffa8129c4b24f21563c09a5bea4ecd65784 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1e663ff… Author: Klaus Darilion <klaus.mailinglists(a)pernau.at> Committer: Klaus Darilion <klaus.mailinglists(a)pernau.at> Date: Wed Dec 30 13:51:24 2009 +0000 - fix bug when RURI was changed before and newuri will be used --- modules_k/siputils/contact_ops.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/modules_k/siputils/contact_ops.c b/modules_k/siputils/contact_ops.c index f0d0dbb..5911f8c 100644 --- a/modules_k/siputils/contact_ops.c +++ b/modules_k/siputils/contact_ops.c @@ -165,6 +165,10 @@ decode_contact (struct sip_msg *msg,char *unused1,char *unused2) if (uri.s == NULL) return -1; } + else + { + uri = msg->new_uri; + } res = decode_uri (uri, separator, &newUri);

15 years

git:kamailio_3.0: nathelper(k): fix for swap scenario

by Daniel-Constantin Mierla

Module: sip-router Branch: kamailio_3.0 Commit: 839e2fee663846f68289a519c27aef433a872afa URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=839e2fe… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Jan 7 11:33:37 2010 +0100 nathelper(k): fix for swap scenario A -> RTPP -> B, offer in INVITE, answer in 200 OK (SDP in INVITE+200 OK) A -> RTPP -> B, re-Invite on session expiry, SDP in 200 OK + ACK The from-tag and to-tag are not swapped in the ACK with rtpproxy_answer(). Options "sl" would do the trick also, but "s" is deprecated soon Patch by Walter Schober --- modules_k/nathelper/nathelper.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/modules_k/nathelper/nathelper.c b/modules_k/nathelper/nathelper.c index 90fca05..3e66ee4 100644 --- a/modules_k/nathelper/nathelper.c +++ b/modules_k/nathelper/nathelper.c @@ -2985,12 +2985,14 @@ force_rtp_proxy(struct sip_msg* msg, char* str1, char* str2, int offer) FORCE_RTP_PROXY_RET (-1); } create = 0; - if (swap != 0 || (msg->first_line.type == SIP_REPLY && offer != 0)) { + if (swap != 0 || (msg->first_line.type == SIP_REPLY && offer != 0) + || (msg->first_line.type == SIP_REQUEST && offer == 0)) { tmp = from_tag; from_tag = to_tag; to_tag = tmp; } - } else if (swap != 0 || (msg->first_line.type == SIP_REPLY && offer != 0)) { + } else if (swap != 0 || (msg->first_line.type == SIP_REPLY && offer != 0) + || (msg->first_line.type == SIP_REQUEST && offer == 0)) { if (to_tag.len == 0) { FORCE_RTP_PROXY_RET (-1); }

15 years

SF.net SVN: openser:[5965] branches/1.5/modules/nathelper/nathelper.c

by Daniel-Constantin Mierla

Revision: 5965 http://openser.svn.sourceforge.net/openser/?rev=5965&view=rev Author: miconda Date: 2010-01-07 10:31:02 +0000 (Thu, 07 Jan 2010) Log Message: ----------- fix for scenario: A -> RTPP -> B, offer in INVITE, answer in 200 OK (SDP in INVITE+200 OK) A -> RTPP -> B, re-Invite on session expiry, SDP in 200 OK + ACK The from-tag and to-tag are not swapped in the ACK with rtpproxy_answer(). Options "sl" would do the trick also, but "s" is deprecated soon Patch by Walter Schober Modified Paths: -------------- branches/1.5/modules/nathelper/nathelper.c This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

15 years

git:kamailio_3.0: Backport of the changes (see previous commits).

by Carsten Bock

Module: sip-router Branch: kamailio_3.0 Commit: 7956c7387a1217016d0e5192e7583e8385b27e14 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=7956c73… Author: Carsten Bock <lists(a)bock.info> Committer: Carsten Bock <lists(a)bock.info> Date: Wed Jan 6 16:53:17 2010 +0100 Backport of the changes (see previous commits). --- modules_k/nathelper/rtpproxy_stream.c | 11 +++++++---- 1 files changed, 7 insertions(+), 4 deletions(-) diff --git a/modules_k/nathelper/rtpproxy_stream.c b/modules_k/nathelper/rtpproxy_stream.c index 437cc5c..520bd2d 100644 --- a/modules_k/nathelper/rtpproxy_stream.c +++ b/modules_k/nathelper/rtpproxy_stream.c @@ -38,7 +38,6 @@ int fixup_var_str_int(void **param, int param_no) { - unsigned long go_to; int ret; pv_elem_t *model; str s; @@ -57,10 +56,14 @@ fixup_var_str_int(void **param, int param_no) } *param = (void *)model; } else if (param_no == 2) { - go_to = str2s(*param, strlen(*param), &ret); - if (ret == 0) { + /* According to + * http://www.kamailio.org/docs/modules/1.5.x/nathelper.html#rtpproxy_stream2x… + * this could be -1 */ + s.s = (char *)(*param); + s.len = strlen(s.s); + if (str2sint(&s, &ret)==0) { pkg_free(*param); - *param = (void *)go_to; + *param = (void *)ret; } else { LM_ERR("bad number <%s>\n", (char *)(*param)); return E_CFG;

15 years

[tracker] Comment added: Strange symbols in NOTIFY dialog-info+xml body

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#25 - Strange symbols in NOTIFY dialog-info+xml body User who did this - Klaus Darilion (klaus3000) ---------- Hi Andrey! I checked the code but could not find a bug. Can you send me the MI command you use to generate the PUBLISH, so that I can reproduce the bug? ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=25#comment24 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.

15 years

git:kamailio_3.0: presence(k): aliased MLA to SLA

by Daniel-Constantin Mierla

Module: sip-router Branch: kamailio_3.0 Commit: 1a9f379ed66dbcc05f7750ff961c0a7a6607b0db URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1a9f379… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Jan 6 18:50:31 2010 +0100 presence(k): aliased MLA to SLA - doc updated to reflect RFCs and drafts --- modules_k/presence/subscribe.c | 19 +++++++++------- modules_k/presence/utils_func.h | 3 +- modules_k/presence_xml/README | 11 ++++++--- modules_k/presence_xml/doc/presence_xml_admin.xml | 24 ++++++++++++++++---- 4 files changed, 39 insertions(+), 18 deletions(-) diff --git a/modules_k/presence/subscribe.c b/modules_k/presence/subscribe.c index 3067919..6ea70ae 100644 --- a/modules_k/presence/subscribe.c +++ b/modules_k/presence/subscribe.c @@ -802,7 +802,7 @@ int extract_sdialog_info(subs_t* subs,struct sip_msg* msg, int mexp, else { memset( &TO , 0, sizeof(TO) ); - if( !parse_to(msg->to->body.s,msg->to->body.s + msg->to->body.len + 1, &TO)); + if( !parse_to(msg->to->body.s,msg->to->body.s + msg->to->body.len + 1, &TO)) { LM_DBG("'To' header NOT parsed\n"); goto error; @@ -1030,16 +1030,19 @@ int get_stored_info(struct sip_msg* msg, subs_t* subs, int* reply_code, { lock_get(&subs_htable[i].lock); s= search_shtable(subs_htable, subs->callid,subs->to_tag,subs->from_tag, i); - if (s && !EVENT_DIALOG_SLA(s->event->evp)) + if (s) { - pres_uri.s= (char*)pkg_malloc(s->pres_uri.len* sizeof(char)); - if(pres_uri.s== NULL) + if(!EVENT_DIALOG_SLA(s->event->evp)) { - lock_release(&subs_htable[i].lock); - ERR_MEM(PKG_MEM_STR); + pres_uri.s= (char*)pkg_malloc(s->pres_uri.len* sizeof(char)); + if(pres_uri.s== NULL) + { + lock_release(&subs_htable[i].lock); + ERR_MEM(PKG_MEM_STR); + } + memcpy(pres_uri.s, s->pres_uri.s, s->pres_uri.len); + pres_uri.len= s->pres_uri.len; } - memcpy(pres_uri.s, s->pres_uri.s, s->pres_uri.len); - pres_uri.len= s->pres_uri.len; goto found_rec; } lock_release(&subs_htable[i].lock); diff --git a/modules_k/presence/utils_func.h b/modules_k/presence/utils_func.h index c6590c8..5f94f0b 100644 --- a/modules_k/presence/utils_func.h +++ b/modules_k/presence/utils_func.h @@ -50,7 +50,8 @@ #define EVENT_DIALOG_SLA(ev) \ - ((ev)->type == EVENT_DIALOG && (ev)->params.dialog.sla) + ((ev)->type == EVENT_DIALOG \ + && ((ev)->params.dialog.sla || (ev)->params.dialog.ma)) static inline int uandd_to_uri(str user, str domain, str *out) diff --git a/modules_k/presence_xml/README b/modules_k/presence_xml/README index e38262c..1b6e0d6 100644 --- a/modules_k/presence_xml/README +++ b/modules_k/presence_xml/README @@ -90,10 +90,13 @@ Chapter 1. Admin Guide The module does specific handling for notify-subscribe events using xml bodies. It is used with the general event handling module, presence. It - constructs and adds 3 events to it: presence, presence.winfo, - dialog;sla. By default all these events will be handled. If you do want - to activate only certain events then disable the unneeded events via - the module parameters. + constructs and adds 3 events to it: + * presence - SIMPLE status presence: RFC 3856 + * presence.winfo - SIMPLE watcher info: RFC 3857 + * dialog;sla (or dialog;ma) - Bridged Line Appearances (BLA) (or + Multiple Line Appearances (MLA)): draft-anil-sipping-bla + + You can control which events are enabled via module parameters. This module takes the XCAP permission rule documents from xcap_table. The presence permission rules are interpreted according to the diff --git a/modules_k/presence_xml/doc/presence_xml_admin.xml b/modules_k/presence_xml/doc/presence_xml_admin.xml index cefadf0..1ee17c6 100644 --- a/modules_k/presence_xml/doc/presence_xml_admin.xml +++ b/modules_k/presence_xml/doc/presence_xml_admin.xml @@ -16,11 +16,25 @@ <section> <title>Overview</title> <para> - The module does specific handling for notify-subscribe events using xml bodies. - It is used with the general event handling module, presence. It constructs and adds - 3 events to it: presence, presence.winfo, dialog;sla. By default all these events - will be handled. If you do want to activate only certain events then disable the - unneeded events via the module parameters. + The module does specific handling for notify-subscribe events using xml + bodies. It is used with the general event handling module, presence. + It constructs and adds 3 events to it: + <itemizedlist> + <listitem> + <para>presence - SIMPLE status presence: RFC 3856</para> + </listitem> + <listitem> + <para>presence.winfo - SIMPLE watcher info: RFC 3857</para> + </listitem> + <listitem> + <para>dialog;sla (or dialog;ma) - Bridged Line Appearances + (BLA) (or Multiple Line Appearances (MLA)): + draft-anil-sipping-bla</para> + </listitem> + </itemizedlist> + </para> + <para> + You can control which events are enabled via module parameters. </para> <para> This module takes the XCAP permission rule documents from xcap_table.

15 years

git:kamailio_3.0: core: event parsing enahced for dialog; ma

by Daniel-Constantin Mierla

Module: sip-router Branch: kamailio_3.0 Commit: 1e99795d2ddebe8a6794c3b407f96d30378b872f URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1e99795… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Jan 6 18:47:36 2010 +0100 core: event parsing enahced for dialog;ma - event defined by draft-anil-sipping-bla for Multiple Line Appearances, folloing to Shared/Bridged Line Appearances (SLA, BLA) - draft is defunct but still some implementations out there --- parser/parse_param.c | 9 +++++++++ parser/parse_param.h | 4 +++- 2 files changed, 12 insertions(+), 1 deletions(-) diff --git a/parser/parse_param.c b/parser/parse_param.c index 5f204bb..20491f8 100644 --- a/parser/parse_param.c +++ b/parser/parse_param.c @@ -99,6 +99,15 @@ static inline void parse_event_dialog_class(param_hooks_t* h, param_t* p) h->event_dialog.sla = p; } break; + + case 'm': + case 'M': + if ((p->name.len == 2) && + (!strncasecmp(p->name.s + 1, "a", 1))) { + p->type = P_MA; + h->event_dialog.ma = p; + } + break; } } diff --git a/parser/parse_param.h b/parser/parse_param.h index 9d1060d..c50edd9 100644 --- a/parser/parse_param.h +++ b/parser/parse_param.h @@ -61,7 +61,8 @@ typedef enum ptype { P_FROM_TAG, /* Dialog event package: from-tag */ P_TO_TAG, /* Dialog event package: to-tag */ P_ISD, /* Dialog event package: include-session-description */ - P_SLA /* Dialog event package: sla */ + P_SLA, /* Dialog event package: sla */ + P_MA /* Dialog event package: ma */ } ptype_t; @@ -121,6 +122,7 @@ struct event_dialog_hooks { struct param* to_tag; struct param* include_session_description; struct param* sla; + struct param* ma; }; /*

15 years

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev