pgsql: Fix copy-pasto in freeing memory on error in vacuumlo. - Mailing list pgsql-committers

From Heikki Linnakangas
Subject pgsql: Fix copy-pasto in freeing memory on error in vacuumlo.
Date
Msg-id E1hZBSn-0008NA-Aw@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
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

Branch
------
REL_10_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/88ca787b16cc5eee79e2e543b2da74ea5b872a85

Modified Files
--------------
contrib/vacuumlo/vacuumlo.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)


pgsql-committers by date:

Previous
From: Alvaro Herrera
Date:
Subject: pgsql: Fix default_tablespace usage for partitioned tables
Next
From: Michael Paquier
Date:
Subject: pgsql: Improve some comments in pg_checksums about the needed cleanshu