aboutsummaryrefslogtreecommitdiff
path: root/contrib/postgres_fdw/postgres_fdw.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2015-07-01 18:07:48 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2015-07-01 18:07:48 -0400
commitd7c19d68550eb6018e8581a73a351905f4cc435c (patch)
treed894366d75e293ae8769d7cfe39b93179f5684d1 /contrib/postgres_fdw/postgres_fdw.c
parent8217370864c950ea28c7f940442fe48c701461c2 (diff)
downloadpostgresql-d7c19d68550eb6018e8581a73a351905f4cc435c.tar.gz
postgresql-d7c19d68550eb6018e8581a73a351905f4cc435c.zip
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
Diffstat (limited to 'contrib/postgres_fdw/postgres_fdw.c')
0 files changed, 0 insertions, 0 deletions