[Asterisk-code-review] cdr adaptive odbc: Fix DNSs mixed config quote quoted identi... (asterisk[master])
    Rodrigo Ramirez Norambuena 
    asteriskteam at digium.com
       
    Fri Aug  5 23:42:46 CDT 2016
    
    
  
Rodrigo Ramirez Norambuena has uploaded a new change for review.
  https://gerrit.asterisk.org/3423
Change subject: cdr_adaptive_odbc: Fix DNSs mixed config quote quoted_identifiers
......................................................................
cdr_adaptive_odbc: Fix DNSs mixed config quote quoted_identifiers
When haved more than once DNSs config and one of their dont set
quoted_identifiers and before this is with configurated with
quoted_identifiers resulting a truncate statement for a reference null
for quote character identifier.
This patch initializes quoted flag before build SQL Query
Example config for this bugfix case in cdr_adaptive_odbc.conf file
	[first]
	connection=asterisk-server1
	table=cdr
	quoted_identifiers="
	[second]
	connection=asterisk-server2
	table=cdr
	[third]
	connection=asterisk-server3
	table=cdr
	quoted_identifiers=`
Change-Id: Ibd95667b468e10d4a19a2b9d88b9934ec7207e1d
---
M cdr/cdr_adaptive_odbc.c
1 file changed, 1 insertion(+), 0 deletions(-)
  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/23/3423/1
diff --git a/cdr/cdr_adaptive_odbc.c b/cdr/cdr_adaptive_odbc.c
index 0da8c01..f0bcc09 100644
--- a/cdr/cdr_adaptive_odbc.c
+++ b/cdr/cdr_adaptive_odbc.c
@@ -408,6 +408,7 @@
 	AST_LIST_TRAVERSE(&odbc_tables, tableptr, list) {
 		separator = "";
 
+		quoted = 0;
 		if (tableptr->quoted_identifiers != '\0'){
 			quoted = 1;
 		}
-- 
To view, visit https://gerrit.asterisk.org/3423
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibd95667b468e10d4a19a2b9d88b9934ec7207e1d
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
    
    
More information about the asterisk-code-review
mailing list