pgsql: Fix pgstat_report_replslot() to use proper data types for its ar - Mailing list pgsql-committers

From Fujii Masao
Subject pgsql: Fix pgstat_report_replslot() to use proper data types for its ar
Date
Msg-id E1lSFAP-0000OW-CH@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix pgstat_report_replslot() to use proper data types for its arguments.

The caller of pgstat_report_replslot() passes int64 values to the function.
Also the function stores those values in PgStat_Counter (i.e., int64) fields
of PgStat_MsgReplSlot struct. But previously the function used "int" as
the data types of some arguments for those values, which could lead to
the overflow of values.

To avoid this risk, this commit fixes pgstat_report_replslot() to use
PgStat_Counter type for the arguments. Since they are the statistics counters,
PgStat_Counter, the data type used for counters, is used for them
instead of int64.

Reported-by: Vignesh C
Author: Vignesh C
Reviewed-by: Jeevan Ladhe, Fujii Masao
Discussion: https://postgr.es/m/CALDaNm080OpG=ZwOb0i8EyChH5SyHAMFWJCKaKTXmrfvJLbgaA@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/96bdb7e19de80a0c9521c5696455bca2a685c919

Modified Files
--------------
src/backend/postmaster/pgstat.c | 6 ++++--
src/include/pgstat.h            | 6 ++++--
2 files changed, 8 insertions(+), 4 deletions(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: doc: Clarify how to generate backup files with non-exclusive bac
Next
From: Peter Eisentraut
Date:
Subject: pgsql: Add support for NullIfExpr in eval_const_expressions