diff options
author | /C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org </C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org> | 2008-02-03 12:31:30 +0000 |
---|---|---|
committer | /C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org </C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org> | 2008-02-03 12:31:30 +0000 |
commit | 5f081c0865646df563fb58e2501d2b3d5fb7300d (patch) | |
tree | 3c77c1e45f6e85b8ac16bcd7e7e166bf9a34dbc7 /output/pgsql/ulogd_output_PGSQL.c | |
parent | 63f517f41f190877dd024c1002b6127e89927424 (diff) |
From: Pierre Chifflier <chifflier@inl.fr>
This patch fixes the type of some fields in the SQL schema to sync
with datatype of the corresponding ulogd2 keys.
Diffstat (limited to 'output/pgsql/ulogd_output_PGSQL.c')
-rw-r--r-- | output/pgsql/ulogd_output_PGSQL.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/output/pgsql/ulogd_output_PGSQL.c b/output/pgsql/ulogd_output_PGSQL.c index 0882357..7ece626 100644 --- a/output/pgsql/ulogd_output_PGSQL.c +++ b/output/pgsql/ulogd_output_PGSQL.c @@ -280,8 +280,7 @@ static int open_db_pgsql(struct ulogd_pluginstance *upi) static int escape_string_pgsql(struct ulogd_pluginstance *upi, char *dst, const char *src, unsigned int len) { - PQescapeString(dst, src, strlen(src)); - return 0; + return PQescapeString(dst, src, strlen(src)); } static int execute_pgsql(struct ulogd_pluginstance *upi, @@ -290,7 +289,8 @@ static int execute_pgsql(struct ulogd_pluginstance *upi, struct pgsql_instance *pi = (struct pgsql_instance *) upi->private; pi->pgres = PQexec(pi->dbh, stmt); - if (!pi->pgres || PQresultStatus(pi->pgres) != PGRES_COMMAND_OK) { + if (!(pi->pgres && (PQresultStatus(pi->pgres) == PGRES_COMMAND_OK) + || (PQresultStatus(pi->pgres) == PGRES_TUPLES_OK))) { ulogd_log(ULOGD_ERROR, "execute failed (%s)\n", PQerrorMessage(pi->dbh)); return -1; |