mirror of
https://git.postgresql.org/git/postgresql.git
synced 2024-08-30 00:07:20 +02:00
Don't depend on -fwrapv semantics in pgbench's random() function.
Instead use the common/int.h functions to check for integer overflow in a more C-standard-compliant fashion. This is motivated by recent failures on buildfarm member moonjelly, where it appears that development-tip gcc is optimizing without regard to the -fwrapv switch. Presumably that's a gcc bug that will be fixed soon, but we might as well install cleaner coding here rather than wait. (This does not address the question of whether we'll ever be able to get rid of using -fwrapv. Testing shows that this spot is the only place where doing so creates visible regression test failures, but unfortunately that proves very little.) Back-patch to v12. The common/int.h functions exist in v11, but that branch doesn't use them in any client-side code. I judge that this case isn't interesting enough in the real world to take even a small risk of issues from being the first such use. Tom Lane and Fabien Coelho Discussion: https://postgr.es/m/73927.1624815543@sss.pgh.pa.us
This commit is contained in:
parent
741deb2603
commit
be567deb3a
@ -2278,7 +2278,8 @@ evalStandardFunc(CState *st,
|
|||||||
case PGBENCH_RANDOM_ZIPFIAN:
|
case PGBENCH_RANDOM_ZIPFIAN:
|
||||||
{
|
{
|
||||||
int64 imin,
|
int64 imin,
|
||||||
imax;
|
imax,
|
||||||
|
delta;
|
||||||
|
|
||||||
Assert(nargs >= 2);
|
Assert(nargs >= 2);
|
||||||
|
|
||||||
@ -2287,12 +2288,13 @@ evalStandardFunc(CState *st,
|
|||||||
return false;
|
return false;
|
||||||
|
|
||||||
/* check random range */
|
/* check random range */
|
||||||
if (imin > imax)
|
if (unlikely(imin > imax))
|
||||||
{
|
{
|
||||||
pg_log_error("empty range given to random");
|
pg_log_error("empty range given to random");
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
else if (imax - imin < 0 || (imax - imin) + 1 < 0)
|
else if (unlikely(pg_sub_s64_overflow(imax, imin, &delta) ||
|
||||||
|
pg_add_s64_overflow(delta, 1, &delta)))
|
||||||
{
|
{
|
||||||
/* prevent int overflows in random functions */
|
/* prevent int overflows in random functions */
|
||||||
pg_log_error("random range is too large");
|
pg_log_error("random range is too large");
|
||||||
|
Loading…
Reference in New Issue
Block a user