sr-dev

sr-dev@lists.kamailio.org

12 participants
44199 discussions

git:janakj/postgres: - fix regression during value conversion when input SQL string is NULL,

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: fd355ee0b0d815fd42a0a4512566802f63c924f6 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=fd355ee… Author: Henning Westerholt <henning.westerholt(a)1und1.de> Committer: Henning Westerholt <henning.westerholt(a)1und1.de> Date: Wed Jan 14 13:58:51 2009 +0000 - fix regression during value conversion when input SQL string is NULL, and value type is BLOB, reported from jordan at flowroute dot com git-svn-id: https://openser.svn.sourceforge.net/svnroot/openser/trunk@5459 689a6050-402a-0410-94f2-e92a70836424 --- modules/db_postgres/km_val.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/modules/db_postgres/km_val.c b/modules/db_postgres/km_val.c index 7cc308a..e47467e 100644 --- a/modules/db_postgres/km_val.c +++ b/modules/db_postgres/km_val.c @@ -57,7 +57,8 @@ */ int db_postgres_str2val(const db_type_t _t, db_val_t* _v, const char* _s, const int _l) { - if ( (_t != DB_BLOB && _v != NULL) || _v == NULL) { + /* use common function for non BLOB, NULL setting and input parameter checking */ + if ( _t != DB_BLOB || _s == NULL || _v == NULL) { return db_str2val(_t, _v, _s, _l, 1); } else { char * tmp_s = NULL;

15 years, 10 months

git:janakj/postgres: Align defines protecting headers with filenames.

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: d5d6356037c2684425108ec4a4105a3453713f0e URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d5d6356… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Wed Feb 18 00:39:53 2009 +0100 Align defines protecting headers with filenames. --- modules/db_postgres/km_dbase.h | 6 +++--- modules/db_postgres/km_pg_con.h | 6 +++--- modules/db_postgres/km_res.h | 4 ++-- modules/db_postgres/km_val.h | 4 ++-- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/modules/db_postgres/km_dbase.h b/modules/db_postgres/km_dbase.h index ad05f54..531ec59 100644 --- a/modules/db_postgres/km_dbase.h +++ b/modules/db_postgres/km_dbase.h @@ -31,8 +31,8 @@ * Module: \ref db_postgres */ -#ifndef DBASE_H -#define DBASE_H +#ifndef KM_DBASE_H +#define KM_DBASE_H #include "../../db/db_con.h" #include "../../db/db_res.h" @@ -110,4 +110,4 @@ int db_postgres_fetch_result(const db_con_t* _h, db_res_t** _r, const int nrows) int db_postgres_use_table(db_con_t* _h, const str* _t); -#endif /* DBASE_H */ +#endif /* KM_DBASE_H */ diff --git a/modules/db_postgres/km_pg_con.h b/modules/db_postgres/km_pg_con.h index c7cf63e..ee46846 100644 --- a/modules/db_postgres/km_pg_con.h +++ b/modules/db_postgres/km_pg_con.h @@ -31,8 +31,8 @@ * Module: \ref db_mysql */ -#ifndef PG_CON_H -#define PG_CON_H +#ifndef KM_PG_CON_H +#define KM_PG_CON_H #include "../../db/db_pool.h" #include "../../db/db_id.h" @@ -75,4 +75,4 @@ struct pg_con* db_postgres_new_connection(struct db_id* id); */ void db_postgres_free_connection(struct pool_con* con); -#endif /* PG_CON_H */ +#endif /* KM_PG_CON_H */ diff --git a/modules/db_postgres/km_res.h b/modules/db_postgres/km_res.h index 5b2627f..959138e 100644 --- a/modules/db_postgres/km_res.h +++ b/modules/db_postgres/km_res.h @@ -27,8 +27,8 @@ * Module: \ref db_postgres */ -#ifndef DB_PG_RES_H -#define DB_PG_RES_H +#ifndef KM_RES_H +#define KM_RES_H #include "../../db/db_row.h" diff --git a/modules/db_postgres/km_val.h b/modules/db_postgres/km_val.h index cbee53a..6e0b253 100644 --- a/modules/db_postgres/km_val.h +++ b/modules/db_postgres/km_val.h @@ -26,8 +26,8 @@ * Module: \ref db_postgres */ -#ifndef DB_PG_VAL_H -#define DB_PG_VAL_H +#ifndef KM_VAL_H +#define KM_VAL_H int db_postgres_str2val(const db_type_t _t, db_val_t* _v, const char* _s, const int _l);

15 years, 10 months

git:janakj/postgres: Link with both srdb1 and srdb2

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: b6d503bca6d9cb6f0db78f80601bb9dbba7b0a91 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b6d503b… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Tue Feb 17 23:47:02 2009 +0100 Link with both srdb1 and srdb2 --- modules/db_postgres/Makefile | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules/db_postgres/Makefile b/modules/db_postgres/Makefile index 48857cb..f32f99b 100644 --- a/modules/db_postgres/Makefile +++ b/modules/db_postgres/Makefile @@ -15,6 +15,6 @@ DEFS +=-DSER_MOD_INTERFACE -I$(LOCALBASE)/include -I$(LOCALBASE)/pgsql/include LIBS=-L$(LOCALBASE)/lib -L$(LOCALBASE)/pgsql/lib -L$(LOCALBASE)/lib/pgsql -L /usr/lib -lpq SERLIBPATH=../../lib -SER_LIBS=$(SERLIBPATH)/srdb2/srdb2 +SER_LIBS=$(SERLIBPATH)/srdb2/srdb2 $(SERLIBPATH)/srdb1/srdb1 include ../../Makefile.modules

15 years, 10 months

git:janakj/postgres: Removed km_Makefile and km_README.

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: 1a3eac890f65ad6312fc28bffa4a800304524060 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1a3eac8… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Wed Feb 18 00:51:43 2009 +0100 Removed km_Makefile and km_README. We will integrate the contents of km_Makefile into Makefile. Documentation will be merged later. --- modules/db_postgres/km_Makefile | 36 ---------------------- modules/db_postgres/km_README | 62 --------------------------------------- 2 files changed, 0 insertions(+), 98 deletions(-) diff --git a/modules/db_postgres/km_Makefile b/modules/db_postgres/km_Makefile deleted file mode 100644 index 699f9be..0000000 --- a/modules/db_postgres/km_Makefile +++ /dev/null @@ -1,36 +0,0 @@ -# $Id$ -# -# WARNING: do not run this directly, it should be run by the master Makefile - -include ../../Makefile.defs -auto_gen= -NAME=db_postgres.so - - -# set CROSS_COMPILE to true if you want to skip -# the autodetection -# CROSS_COMPILE=true - -ifeq ($(CROSS_COMPILE),) -PGCFG=$(shell which pg_config) -endif - -ifneq ($(PGCFG),) - - # use autodetection - DEFS += -I$(shell $(PGCFG) --includedir) - LIBS = -L$(shell $(PGCFG) --libdir) -lpq - -else - - # use standard know paths - # libpq-fe.h locations - DEFS +=-I$(LOCALBASE)/include -I$(LOCALBASE)/pgsql/include \ - -I$(SYSBASE)/include/pgsql -I$(SYSBASE)/include/postgresql \ - -I$(SYSBASE)/include/postgresql/8.0 - LIBS=-L$(LOCALBASE)/lib -L$(LOCALBASE)/pgsql/lib -L$(LOCALBASE)/lib/pgsql \ - -lpq - -endif - -include ../../Makefile.modules diff --git a/modules/db_postgres/km_README b/modules/db_postgres/km_README deleted file mode 100644 index 3a5e085..0000000 --- a/modules/db_postgres/km_README +++ /dev/null @@ -1,62 +0,0 @@ -postgres Module - -Greg Fausak - - August.net - -Edited by - -Greg Fausak - - Copyright � 2003 Greg Fausak - Revision History - Revision $Revision$ $Date: 2008-08-06 12:08:33 +0200 - (Mi, 06 Aug 2008) $ - __________________________________________________________ - - Table of Contents - - 1. Admin Guide - - 1.1. Overview - 1.2. Dependencies - - 1.2.1. Kamailio Modules - 1.2.2. External Libraries or Applications - - 1.3. Exported Parameters - 1.4. Exported Functions - 1.5. Installation and Running - -Chapter 1. Admin Guide - -1.1. Overview - - Module description - -1.2. Dependencies - -1.2.1. Kamailio Modules - - The following modules must be loaded before this module: - * No dependencies on other Kamailio modules. - -1.2.2. External Libraries or Applications - - The following libraries or applications must be installed - before running Kamailio with this module loaded: - * PostgreSQL library - e.g., libpq5. - * PostgreSQL devel library - to compile the module (e.g., - libpq-dev). - -1.3. Exported Parameters - - NONE - -1.4. Exported Functions - - NONE - -1.5. Installation and Running - - Notes about installation and running.

15 years, 10 months

git:janakj/postgres: Merge db_postgres module from kamailio/ trunk into db_postgres modules

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: bd30d4f936fabd4cb273fc09d9d82fe08200f5e3 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=bd30d4f… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Tue Feb 17 23:20:09 2009 +0100 Merge db_postgres module from kamailio/trunk into db_postgres modules in sip-router. * kamailio/db_postgres: (58 commits) - fix regression during value conversion when input SQL string is NULL, - another error condition fix for a problem that gets introduced by a - add one DBG log to each drivers error condition, that the mem is freed - improve two errors messages, inform the user that the query is aborted - db_postgres_convert_rows needs to free the row_buf on all error conditions - partial revert of commit rev5359 for db_mysql module - fix a few line breaks in errors logs - unify common rows and row allocation functionality in the DB API core - remove LM_ERR probe that slipped into previous commit - change behaviour of db_str2val, this now copy strings - add 'db_postgres' prefix to free_query function - fix a bunch of doxygen errors (mostly in modules, some in the core) - fix a bunch of errors in doxygen - fix memory leak in db_postgres module related to BLOBs (also caused an - fix postgres NULL value behaviour: as in postgres a NULL value is - docs extension: explain fetch_result functionality better to prevent errors - remove not reached return statement at the end of val2str functions - make small wrapper around PQclear void, nobody checks the return state - doxygen conversion, write new documentation, small cleanups - initial support for BIGINT database in DB core and SQL based database ... ---

15 years, 10 months

git:janakj/postgres: Various changes to get rid of conflicts and make km_ files compile.

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: a7bac3b9ab8829dcb2cd933cf3302adaf8f81cfa URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a7bac3b… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Wed Feb 18 01:22:22 2009 +0100 Various changes to get rid of conflicts and make km_ files compile. List of changes: * Path to header files updated to point to lib/srdb1 * Comment out MODULE_VERSION (already used pg_mod.c) * cmd_export_t renamed to kam_cmd_export_t * exports structure renamed to kam_exports (exports symbol is already defined by pg_mod.c) * module_exports renamed to kam_module_exports * db_con_t renamed to db1_con_t * db_res_t renamed to db1_res_t * DB field types renamed to DB1_* --- modules/db_postgres/km_db_postgres.c | 10 +++--- modules/db_postgres/km_dbase.c | 42 +++++++++++++++--------------- modules/db_postgres/km_dbase.h | 34 ++++++++++++------------ modules/db_postgres/km_pg_con.h | 4 +- modules/db_postgres/km_res.c | 48 +++++++++++++++++----------------- modules/db_postgres/km_res.h | 10 +++--- modules/db_postgres/km_val.c | 16 +++++----- modules/db_postgres/km_val.h | 2 +- 8 files changed, 83 insertions(+), 83 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=a7b…

15 years, 10 months

git:janakj/postgres: Module renamed to db_postgres.

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: 11b5019c3f38fbd96a215ad821e46882bfe773fc URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=11b5019… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Wed Feb 18 00:54:51 2009 +0100 Module renamed to db_postgres. --- modules/db_postgres/Makefile | 2 +- modules/db_postgres/pg_mod.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/db_postgres/Makefile b/modules/db_postgres/Makefile index aa86b51..9c578b9 100644 --- a/modules/db_postgres/Makefile +++ b/modules/db_postgres/Makefile @@ -4,7 +4,7 @@ include ../../Makefile.defs auto_gen= -NAME=postgres.so +NAME=db_postgres.so # set CROSS_COMPILE to true if you want to skip # the autodetection diff --git a/modules/db_postgres/pg_mod.c b/modules/db_postgres/pg_mod.c index 11aa9a9..1bc4ad4 100644 --- a/modules/db_postgres/pg_mod.c +++ b/modules/db_postgres/pg_mod.c @@ -91,7 +91,7 @@ static param_export_t params[] = { struct module_exports exports = { - "postgres", + "db_postgres", cmds, 0, /* RPC method */ params, /* module parameters */

15 years, 10 months

git:janakj/postgres: Merge branch 'master' of ssh://janakj@git.sip-router. org/sip-router into postgres

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: f7cc8a190ae22ae3f99e2ceaa6f609061e281d8d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f7cc8a1… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Wed Feb 18 01:10:10 2009 +0100 Merge branch 'master' of ssh://janakj@git.sip-router.org/sip-router into postgres * 'master' of ssh://janakj@git.sip-router.org/sip-router: Fixed matching of db flags names at the beginning of line. Two small fixes in regular expressions. ---

15 years, 10 months

git:janakj/postgres: - remove not reached return statement at the end of val2str functions

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: 8737295bc65e8b4bdf549a967d8dd8c0176f363e URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8737295… Author: Henning Westerholt <henning.westerholt(a)1und1.de> Committer: Henning Westerholt <henning.westerholt(a)1und1.de> Date: Fri Sep 19 11:56:49 2008 +0000 - remove not reached return statement at the end of val2str functions git-svn-id: https://openser.svn.sourceforge.net/svnroot/openser/trunk@4965 689a6050-402a-0410-94f2-e92a70836424 --- modules/db_postgres/km_val.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/modules/db_postgres/km_val.c b/modules/db_postgres/km_val.c index 21468d7..8024d52 100644 --- a/modules/db_postgres/km_val.c +++ b/modules/db_postgres/km_val.c @@ -327,5 +327,4 @@ int db_postgres_val2str(const db_con_t* _con, const db_val_t* _v, char* _s, int* LM_DBG("unknown data type\n"); return -10; } - return -8; }

15 years, 10 months

git:janakj/postgres: - fix memory leak in db_postgres module related to BLOBs (also caused an

by Jan Janak

Module: sip-router Branch: janakj/postgres Commit: f09596d32a99108f6c34f5fa3641be3dd6470b24 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f09596d… Author: Henning Westerholt <henning.westerholt(a)1und1.de> Committer: Henning Westerholt <henning.westerholt(a)1und1.de> Date: Mon Nov 24 17:38:46 2008 +0000 - fix memory leak in db_postgres module related to BLOBs (also caused an abort in a assertion when DBG_QM_MALLOC memory manager is used) - PQunescapeBytea is used for BLOBs, this allocates new memory, which could not freed from us later in pkg_free in the DB core - thus we need to allocate new memory for this datatype, copy the string and free the postgres result after that - small comment fix git-svn-id: https://openser.svn.sourceforge.net/svnroot/openser/trunk@5250 689a6050-402a-0410-94f2-e92a70836424 --- modules/db_postgres/km_val.c | 30 ++++++++++++++++++++++-------- 1 files changed, 22 insertions(+), 8 deletions(-) diff --git a/modules/db_postgres/km_val.c b/modules/db_postgres/km_val.c index d5a3115..9b7d12b 100644 --- a/modules/db_postgres/km_val.c +++ b/modules/db_postgres/km_val.c @@ -61,6 +61,7 @@ int db_postgres_str2val(const db_type_t _t, db_val_t* _v, const char* _s, const int _l) { static str dummy_string = {"", 0}; + char *tmp_s; if (!_v) { LM_ERR("invalid parameter value\n"); @@ -159,19 +160,32 @@ int db_postgres_str2val(const db_type_t _t, db_val_t* _v, const char* _s, const case DB_BLOB: LM_DBG("converting BLOB [%.*s]\n", _l, _s); - /* PQunescapeBytea: Converts a string representation of binary data - * into binary data - the reverse of PQescapeBytea. - * This is needed when retrieving bytea data in text format, - * but not when retrieving it in binary format. + /* + * The string is stored in new allocated memory, which we could + * not free later thus we need to copy it to some new memory here. */ - VAL_BLOB(_v).s = (char*)PQunescapeBytea((unsigned char*)_s, - (size_t*)(void*)&(VAL_BLOB(_v).len) ); + tmp_s = (char*)PQunescapeBytea((unsigned char*)_s, (size_t*)(void*)&(VAL_BLOB(_v).len)); + if(tmp_s==NULL) { + LM_ERR("PQunescapeBytea failed\n"); + return -7; + } + VAL_BLOB(_v).s = pkg_malloc(VAL_BLOB(_v).len); + if (VAL_BLOB(_v).s == NULL) { + LM_ERR("no private memory left\n"); + PQfreemem(tmp_s); + return -8; + } + LM_DBG("allocate %d bytes memory for BLOB at %p", VAL_BLOB(_v).len, VAL_BLOB(_v).s); + memcpy(VAL_BLOB(_v).s, tmp_s, VAL_BLOB(_v).len); + PQfreemem(tmp_s); + VAL_TYPE(_v) = DB_BLOB; VAL_FREE(_v) = 1; + LM_DBG("got blob len %d\n", _l); return 0; } - return -7; + return -9; } @@ -314,7 +328,7 @@ int db_postgres_val2str(const db_con_t* _con, const db_val_t* _v, char* _s, int* (size_t)l, (size_t*)&tmp_len); if(tmp_s==NULL) { - LM_ERR("PQescapeBytea failed\n"); + LM_ERR("PQescapeByteaConn failed\n"); return -9; } if (tmp_len > *_len) {

15 years, 10 months

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev