This patch adds parenthesis around an expression to avoid confusion between
order preference of && and || operators.
Signed-off-by: Eric Leblond <eric@xxxxxx>
---
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;
--
1.5.4.3
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
