diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2022-06-17 11:51:38 +0200 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2022-07-03 09:05:34 +0200 |
| commit | 098c703d308fa88dc9e3f9f623ca023ce4717794 (patch) | |
| tree | 81e03668d6fc6ea832f16996da78e3483beef4f7 /src/bin/pg_basebackup/walmethods.c | |
| parent | e2bc242833da27cd73c279bebfb321a65384808f (diff) | |
Remove redundant null pointer checks before pg_free()
These are especially useless because the whole point of pg_free() was
to do that very check before calling free().
pg_free() could be removed altogether, but I'm keeping it here to keep
the API consistent.
Discussion: https://www.postgresql.org/message-id/flat/dac5d2d0-98f5-94d9-8e69-46da2413593d%40enterprisedb.com
Diffstat (limited to 'src/bin/pg_basebackup/walmethods.c')
| -rw-r--r-- | src/bin/pg_basebackup/walmethods.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/bin/pg_basebackup/walmethods.c b/src/bin/pg_basebackup/walmethods.c index cc292718da9..ef4c11277ac 100644 --- a/src/bin/pg_basebackup/walmethods.c +++ b/src/bin/pg_basebackup/walmethods.c @@ -500,8 +500,7 @@ dir_close(Walfile f, WalCloseMethod method) pg_free(df->pathname); pg_free(df->fullpath); - if (df->temp_suffix) - pg_free(df->temp_suffix); + pg_free(df->temp_suffix); pg_free(df); return r; |