From f8332e5d2797b1e7d4081a746412a49f8595d38f Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Mon, 22 Aug 2022 16:51:52 +0200 Subject: [PATCH 4/5] Remove unneeded null pointer checks before PQfreemem() --- contrib/vacuumlo/vacuumlo.c | 9 +++------ .../replication/libpqwalreceiver/libpqwalreceiver.c | 6 ++---- src/bin/pg_basebackup/receivelog.c | 9 +++------ 3 files changed, 8 insertions(+), 16 deletions(-) diff --git a/contrib/vacuumlo/vacuumlo.c b/contrib/vacuumlo/vacuumlo.c index b7c8f2c805..264b879bd3 100644 --- a/contrib/vacuumlo/vacuumlo.c +++ b/contrib/vacuumlo/vacuumlo.c @@ -231,12 +231,9 @@ vacuumlo(const char *database, const struct _param *param) pg_log_error("%s", PQerrorMessage(conn)); PQclear(res); PQfinish(conn); - if (schema != NULL) - PQfreemem(schema); - if (table != NULL) - PQfreemem(table); - if (field != NULL) - PQfreemem(field); + PQfreemem(schema); + PQfreemem(table); + PQfreemem(field); return -1; } diff --git a/src/backend/replication/libpqwalreceiver/libpqwalreceiver.c b/src/backend/replication/libpqwalreceiver/libpqwalreceiver.c index 2865024524..7f697b0f29 100644 --- a/src/backend/replication/libpqwalreceiver/libpqwalreceiver.c +++ b/src/backend/replication/libpqwalreceiver/libpqwalreceiver.c @@ -741,8 +741,7 @@ static void libpqrcv_disconnect(WalReceiverConn *conn) { PQfinish(conn->streamConn); - if (conn->recvBuf != NULL) - PQfreemem(conn->recvBuf); + PQfreemem(conn->recvBuf); pfree(conn); } @@ -768,8 +767,7 @@ libpqrcv_receive(WalReceiverConn *conn, char **buffer, { int rawlen; - if (conn->recvBuf != NULL) - PQfreemem(conn->recvBuf); + PQfreemem(conn->recvBuf); conn->recvBuf = NULL; /* Try to receive a CopyData message */ diff --git a/src/bin/pg_basebackup/receivelog.c b/src/bin/pg_basebackup/receivelog.c index 976d1e73b1..5f6fd3201f 100644 --- a/src/bin/pg_basebackup/receivelog.c +++ b/src/bin/pg_basebackup/receivelog.c @@ -858,8 +858,7 @@ HandleCopyStream(PGconn *conn, StreamCtl *stream, } error: - if (copybuf != NULL) - PQfreemem(copybuf); + PQfreemem(copybuf); return NULL; } @@ -940,8 +939,7 @@ CopyStreamReceive(PGconn *conn, long timeout, pgsocket stop_socket, char *copybuf = NULL; int rawlen; - if (*buffer != NULL) - PQfreemem(*buffer); + PQfreemem(*buffer); *buffer = NULL; /* Try to receive a CopyData message */ @@ -1204,8 +1202,7 @@ HandleEndOfCopyStream(PGconn *conn, StreamCtl *stream, char *copybuf, } still_sending = false; } - if (copybuf != NULL) - PQfreemem(copybuf); + PQfreemem(copybuf); *stoppos = blockpos; return res; } -- 2.37.1