]> granicus.if.org Git - postgresql/commit
Make sampler_random_fract() actually obey its API contract.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 1 Jul 2015 22:07:48 +0000 (18:07 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 1 Jul 2015 22:07:59 +0000 (18:07 -0400)
commitcd7030ff085f5c378e837b392cb719cf23df9d0b
tree7e0d90e97dc36706fe016275a302a4075e0b4768
parent6cfb6d987419ce1e7bec0cf3ad22830ed3c2dc08
Make sampler_random_fract() actually obey its API contract.

This function is documented to return a value in the range (0,1),
which is what its predecessor anl_random_fract() did.  However, the
new version depends on pg_erand48() which returns a value in [0,1).
The possibility of returning zero creates hazards of division by zero
or trying to compute log(0) at some call sites, and it might well
break third-party modules using anl_random_fract() too.  So let's
change it to never return zero.  Spotted by Coverity.

Michael Paquier, cosmetically adjusted by me
src/backend/utils/misc/sampling.c