sr-dev August 2020

sr-dev@lists.kamailio.org

17 participants
244 discussions

git:master:5ea7ca4d: modules: readme files regenerated - textopsx ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: 5ea7ca4d6408f0bdb10afe12ece44d8b51dd1514 URL: https://github.com/kamailio/kamailio/commit/5ea7ca4d6408f0bdb10afe12ece44d8… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2020-08-02T17:16:16+02:00 modules: readme files regenerated - textopsx ... [skip ci] --- Modified: src/modules/textopsx/README --- Diff: https://github.com/kamailio/kamailio/commit/5ea7ca4d6408f0bdb10afe12ece44d8… Patch: https://github.com/kamailio/kamailio/commit/5ea7ca4d6408f0bdb10afe12ece44d8…

3 years, 9 months

git:master:dfa16778: textopsx: docs for change_reply_status_code() function

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: dfa1677800a6b78aabaa1217869d23fac5e1a345 URL: https://github.com/kamailio/kamailio/commit/dfa1677800a6b78aabaa1217869d23f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-02T17:08:49+02:00 textopsx: docs for change_reply_status_code() function --- Modified: src/modules/textopsx/doc/functions.xml --- Diff: https://github.com/kamailio/kamailio/commit/dfa1677800a6b78aabaa1217869d23f… Patch: https://github.com/kamailio/kamailio/commit/dfa1677800a6b78aabaa1217869d23f… --- diff --git a/src/modules/textopsx/doc/functions.xml b/src/modules/textopsx/doc/functions.xml index 717b8ddb40..0ef8dab411 100644 --- a/src/modules/textopsx/doc/functions.xml +++ b/src/modules/textopsx/doc/functions.xml @@ -82,6 +82,38 @@ reply_route { exit; } } +... + </programlisting> + </example> + </section> + + <section id="textopsx.f.change_reply_status_code"> + <title> + <function>change_reply_status_code(vcode)</function> + </title> + <para> + Change the status code for a SIP reply . + </para> + <para>Meaning of the parameters is as follows:</para> + <itemizedlist> + <listitem> + <para><emphasis>vcode</emphasis> - the new status code. + </para> + </listitem> + </itemizedlist> + <para> + This function can be used from ONREPLY_ROUTE. + </para> + <example> + <title><function>change_reply_status_code</function> usage</title> + <programlisting> +... +reply_route { + if (status == "604") { + change_reply_status_code("404"); + exit; + } +} ... </programlisting> </example>

3 years, 9 months

git:master:7b95ddda: textopsx: added change_reply_status_code(vcode)

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 7b95ddda1346faea64a85c2593a9a579ea46a5da URL: https://github.com/kamailio/kamailio/commit/7b95ddda1346faea64a85c2593a9a57… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-02T17:08:49+02:00 textopsx: added change_reply_status_code(vcode) - function to change only the status code for non-2xx replies --- Modified: src/modules/textopsx/textopsx.c --- Diff: https://github.com/kamailio/kamailio/commit/7b95ddda1346faea64a85c2593a9a57… Patch: https://github.com/kamailio/kamailio/commit/7b95ddda1346faea64a85c2593a9a57… --- diff --git a/src/modules/textopsx/textopsx.c b/src/modules/textopsx/textopsx.c index 1452e1c3d1..4602977264 100644 --- a/src/modules/textopsx/textopsx.c +++ b/src/modules/textopsx/textopsx.c @@ -50,6 +50,8 @@ static int msg_apply_changes_f(sip_msg_t *msg, char *str1, char *str2); static int change_reply_status_f(sip_msg_t *, char *, char *); static int change_reply_status_fixup(void **param, int param_no); +static int change_reply_status_code_f(sip_msg_t *, char *, char *); + static int w_keep_hf_f(sip_msg_t *, char *, char *); static int w_fnmatch2_f(sip_msg_t *, char *, char *); @@ -84,6 +86,8 @@ static cmd_export_t cmds[] = { REQUEST_ROUTE | ONREPLY_ROUTE}, {"change_reply_status", change_reply_status_f, 2, change_reply_status_fixup, 0, ONREPLY_ROUTE}, + {"change_reply_status_code", change_reply_status_code_f, 1, + fixup_igp_null, 0, ONREPLY_ROUTE}, {"remove_body", (cmd_function)w_remove_body_f, 0, 0, 0, ANY_ROUTE}, {"keep_hf", (cmd_function)w_keep_hf_f, 0, fixup_regexp_null, 0, ANY_ROUTE}, {"keep_hf", (cmd_function)w_keep_hf_f, 1, fixup_regexp_null, 0, ANY_ROUTE}, @@ -275,6 +279,49 @@ static int change_reply_status_f( } +/** + * + */ +static int ki_change_reply_status_code(sip_msg_t *msg, int code) +{ + if((code < 100) || (code > 699)) { + LM_ERR("wrong status code: %d\n", code); + return -1; + } + + if(((code < 300) || (msg->REPLY_STATUS < 300)) + && (code / 100 != msg->REPLY_STATUS / 100)) { + LM_ERR("the class of provisional or " + "positive final replies cannot be changed\n"); + return -1; + } + + /* rewrite the status code directly in the message buffer */ + msg->first_line.u.reply.statuscode = code; + msg->first_line.u.reply.status.s[2] = code % 10 + '0'; + code /= 10; + msg->first_line.u.reply.status.s[1] = code % 10 + '0'; + code /= 10; + msg->first_line.u.reply.status.s[0] = code + '0'; + + return 1l; +} + +/** + * + */ +static int change_reply_status_code_f(sip_msg_t *msg, char *pcode, char *p2) +{ + int code; + + if(fixup_get_ivalue(msg, (gparam_t*)pcode, &code)<0) { + LM_ERR("cannot get parameters\n"); + return -1; + } + + return ki_change_reply_status_code(msg, code); +} + /** * */ @@ -2195,6 +2242,11 @@ static sr_kemi_t sr_kemi_textopsx_exports[] = { { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } }, + { str_init("textopsx"), str_init("change_reply_status_code"), + SR_KEMIP_INT, ki_change_reply_status_code, + { SR_KEMIP_INT, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, { str_init("textopsx"), str_init("keep_hf"), SR_KEMIP_INT, ki_keep_hf, { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE,

3 years, 9 months

git:master:bddf16db: db_text: free pointer in case of error

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: bddf16db74ea779bb01805fc2cbabfdaf8f4d62b URL: https://github.com/kamailio/kamailio/commit/bddf16db74ea779bb01805fc2cbabfd… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-02T17:08:49+02:00 db_text: free pointer in case of error --- Modified: src/modules/db_text/dbt_base.c --- Diff: https://github.com/kamailio/kamailio/commit/bddf16db74ea779bb01805fc2cbabfd… Patch: https://github.com/kamailio/kamailio/commit/bddf16db74ea779bb01805fc2cbabfd… --- diff --git a/src/modules/db_text/dbt_base.c b/src/modules/db_text/dbt_base.c index d6d9d73f21..1e5dd01514 100644 --- a/src/modules/db_text/dbt_base.c +++ b/src/modules/db_text/dbt_base.c @@ -100,6 +100,7 @@ db1_con_t* dbt_init(const str* _sqlurl) if (!DBT_CON_CONNECTION(_res)) { LM_ERR("cannot get the link to database\n"); + pkg_free(_res); return NULL; }

3 years, 9 months

git:master:6ce5c0bd: xcap_server: check before going to the end of the buffer

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 6ce5c0bdf136ecffd7917f2d60093e1b2c49fa54 URL: https://github.com/kamailio/kamailio/commit/6ce5c0bdf136ecffd7917f2d60093e1… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-02T17:08:49+02:00 xcap_server: check before going to the end of the buffer - use marcros to set and restore char for ending 0 --- Modified: src/modules/xcap_server/xcap_server.c --- Diff: https://github.com/kamailio/kamailio/commit/6ce5c0bdf136ecffd7917f2d60093e1… Patch: https://github.com/kamailio/kamailio/commit/6ce5c0bdf136ecffd7917f2d60093e1… --- diff --git a/src/modules/xcap_server/xcap_server.c b/src/modules/xcap_server/xcap_server.c index ec4a3d1975..1556aa14b9 100644 --- a/src/modules/xcap_server/xcap_server.c +++ b/src/modules/xcap_server/xcap_server.c @@ -321,14 +321,12 @@ int xcaps_xpath_hack(str *buf, int type) } start = buf->s; - c = buf->s[buf->len-1]; - buf->s[buf->len-1] = '\0'; - while((p = strstr(start, match))!=NULL) - { + STR_VTOZ(buf->s[buf->len-1], c); + while((start < buf->s + buf->len - 8) && (p = strstr(start, match))!=NULL) { memcpy(p, repl, 7); start = p + 7; } - buf->s[buf->len-1] = c; + STR_ZTOV(buf->s[buf->len-1], c); return 0; }

3 years, 9 months

git:5.4:73c97ee8: pua: update_version_puadb() use pres_id to select record

by Victor Seva

Module: kamailio Branch: 5.4 Commit: 73c97ee8ca89bea202b09f6b75722396fbe05ba6 URL: https://github.com/kamailio/kamailio/commit/73c97ee8ca89bea202b09f6b7572239… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-08-01T10:44:00+02:00 pua: update_version_puadb() use pres_id to select record * only relevant when db_mode is PUA_DB_ONLY * call_id/to_tag/from_tag values can be "", for instance with DIALOG_PUBLISH.* records. Then **ALL** records get version field update * update_vesion_puadb() is called from send_publish() and pres->id value is valid after a call to get_record_puadb() related to #2414 (cherry picked from commit e4aed5c272c8144dd1ddf58163c3ba501bb46a5b) --- Modified: src/modules/pua/pua_db.c --- Diff: https://github.com/kamailio/kamailio/commit/73c97ee8ca89bea202b09f6b7572239… Patch: https://github.com/kamailio/kamailio/commit/73c97ee8ca89bea202b09f6b7572239… --- diff --git a/src/modules/pua/pua_db.c b/src/modules/pua/pua_db.c index 9a5c34b8ec..9f9b36cab4 100644 --- a/src/modules/pua/pua_db.c +++ b/src/modules/pua/pua_db.c @@ -1485,8 +1485,8 @@ int update_contact_puadb(ua_pres_t *pres, str *contact) int update_version_puadb(ua_pres_t *pres) { - db_key_t q_cols[3], db_cols[1]; - db_val_t q_vals[3], db_vals[1]; + db_key_t q_cols[1], db_cols[1]; + db_val_t q_vals[1], db_vals[1]; int n_query_cols= 0, n_update_cols=0; if (pres==NULL) @@ -1496,22 +1496,10 @@ int update_version_puadb(ua_pres_t *pres) } /* cols and values used for search query */ - q_cols[n_query_cols] = &str_call_id_col; - q_vals[n_query_cols].type = DB1_STR; - q_vals[n_query_cols].nul = 0; - q_vals[n_query_cols].val.str_val = pres->call_id; - n_query_cols++; - - q_cols[n_query_cols] = &str_to_tag_col; - q_vals[n_query_cols].type = DB1_STR; - q_vals[n_query_cols].nul = 0; - q_vals[n_query_cols].val.str_val = pres->to_tag; - n_query_cols++; - - q_cols[n_query_cols] = &str_from_tag_col; + q_cols[n_query_cols] = &str_pres_id_col; q_vals[n_query_cols].type = DB1_STR; q_vals[n_query_cols].nul = 0; - q_vals[n_query_cols].val.str_val = pres->from_tag; + q_vals[n_query_cols].val.str_val = pres->id; n_query_cols++; /* we overwrite contact even if not changed */

3 years, 9 months

git:5.4:46b4cc31: pua: fix send_publish() behavior on PUA_DB_ONLY

by Victor Seva

Module: kamailio Branch: 5.4 Commit: 46b4cc31bb2cb0a513ddf283db5d0690b0b04f68 URL: https://github.com/kamailio/kamailio/commit/46b4cc31bb2cb0a513ddf283db5d069… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-08-01T10:43:46+02:00 pua: fix send_publish() behavior on PUA_DB_ONLY * dialog PUBLISH was missing SIP-If-Mach * pua was inserting a new record for every dialog state fixes #2414 (cherry picked from commit 91d9441a242da4746171bfa532fa2378328e8d73) --- Modified: src/modules/pua/send_publish.c --- Diff: https://github.com/kamailio/kamailio/commit/46b4cc31bb2cb0a513ddf283db5d069… Patch: https://github.com/kamailio/kamailio/commit/46b4cc31bb2cb0a513ddf283db5d069… --- diff --git a/src/modules/pua/send_publish.c b/src/modules/pua/send_publish.c index e88c342f5c..338ee892b4 100644 --- a/src/modules/pua/send_publish.c +++ b/src/modules/pua/send_publish.c @@ -516,14 +516,11 @@ int send_publish( publ_info_t* publ ) if (dbmode==PUA_DB_ONLY) { - if (publ->etag) { - memset(&dbpres, 0, sizeof(dbpres)); - dbpres.pres_uri = &pres_uri; - dbpres.watcher_uri = &watcher_uri; - dbpres.extra_headers = &extra_headers; - presentity = get_record_puadb(publ->id, publ->etag, - &dbpres, &res); - } + memset(&dbpres, 0, sizeof(dbpres)); + dbpres.pres_uri = &pres_uri; + dbpres.watcher_uri = &watcher_uri; + dbpres.extra_headers = &extra_headers; + presentity = get_record_puadb(publ->id, publ->etag, &dbpres, &res); } else {

3 years, 9 months

git:5.4:fefe5aa7: pua_bla: clang-format

by Victor Seva

Module: kamailio Branch: 5.4 Commit: fefe5aa727147de608ee9770b06b7f78b72ba4f9 URL: https://github.com/kamailio/kamailio/commit/fefe5aa727147de608ee9770b06b7f7… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-08-01T10:43:27+02:00 pua_bla: clang-format * use STR_NULL (cherry picked from commit e542b779b29bda2d0a953a4e55921edc466a3936) --- Modified: src/modules/pua_bla/notify.c Modified: src/modules/pua_bla/pua_bla.c Modified: src/modules/pua_bla/pua_bla.h Modified: src/modules/pua_bla/registrar_cb.c Modified: src/modules/pua_bla/registrar_cb.h --- Diff: https://github.com/kamailio/kamailio/commit/fefe5aa727147de608ee9770b06b7f7… Patch: https://github.com/kamailio/kamailio/commit/fefe5aa727147de608ee9770b06b7f7…

3 years, 9 months

git:5.4:20c9f514: pua_bla: convert to PKG/SHM memory logging helper functions

by Victor Seva

Module: kamailio Branch: 5.4 Commit: 20c9f514b61f5c0c41890650691483ae2b261007 URL: https://github.com/kamailio/kamailio/commit/20c9f514b61f5c0c41890650691483a… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-08-01T10:43:12+02:00 pua_bla: convert to PKG/SHM memory logging helper functions * remove trailing spaces * remove history notes and $Id (cherry picked from commit e4895abf5a533b96bf5e7112488a0facaabfc1a7) --- Modified: src/modules/pua_bla/notify.c Modified: src/modules/pua_bla/pua_bla.c Modified: src/modules/pua_bla/pua_bla.h Modified: src/modules/pua_bla/registrar_cb.c Modified: src/modules/pua_bla/registrar_cb.h --- Diff: https://github.com/kamailio/kamailio/commit/20c9f514b61f5c0c41890650691483a… Patch: https://github.com/kamailio/kamailio/commit/20c9f514b61f5c0c41890650691483a…

3 years, 9 months

git:5.4:b5d848e7: pua_usrloc: convert to PKG/SHM memory logging helper functions

by Victor Seva

Module: kamailio Branch: 5.4 Commit: b5d848e774814b0673c4fec1734d6f3de3e1c877 URL: https://github.com/kamailio/kamailio/commit/b5d848e774814b0673c4fec1734d6f3… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-08-01T10:42:53+02:00 pua_usrloc: convert to PKG/SHM memory logging helper functions (cherry picked from commit be07ed5dd927448ffad3bd77f976249a888cfb2e) --- Modified: src/modules/pua_usrloc/ul_publish.c --- Diff: https://github.com/kamailio/kamailio/commit/b5d848e774814b0673c4fec1734d6f3… Patch: https://github.com/kamailio/kamailio/commit/b5d848e774814b0673c4fec1734d6f3… --- diff --git a/src/modules/pua_usrloc/ul_publish.c b/src/modules/pua_usrloc/ul_publish.c index 0afd6d4201..cb8af0c7fa 100644 --- a/src/modules/pua_usrloc/ul_publish.c +++ b/src/modules/pua_usrloc/ul_publish.c @@ -174,7 +174,7 @@ str* build_pidf(ucontact_t* c) body = (str*)pkg_malloc(sizeof(str)); if(body == NULL) { - LM_ERR("while allocating memory\n"); + PKG_MEM_ERROR; return NULL; } memset(body, 0, sizeof(str)); @@ -253,8 +253,10 @@ void ul_publish(ucontact_t* c, int type, void* param) body = NULL; uri.s = (char*)pkg_malloc(sizeof(char)*(c->aor->len+default_domain.len+6)); - if(uri.s == NULL) + if(uri.s == NULL) { + PKG_MEM_ERROR; goto error; + } memcpy(uri.s, "sip:", 4); uri.len = 4; @@ -278,7 +280,7 @@ void ul_publish(ucontact_t* c, int type, void* param) publ= (publ_info_t*)pkg_malloc(size); if(publ== NULL) { - LM_ERR("no more share memory\n"); + SHM_MEM_ERROR; goto error; } memset(publ, 0, size);

3 years, 9 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2020