vacuumlo: Avoid unlikely memory leak.

Spotted by Coverity.  This isn't likely to matter in practice, but
there's no harm in fixing it.

Michael Paquier

diff --git a/contrib/vacuumlo/vacuumlo.c b/contrib/vacuumlo/vacuumlo.c
index 43130dca8afcf181a990dd1bdfe260896c2d3fa8..ca0d3048b82a201290f83c19113931e0a5765248 100644 (file)
--- a/contrib/vacuumlo/vacuumlo.c
+++ b/contrib/vacuumlo/vacuumlo.c
@@ -240,6 +240,12 @@ vacuumlo(const char *database, const struct _param * param)
            fprintf(stderr, "Out of memory\n");
            PQclear(res);
            PQfinish(conn);
+           if (schema != NULL)
+               PQfreemem(schema);
+           if (schema != NULL)
+               PQfreemem(table);
+           if (schema != NULL)
+               PQfreemem(field);
            return -1;
        }
 
@@ -256,6 +262,9 @@ vacuumlo(const char *database, const struct _param * param)
            PQclear(res2);
            PQclear(res);
            PQfinish(conn);
+           PQfreemem(schema);
+           PQfreemem(table);
+           PQfreemem(field);
            return -1;
        }
        PQclear(res2);