mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove unneeded null pointer checks before PQfreemem()

Browse Source 
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
This commit is contained in:
Peter Eisentraut 2022-08-26 19:16:28 +02:00
parent 45987aae26
commit e890ce7a4f
3 changed files with 8 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
contrib/vacuumlo/vacuumlo.c
View File

@ -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;
}

6
src/backend/replication/libpqwalreceiver/libpqwalreceiver.c
View File

@ -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 */

9
src/bin/pg_basebackup/receivelog.c
View File

@ -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;
}