diff options
| author | Peter Eisentraut | 2022-08-26 17:16:28 +0000 |
|---|---|---|
| committer | Peter Eisentraut | 2022-08-26 17:16:28 +0000 |
| commit | e890ce7a4feb9d72cd502d52c5a4c4d853801974 (patch) | |
| tree | 7535666c7981bff5d29839ef1733d8896dcebdca /src/backend/replication | |
| parent | 45987aae260a441886a010323bf3e143ce8e82d6 (diff) | |
Remove unneeded null pointer checks before PQfreemem()
PQfreemem() just calls free(), and the latter already checks for null
pointers.
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://www.postgresql.org/message-id/flat/cf26e970-8e92-59f1-247a-aa265235075b%40enterprisedb.com
Diffstat (limited to 'src/backend/replication')
| -rw-r--r-- | src/backend/replication/libpqwalreceiver/libpqwalreceiver.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/backend/replication/libpqwalreceiver/libpqwalreceiver.c b/src/backend/replication/libpqwalreceiver/libpqwalreceiver.c index 28650245245..7f697b0f292 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 */ |