diff options
| author | Elazar Broad <elazar@thebroadfamily.com> | 2015-07-20 16:59:32 -0400 |
|---|---|---|
| committer | Richard Mudgett <rmudgett@digium.com> | 2015-07-20 17:46:19 -0500 |
| commit | 293c9f68944bba86329b8846cb2d14cb11c318d8 (patch) | |
| tree | 05cd007133b3b791854e273f492af91705f10288 | |
| parent | 6741eedeceba083267a3d8223c911b0b5c6a98d0 (diff) | |
cdr/cdr_adaptive_odbc.c: Fix quoted identifier usage when inserting CDR records
Commit a24ce38 added support for the use of quoted indentifiers when inserting
CDR records into the database. However, the if statement logic responsible for
determining whether to use those identifiers is reversed, resulting in a
reference to the quoted identifier character buffer which will be null, hence
null terminating the SQL query, resulting in a truncated statement which
fails to execute.
ASTERISK-25263 #close
Reported by: Elazar Broad
Tested by: Elazar Broad
Change-Id: I40da47309b67cc1572207b1515dcc08ec9b1f644
| -rw-r--r-- | cdr/cdr_adaptive_odbc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cdr/cdr_adaptive_odbc.c b/cdr/cdr_adaptive_odbc.c index f276959a0..8bd9d0da6 100644 --- a/cdr/cdr_adaptive_odbc.c +++ b/cdr/cdr_adaptive_odbc.c @@ -740,9 +740,9 @@ static int odbc_log(struct ast_cdr *cdr) continue; } if (quoted) { - ast_str_append(&sql, 0, "%s%s", separator, entry->name); - } else { ast_str_append(&sql, 0, "%s%c%s%c", separator, tableptr->quoted_identifiers, entry->name, tableptr->quoted_identifiers); + } else { + ast_str_append(&sql, 0, "%s%s", separator, entry->name); } separator = ", "; } else if (entry->filtervalue |