aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Eisentraut <peter@eisentraut.org>2023-04-03 07:13:52 +0200
committerPeter Eisentraut <peter@eisentraut.org>2023-04-03 07:21:06 +0200
commit1980d3585e3c7fce74b811ca8d77733036cae76c (patch)
tree0aeeab0bc9b4e2202839849bcaf2d4e3ee12253b
parent3f476c953495e4d52a1ac776f976df16964ba592 (diff)
downloadpostgresql-1980d3585e3c7fce74b811ca8d77733036cae76c.tar.gz
postgresql-1980d3585e3c7fce74b811ca8d77733036cae76c.zip
pg_basebackup: Correct type of WalSegSz
The pg_basebackup code had WalSegSz as uint32, whereas the rest of the code has it as int. This seems confusing, and using the extra range wouldn't actually work. Reviewed-by: Daniel Gustafsson <daniel@yesql.se> Discussion: https://www.postgresql.org/message-id/flat/1bf15c7a-0acd-1864-081e-7a28814310fe%40enterprisedb.com
-rw-r--r--src/bin/pg_basebackup/streamutil.c2
-rw-r--r--src/bin/pg_basebackup/streamutil.h2
2 files changed, 2 insertions, 2 deletions
diff --git a/src/bin/pg_basebackup/streamutil.c b/src/bin/pg_basebackup/streamutil.c
index e7618f46172..15514599c4e 100644
--- a/src/bin/pg_basebackup/streamutil.c
+++ b/src/bin/pg_basebackup/streamutil.c
@@ -31,7 +31,7 @@
#define ERRCODE_DUPLICATE_OBJECT "42710"
-uint32 WalSegSz;
+int WalSegSz;
static bool RetrieveDataDirCreatePerm(PGconn *conn);
diff --git a/src/bin/pg_basebackup/streamutil.h b/src/bin/pg_basebackup/streamutil.h
index 04ceb309719..268c163213c 100644
--- a/src/bin/pg_basebackup/streamutil.h
+++ b/src/bin/pg_basebackup/streamutil.h
@@ -24,7 +24,7 @@ extern char *dbuser;
extern char *dbport;
extern char *dbname;
extern int dbgetpassword;
-extern uint32 WalSegSz;
+extern int WalSegSz;
/* Connection kept global so we can disconnect easily */
extern PGconn *conn;