From 5faef9d582012433db9ad05af27a77bd591508e1 Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Sun, 3 Jul 2022 20:11:05 +0200 Subject: Remove redundant null pointer checks before PQclear and PQconninfoFree These functions already had the free()-like behavior of handling null pointers as a no-op. But it wasn't documented, so add it explicitly to the documentation, too. Discussion: https://www.postgresql.org/message-id/flat/dac5d2d0-98f5-94d9-8e69-46da2413593d%40enterprisedb.com --- src/fe_utils/query_utils.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'src/fe_utils/query_utils.c') diff --git a/src/fe_utils/query_utils.c b/src/fe_utils/query_utils.c index 2fc6e2405b4..6575b24c78f 100644 --- a/src/fe_utils/query_utils.c +++ b/src/fe_utils/query_utils.c @@ -85,8 +85,7 @@ executeMaintenanceCommand(PGconn *conn, const char *query, bool echo) r = (res && PQresultStatus(res) == PGRES_COMMAND_OK); - if (res) - PQclear(res); + PQclear(res); return r; } -- cgit v1.2.3