postgres.git / summary / log / commit / refs

commit    8c2dd212d1bcefa8d2cbcbe53398de367c2d5f63
Author:   Nathan Bossart <nathan@postgresql.org>
Date:     Tue Feb 04 19:26:57 2025 +0000
First released in: 18.0, 13.19, 14.16, 15.11, 16.7, 17.3

    vacuumdb: Add missing PQfinish() calls to vacuum_one_database().
    
    A few of the version checks in vacuum_one_database() do not call
    PQfinish() before exiting.  This precedent was unintentionally
    established in commit 00d1e88d36, and while it's probably not too
    problematic, it seems better to properly close the connection.
    
    Reviewed-by: Daniel Gustafsson <daniel@yesql.se>
    Discussion: https://postgr.es/m/Z6JAwqN1I8ljTuXp%40nathan
    Backpatch-through: 13


 src/bin/scripts/vacuumdb.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/bin/scripts/vacuumdb.c b/src/bin/scripts/vacuumdb.c
index a4039213e9f..cc43141f0a0 100644
--- a/src/bin/scripts/vacuumdb.c
+++ b/src/bin/scripts/vacuumdb.c
@@ -461,16 +461,25 @@ vacuum_one_database(ConnParams *cparams,
 	}
 
 	if (vacopts->min_xid_age != 0 && PQserverVersion(conn) < 90600)
+	{
+		PQfinish(conn);
 		pg_fatal("cannot use the \"%s\" option on server versions older than PostgreSQL %s",
 				 "--min-xid-age", "9.6");
+	}
 
 	if (vacopts->min_mxid_age != 0 && PQserverVersion(conn) < 90600)
+	{
+		PQfinish(conn);
 		pg_fatal("cannot use the \"%s\" option on server versions older than PostgreSQL %s",
 				 "--min-mxid-age", "9.6");
+	}
 
 	if (vacopts->parallel_workers >= 0 && PQserverVersion(conn) < 130000)
+	{
+		PQfinish(conn);
 		pg_fatal("cannot use the \"%s\" option on server versions older than PostgreSQL %s",
 				 "--parallel", "13");
+	}
 
 	if (!quiet)
 	{


[parent: 061994fb3148]