]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix for loop variables
authorPeter Eisentraut <peter@eisentraut.org>
Sat, 11 Jul 2026 12:38:33 +0000 (14:38 +0200)
committerPeter Eisentraut <peter@eisentraut.org>
Sat, 11 Jul 2026 12:39:15 +0000 (14:39 +0200)
commite615da8cb21b3745456da45197fa5dc620f19dab
tree98644d3e250b917b147c1e07e450cbe9a1f385b0
parenteed6c0d33e09c53e5861fa92e99318814c5862f7
Fix for loop variables

A number of for loops used loop variables that did not match the type
of the end condition.  This could lead to wraparound or
signed/unsigned mismatches.  Probably none of these are a problem in
practice, but it's fragile code.

Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://www.postgresql.org/message-id/flat/d639aede-209f-412b-927a-d38d4848b370%40eisentraut.org
70 files changed:
contrib/pageinspect/fsmfuncs.c
contrib/pageinspect/hashfuncs.c
contrib/pg_logicalinspect/pg_logicalinspect.c
contrib/pg_plan_advice/pgpa_identifier.c
contrib/pg_plan_advice/pgpa_join.c
contrib/pg_plan_advice/pgpa_output.c
contrib/pg_plan_advice/pgpa_walker.c
contrib/pg_trgm/trgm_gist.c
src/backend/access/gin/ginget.c
src/backend/access/gin/ginlogic.c
src/backend/access/gin/ginscan.c
src/backend/access/nbtree/nbtpage.c
src/backend/access/rmgrdesc/logicalmsgdesc.c
src/backend/executor/execCurrent.c
src/backend/jit/llvm/llvmjit.c
src/backend/lib/hyperloglog.c
src/backend/parser/parse_relation.c
src/backend/parser/parse_target.c
src/backend/port/win32/socket.c
src/backend/replication/logical/reorderbuffer.c
src/backend/replication/logical/snapbuild.c
src/backend/statistics/dependencies.c
src/backend/statistics/mcv.c
src/backend/statistics/mvdistinct.c
src/backend/storage/aio/aio_init.c
src/backend/storage/aio/method_io_uring.c
src/backend/storage/file/fd.c
src/backend/storage/ipc/shmem.c
src/backend/storage/lmgr/lock.c
src/backend/tsearch/spell.c
src/backend/utils/adt/json.c
src/backend/utils/adt/pg_dependencies.c
src/backend/utils/adt/pg_locale.c
src/backend/utils/adt/pg_locale_icu.c
src/backend/utils/adt/pg_locale_libc.c
src/backend/utils/adt/pg_ndistinct.c
src/backend/utils/adt/selfuncs.c
src/backend/utils/adt/tsgistidx.c
src/backend/utils/adt/tsquery.c
src/backend/utils/adt/tsquery_gist.c
src/backend/utils/adt/varbit.c
src/backend/utils/mb/conversion_procs/euc_tw_and_big5/big5.c
src/backend/utils/misc/injection_point.c
src/backend/utils/misc/pg_config.c
src/backend/utils/mmgr/dsa.c
src/backend/utils/resowner/resowner.c
src/backend/utils/time/snapmgr.c
src/bin/pg_amcheck/pg_amcheck.c
src/bin/pg_basebackup/pg_recvlogical.c
src/bin/pg_combinebackup/reconstruct.c
src/bin/pg_config/pg_config.c
src/bin/pg_ctl/pg_ctl.c
src/bin/pg_dump/dumputils.c
src/bin/pg_dump/pg_backup_archiver.c
src/bin/psql/crosstabview.c
src/common/hmac.c
src/common/unicode_case.c
src/fe_utils/print.c
src/include/lib/radixtree.h
src/interfaces/ecpg/test/expected/sql-sqljson.c
src/interfaces/ecpg/test/sql/sqljson.pgc
src/interfaces/libpq-oauth/oauth-curl.c
src/interfaces/libpq/fe-connect.c
src/interfaces/libpq/win32.c
src/test/modules/test_aio/test_aio.c
src/test/modules/test_binaryheap/test_binaryheap.c
src/test/modules/test_escape/test_escape.c
src/test/modules/test_integerset/test_integerset.c
src/test/modules/test_predtest/test_predtest.c
src/test/modules/test_radixtree/test_radixtree.c