sr-dev

sr-dev@lists.kamailio.org

13 participants
45255 discussions

git:master:9a2fea75: app_sqlang: use file name specific include guard
by Daniel-Constantin Mierla 05 May '20

05 May '20

Module: kamailio Branch: master Commit: 9a2fea75dcee851b1dc22e5a9e7a6ee0d05814ef URL: https://github.com/kamailio/kamailio/commit/9a2fea75dcee851b1dc22e5a9e7a6ee… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-05T07:59:32+02:00 app_sqlang: use file name specific include guard --- Modified: src/modules/app_sqlang/app_sqlang_api.h --- Diff: https://github.com/kamailio/kamailio/commit/9a2fea75dcee851b1dc22e5a9e7a6ee… Patch: https://github.com/kamailio/kamailio/commit/9a2fea75dcee851b1dc22e5a9e7a6ee… --- diff --git a/src/modules/app_sqlang/app_sqlang_api.h b/src/modules/app_sqlang/app_sqlang_api.h index 31c824a99d..eae6de31cf 100644 --- a/src/modules/app_sqlang/app_sqlang_api.h +++ b/src/modules/app_sqlang/app_sqlang_api.h @@ -20,8 +20,8 @@ * */ -#ifndef _APP_JSDT_API_H_ -#define _APP_JSDT_API_H_ +#ifndef _APP_SQLANG_API_H_ +#define _APP_SQLANG_API_H_ #include <squirrel.h>

1 0

git:master:de77beff: rtpmanage: enable hadling of PRACK requests
by Daniel-Constantin Mierla 04 May '20

04 May '20

Module: kamailio Branch: master Commit: de77beffb436e783031a6aa71c1887848cb2cc73 URL: https://github.com/kamailio/kamailio/commit/de77beffb436e783031a6aa71c18878… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-04T20:05:26+02:00 rtpmanage: enable hadling of PRACK requests - they can have SDP - use more compact bitwise match for method types --- Modified: src/modules/rtpengine/rtpengine.c --- Diff: https://github.com/kamailio/kamailio/commit/de77beffb436e783031a6aa71c18878… Patch: https://github.com/kamailio/kamailio/commit/de77beffb436e783031a6aa71c18878… --- diff --git a/src/modules/rtpengine/rtpengine.c b/src/modules/rtpengine/rtpengine.c index d56e88e003..2d28895324 100644 --- a/src/modules/rtpengine/rtpengine.c +++ b/src/modules/rtpengine/rtpengine.c @@ -3634,11 +3634,11 @@ rtpengine_manage(struct sip_msg *msg, const char *flags) method = get_cseq(msg)->method_id; - if (!(method==METHOD_INVITE || method==METHOD_ACK || method==METHOD_CANCEL - || method==METHOD_BYE || method==METHOD_UPDATE)) + if (!(method & (METHOD_INVITE|METHOD_ACK|METHOD_CANCEL|METHOD_BYE + |METHOD_UPDATE|METHOD_PRACK))) return -1; - if (method==METHOD_CANCEL || method==METHOD_BYE) + if (method & (METHOD_CANCEL|METHOD_BYE)) return rtpengine_delete(msg, flags); if (msg->msg_flags & FL_SDP_BODY) @@ -3647,7 +3647,7 @@ rtpengine_manage(struct sip_msg *msg, const char *flags) nosdp = parse_sdp(msg); if (msg->first_line.type == SIP_REQUEST) { - if(method==METHOD_ACK && nosdp==0) + if((method & (METHOD_ACK|METHOD_PRACK)) && nosdp==0) return rtpengine_offer_answer(msg, flags, OP_ANSWER, 0); if(method==METHOD_UPDATE && nosdp==0) return rtpengine_offer_answer(msg, flags, OP_OFFER, 0); @@ -3715,7 +3715,8 @@ rtpengine_answer1_f(struct sip_msg *msg, char *str1, char *str2) { if (msg->first_line.type == SIP_REQUEST) - if (msg->first_line.u.request.method_value != METHOD_ACK) + if (!(msg->first_line.u.request.method_value + & (METHOD_ACK | METHOD_PRACK))) return -1; return rtpengine_rtpp_set_wrap_fparam(msg, rtpengine_answer_wrap, str1, 2, OP_ANSWER);

1 0

git:master:f4f66ad9: usrloc: use file name specific include guard
by Daniel-Constantin Mierla 04 May '20

04 May '20

Module: kamailio Branch: master Commit: f4f66ad91324116e5d4e9e02fa9caedcad2b2d2f URL: https://github.com/kamailio/kamailio/commit/f4f66ad91324116e5d4e9e02fa9caed… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-04T19:38:07+02:00 usrloc: use file name specific include guard --- Modified: src/modules/usrloc/usrloc_mod.h --- Diff: https://github.com/kamailio/kamailio/commit/f4f66ad91324116e5d4e9e02fa9caed… Patch: https://github.com/kamailio/kamailio/commit/f4f66ad91324116e5d4e9e02fa9caed… --- diff --git a/src/modules/usrloc/usrloc_mod.h b/src/modules/usrloc/usrloc_mod.h index 6c3f6e078f..8c24e1230b 100644 --- a/src/modules/usrloc/usrloc_mod.h +++ b/src/modules/usrloc/usrloc_mod.h @@ -26,8 +26,8 @@ * \ingroup usrloc */ -#ifndef UL_MOD_H -#define UL_MOD_H +#ifndef _USRLOC_MOD_H_ +#define _USRLOC_MOD_H_ #include "../../lib/srdb1/db.h" @@ -129,4 +129,4 @@ extern int ul_expires_type; #define UL_DB_EXPIRES_GET(r) ((ul_expires_type==1)?(time_t)VAL_BIGINT(r):VAL_TIME(r)) -#endif /* UL_MOD_H */ +#endif /* _USRLOC_MOD_H_ */

1 0

git:master:b3c981db: uid_domain: use module specific include guards
by Daniel-Constantin Mierla 04 May '20

04 May '20

Module: kamailio Branch: master Commit: b3c981db67845622fb8c85a20e18e1b98c29eb69 URL: https://github.com/kamailio/kamailio/commit/b3c981db67845622fb8c85a20e18e1b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-04T19:01:21+02:00 uid_domain: use module specific include guards --- Modified: src/modules/uid_domain/domain.h Modified: src/modules/uid_domain/domain_api.h Modified: src/modules/uid_domain/domain_rpc.h Modified: src/modules/uid_domain/hash.h Modified: src/modules/uid_domain/uid_domain_mod.h --- Diff: https://github.com/kamailio/kamailio/commit/b3c981db67845622fb8c85a20e18e1b… Patch: https://github.com/kamailio/kamailio/commit/b3c981db67845622fb8c85a20e18e1b… --- diff --git a/src/modules/uid_domain/domain.h b/src/modules/uid_domain/domain.h index 44ad7ee4db..c52417a601 100644 --- a/src/modules/uid_domain/domain.h +++ b/src/modules/uid_domain/domain.h @@ -19,8 +19,8 @@ * with this program; if not, write to the Free Software Foundation, Inc., * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#ifndef _DOMAIN_H -#define _DOMAIN_H +#ifndef _UID_DOMAIN_H_ +#define _UID_DOMAIN_H_ #include <time.h> #include <stdio.h> diff --git a/src/modules/uid_domain/domain_api.h b/src/modules/uid_domain/domain_api.h index acc957971f..4a64a469cb 100644 --- a/src/modules/uid_domain/domain_api.h +++ b/src/modules/uid_domain/domain_api.h @@ -20,8 +20,8 @@ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#ifndef _DOMAIN_API_H -#define _DOMAIN_API_H +#ifndef _UID_DOMAIN_API_H_ +#define _UID_DOMAIN_API_H_ #include "../../core/sr_module.h" #include "../../core/dprint.h" diff --git a/src/modules/uid_domain/domain_rpc.h b/src/modules/uid_domain/domain_rpc.h index 27641b6d47..996e7f01aa 100644 --- a/src/modules/uid_domain/domain_rpc.h +++ b/src/modules/uid_domain/domain_rpc.h @@ -20,8 +20,8 @@ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#ifndef _DOMAIN_RPC_H -#define _DOMAIN_RPC_H +#ifndef _UID_DOMAIN_RPC_H_ +#define _UID_DOMAIN_RPC_H_ #include "../../core/rpc.h" diff --git a/src/modules/uid_domain/hash.h b/src/modules/uid_domain/hash.h index e72616f775..1c07a83321 100644 --- a/src/modules/uid_domain/hash.h +++ b/src/modules/uid_domain/hash.h @@ -20,8 +20,8 @@ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#ifndef _HASH_H -#define _HASH_H +#ifndef _UID_DOMAIN_HASH_H_ +#define _UID_DOMAIN_HASH_H_ #include <stdio.h> #include "uid_domain_mod.h" diff --git a/src/modules/uid_domain/uid_domain_mod.h b/src/modules/uid_domain/uid_domain_mod.h index 4f15b6d0ef..75c21c2175 100644 --- a/src/modules/uid_domain/uid_domain_mod.h +++ b/src/modules/uid_domain/uid_domain_mod.h @@ -20,8 +20,8 @@ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#ifndef _UID_DOMAIN_MOD_H -#define _UID_DOMAIN_MOD_H +#ifndef _UID_DOMAIN_MOD_H_ +#define _UID_DOMAIN_MOD_H_ #include "../../lib/srdb2/db.h" #include "../../core/str.h"

1 0

git:master:c01633dc: libs/srdb1: use thread safe localtime_r()
by Daniel-Constantin Mierla 04 May '20

04 May '20

Module: kamailio Branch: master Commit: c01633dc54d8d4dc3b3bc2cab064f91a49c8257d URL: https://github.com/kamailio/kamailio/commit/c01633dc54d8d4dc3b3bc2cab064f91… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-04T18:26:47+02:00 libs/srdb1: use thread safe localtime_r() --- Modified: src/lib/srdb1/db_ut.c --- Diff: https://github.com/kamailio/kamailio/commit/c01633dc54d8d4dc3b3bc2cab064f91… Patch: https://github.com/kamailio/kamailio/commit/c01633dc54d8d4dc3b3bc2cab064f91… --- diff --git a/src/lib/srdb1/db_ut.c b/src/lib/srdb1/db_ut.c index 39ac11d379..318b442e44 100644 --- a/src/lib/srdb1/db_ut.c +++ b/src/lib/srdb1/db_ut.c @@ -350,7 +350,7 @@ int db_str2time(const char* _s, time_t* _v) */ int db_time2str_ex(time_t _v, char* _s, int* _l, int _qmode) { - struct tm* t; + struct tm t; int l; if ((!_s) || (!_l) || (*_l < 2)) { @@ -361,8 +361,8 @@ int db_time2str_ex(time_t _v, char* _s, int* _l, int _qmode) if(_qmode) *_s++ = '\''; /* Convert time_t structure to format accepted by the database */ - t = localtime(&_v); - l = strftime(_s, *_l -1, "%Y-%m-%d %H:%M:%S", t); + localtime_r(&_v, &t); + l = strftime(_s, *_l -1, "%Y-%m-%d %H:%M:%S", &t); if (l == 0) { LM_ERR("Error during time conversion\n");

1 0

git:master:9691f6e0: core: hash_func.h - use file name specific include guard
by Daniel-Constantin Mierla 04 May '20

04 May '20

Module: kamailio Branch: master Commit: 9691f6e0cab48114ef4067a22cc8f9c1efb8944c URL: https://github.com/kamailio/kamailio/commit/9691f6e0cab48114ef4067a22cc8f9c… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-04T18:35:00+02:00 core: hash_func.h - use file name specific include guard --- Modified: src/core/hash_func.h --- Diff: https://github.com/kamailio/kamailio/commit/9691f6e0cab48114ef4067a22cc8f9c… Patch: https://github.com/kamailio/kamailio/commit/9691f6e0cab48114ef4067a22cc8f9c… --- diff --git a/src/core/hash_func.h b/src/core/hash_func.h index 203e10b1c8..aeb9794bf2 100644 --- a/src/core/hash_func.h +++ b/src/core/hash_func.h @@ -13,8 +13,8 @@ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ /*! @@ -26,14 +26,14 @@ -#ifndef _HASH_H -#define _HASH_H +#ifndef _HASH_FUNC_H_ +#define _HASH_FUNC_H_ #include "str.h" #include "hashes.h" /* always use a power of 2 for hash table size */ -#define T_TABLE_POWER 16 +#define T_TABLE_POWER 16 #define TABLE_ENTRIES (1 << (T_TABLE_POWER)) unsigned int new_hash( str call_id, str cseq_nr );

1 0

git:5.2:4061a1d7: utils/kamctl: dbtextdb.py close previous opened file properly
by Victor Seva 04 May '20

04 May '20

Module: kamailio Branch: 5.2 Commit: 4061a1d7cd130a5c3e4236c2abb999658ee02e36 URL: https://github.com/kamailio/kamailio/commit/4061a1d7cd130a5c3e4236c2abb9996… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-05-04T17:07:58+02:00 utils/kamctl: dbtextdb.py close previous opened file properly related #1747 (cherry picked from commit 5f52f9905bbdf63d636080f7d0340af4f71ed12a) (cherry picked from commit e26e587f31eb23c431a2cfb149ec7a86b4f648bc) --- Modified: utils/kamctl/dbtextdb/dbtextdb.py --- Diff: https://github.com/kamailio/kamailio/commit/4061a1d7cd130a5c3e4236c2abb9996… Patch: https://github.com/kamailio/kamailio/commit/4061a1d7cd130a5c3e4236c2abb9996… --- diff --git a/utils/kamctl/dbtextdb/dbtextdb.py b/utils/kamctl/dbtextdb/dbtextdb.py index 7e1de04687..581496c309 100755 --- a/utils/kamctl/dbtextdb/dbtextdb.py +++ b/utils/kamctl/dbtextdb/dbtextdb.py @@ -60,6 +60,10 @@ def __init__(self, location): if not os.path.isdir(location): raise ParseError(location + ' is not a directory') + def __del__(self): + if getattr(self, 'fd', False): + self.fd.close() + def _ParseOrderBy(self): """Parse out the column name to be used for ordering the dataset. @@ -438,6 +442,8 @@ def CleanUp(self): self.command = '' # which command are we executing self.strings = [] # list of string literals parsed from the query self.parens = [] # list of parentheses parsed from the query + if getattr(self, 'fd', False): + self.fd.close() def ParseQuery(self, query): """External wrapper for the query parsing routines.

1 0

git:5.2:0d9c72b2: utils/kamctl: dbtextdb remove DeprecationWarning on test
by Victor Seva 04 May '20

04 May '20

Module: kamailio Branch: 5.2 Commit: 0d9c72b27d75fb2e4ec879e855ce141394ce1918 URL: https://github.com/kamailio/kamailio/commit/0d9c72b27d75fb2e4ec879e855ce141… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-05-04T17:07:05+02:00 utils/kamctl: dbtextdb remove DeprecationWarning on test related #1747 (cherry picked from commit 765efa80c7d27e9447bc1ad8a354ee24723580ef) (cherry picked from commit 5a90c8bc600fe9a93a5ce4f828ce50f3bf532b99) --- Modified: utils/kamctl/dbtextdb/dbtextdb_test.py --- Diff: https://github.com/kamailio/kamailio/commit/0d9c72b27d75fb2e4ec879e855ce141… Patch: https://github.com/kamailio/kamailio/commit/0d9c72b27d75fb2e4ec879e855ce141…

1 0

git:5.2:23062ed5: utils/kamctl: fix handling of Exceptions
by Victor Seva 04 May '20

04 May '20

Module: kamailio Branch: 5.2 Commit: 23062ed50ea4e6b6e00468cc7f8ec57ccc613429 URL: https://github.com/kamailio/kamailio/commit/23062ed50ea4e6b6e00468cc7f8ec57… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-05-04T17:07:04+02:00 utils/kamctl: fix handling of Exceptions > Traceback (most recent call last): > File "/usr/lib/x86_64-linux-gnu/kamailio/kamctl/dbtextdb/dbtextdb.py", line 1239, in <module> > main(sys.argv) > File "/usr/lib/x86_64-linux-gnu/kamailio/kamctl/dbtextdb/dbtextdb.py", line 1233, in main > except (Error, e): > NameError: name 'e' is not defined (cherry picked from commit 1aca79c43cb8dbd920c1eb81c0c30cf89f0985b9) (cherry picked from commit 5f6f4331811e654e75b49263f8fb823bea9874a3) --- Modified: utils/kamctl/dbtextdb/dbtextdb.py --- Diff: https://github.com/kamailio/kamailio/commit/23062ed50ea4e6b6e00468cc7f8ec57… Patch: https://github.com/kamailio/kamailio/commit/23062ed50ea4e6b6e00468cc7f8ec57… --- diff --git a/utils/kamctl/dbtextdb/dbtextdb.py b/utils/kamctl/dbtextdb/dbtextdb.py index 3a35554013..3d5ee7ecc5 100755 --- a/utils/kamctl/dbtextdb/dbtextdb.py +++ b/utils/kamctl/dbtextdb/dbtextdb.py @@ -356,7 +356,7 @@ def _EscapeChars(self, value): # test that the value is string, if not return it as is try: value.find('a') - except: + except Exception: return value escaped = value @@ -377,7 +377,7 @@ def _UnEscapeChars(self, value): # test that the value is string, if not return it as is try: value.find('a') - except: + except Exception: return value escaped = value @@ -988,21 +988,19 @@ def _TypeCheck(self, val, col): if not val and not self.header[col]['null']: raise ExecuteError(col + ' cannot be empty or null') - if (self.header[col]['type'].lower() == 'int' or - self.header[col]['type'].lower() == 'double'): + hdr_t = self.header[col]['type'].lower() + if hdr_t == 'int' or hdr_t == 'double': try: if val: val = eval(val) - except (NameError, e): + except NameError as e: raise ExecuteError('Failed to parse %s in %s ' '(unable to convert to type %s): %s' % - (col, self.table, self.header[col]['type'], - e)) - except (SyntaxError, e): + (col, self.table, hdr_t, e)) + except SyntaxError as e: raise ExecuteError('Failed to parse %s in %s ' '(unable to convert to type %s): %s' % - (col, self.table, self.header[col]['type'], - e)) + (col, self.table, hdr_t, e)) return val @@ -1083,7 +1081,7 @@ def OpenTable(self): # save a copy of the data before modifying self.orig_data = self.data[:] - except (IOError, e): + except IOError as e: raise ExecuteError('Unable to open table %s: %s' % (self.table, e)) Debug('Header is: %s' % self.header) @@ -1230,7 +1228,7 @@ def main(argv): print('Updated %s, rows affected: %d' % (conn.table, row)) else: print(row) - except (Error, e): + except Error as e: print(e) sys.exit(1)

1 0

git:5.2:a2660fbe: utils/kamctl: fix dbtestdb
by Victor Seva 04 May '20

04 May '20

Module: kamailio Branch: 5.2 Commit: a2660fbe2e974553f0146c9ee84fcc6fa2f8f705 URL: https://github.com/kamailio/kamailio/commit/a2660fbe2e974553f0146c9ee84fcc6… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2020-05-04T17:07:05+02:00 utils/kamctl: fix dbtestdb some indentantion errors introduced at https://github.com/kamailio/kamailio/commit/bc8bdcaa726f375f6deea8228a1ab0c… related #1747 (cherry picked from commit b722a48144ac9e029bdbc2968dedbe56966a3f7c) (cherry picked from commit c4dd3218e6b723c688762a5c4253fa70f3f588bc) --- Modified: utils/kamctl/dbtextdb/dbtextdb.py --- Diff: https://github.com/kamailio/kamailio/commit/a2660fbe2e974553f0146c9ee84fcc6… Patch: https://github.com/kamailio/kamailio/commit/a2660fbe2e974553f0146c9ee84fcc6… --- diff --git a/utils/kamctl/dbtextdb/dbtextdb.py b/utils/kamctl/dbtextdb/dbtextdb.py index 3d5ee7ecc5..7e1de04687 100755 --- a/utils/kamctl/dbtextdb/dbtextdb.py +++ b/utils/kamctl/dbtextdb/dbtextdb.py @@ -146,11 +146,11 @@ def _ParseColumns(self): # check if there is a function modifier on the columns if self.tokens[0] == 'COUNT': self.count = True - if col_end == 1: - raise ParseError('COUNT must be followed by column name[s]') - if not self.tokens[1].startswith(self._paren_placeholder): - raise ParseError('COUNT must be followed by ()') - cols_str = self._ReplaceParens(self.tokens[1]) + if col_end == 1: + raise ParseError('COUNT must be followed by column name[s]') + if not self.tokens[1].startswith(self._paren_placeholder): + raise ParseError('COUNT must be followed by ()') + cols_str = self._ReplaceParens(self.tokens[1]) cols = cols_str.split(',') for col in cols: @@ -300,7 +300,7 @@ def _ParseTargets(self): if self.tokens.pop(0) != 'SET': raise ParseError('UPDATE command must be followed by SET') - self.targets = self._ParsePairs(' '.join(self.tokens), ',') + self.targets = self._ParsePairs(' '.join(self.tokens), ',') # INSERT if self.command == 'INSERT': @@ -570,19 +570,19 @@ def _ParseOutHelper(self, placeholder, delims, mode): string = '%s%s' % (string, c) continue # wait for matching delim - started -= 1 - if not started: - values.append(string) - new_args = '%s %s' % (new_args, '%s%d' % (placeholder, - my_id)) - my_id += 1 - string = '' + started -= 1 + if not started: + values.append(string) + new_args = '%s %s' % (new_args, '%s%d' % (placeholder, + my_id)) + my_id += 1 + string = '' - else: - if not started: - new_args = '%s%s' % (new_args, c) else: - string = '%s%s' % (string, c) + if not started: + new_args = '%s%s' % (new_args, c) + else: + string = '%s%s' % (string, c) if started: if mode == 'parens': @@ -714,8 +714,8 @@ def _RunInsert(self): elif self.header[col]['auto']: new_row[col] = self._GetNextAuto(col) - else: - raise ExecuteError(col + ' cannot be empty or null') + else: + raise ExecuteError(col + ' cannot be empty or null') self.data.append(new_row) return [1]

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev