]> granicus.if.org Git - postgresql/commit
Fix latent problem with pg_jrand48().
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 28 Dec 2018 19:08:24 +0000 (14:08 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 28 Dec 2018 19:08:35 +0000 (14:08 -0500)
commitf256995e33d2d1074c767a0b38a2a2ed2a583161
treefcd00fff888e532cccc5adfd620a6830e4480fdd
parent3c4ca21093b5d0662389777df853f6f3a36ee476
Fix latent problem with pg_jrand48().

POSIX specifies that jrand48() returns a signed 32-bit value (in the
range [-2^31, 2^31)), but our code was returning an unsigned 32-bit
value (in the range [0, 2^32)).  This doesn't actually matter to any
existing call site, because they all cast the "long" result to int32
or uint32; but it will doubtless bite somebody in the future.
To fix, cast the arithmetic result to int32 explicitly before the
compiler widens it to long (if widening is needed).

While at it, upgrade this file's far-short-of-project-style comments.
Had there been some peer pressure to document pg_jrand48() properly,
maybe this thinko wouldn't have gotten committed to begin with.

Backpatch to v10 where pg_jrand48() was added, just in case somebody
back-patches a fix that uses it and depends on the standard behavior.

Discussion: https://postgr.es/m/17235.1545951602@sss.pgh.pa.us
src/port/erand48.c