diff options
| author | Eric Leblond <eric@inl.fr> | 2008-07-31 10:44:15 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2008-07-31 10:44:15 +0200 |
| commit | 61704a1f28aad1c64cbcdbec67be17e675482065 (patch) | |
| tree | 54a4a36cf65bb6a9102ae2e04e8d167b785b0c27 /output | |
| parent | 1524d689f179eaaae0836ad8ef2da31b10664feb (diff) | |
cleanup: fix compilation warning about lack of parenthesis
This patch adds parenthesis around an expression to avoid confusion
between order preference of && and || operators.
Signed-off-by: Eric Leblond <eric@inl.fr>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'output')
| -rw-r--r-- | output/pgsql/ulogd_output_PGSQL.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/output/pgsql/ulogd_output_PGSQL.c b/output/pgsql/ulogd_output_PGSQL.c index b04526a..afab2cc 100644 --- a/output/pgsql/ulogd_output_PGSQL.c +++ b/output/pgsql/ulogd_output_PGSQL.c @@ -288,8 +288,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) - || (PQresultStatus(pi->pgres) == PGRES_TUPLES_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; |