]> granicus.if.org Git - postgresql/commit
Fix copy-pasto in freeing memory on error in vacuumlo.
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Fri, 7 Jun 2019 09:42:27 +0000 (12:42 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Fri, 7 Jun 2019 09:42:27 +0000 (12:42 +0300)
commit77b68e3a4f854d95350a3366ecfc8b3b506ebaa7
treeee342ceb2c79178cfb90f86dbfc6be4adccb652b
parenta36c84c3e4a9bee6baa740848f67a5db3fa279b0
Fix copy-pasto in freeing memory on error in vacuumlo.

It's harmless to call PQfreemem() with a NULL argument, so the only
consequence was that if allocating 'schema' failed, but allocating 'table'
or 'field' succeeded, we would leak a bit of memory. That's highly
unlikely to happen, so this is just academical, but let's get it right.

Per bug #15838 from Timur Birsh. Backpatch back to 9.5, where the
PQfreemem() calls were introduced.

Discussion: https://www.postgresql.org/message-id/15838-3221652c72c5e69d@postgresql.org
contrib/vacuumlo/vacuumlo.c