diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2000-08-07 00:51:42 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2000-08-07 00:51:42 +0000 |
| commit | 9426047021651d8d7b6595c1377aef1909afcacd (patch) | |
| tree | 737356deee745db64bacbdd589f1562ec9398125 /src/backend/utils/adt/float.c | |
| parent | 259489bab7d01b6acc0bdc2c2938ecb22b1f5c8d (diff) | |
| download | postgresql-9426047021651d8d7b6595c1377aef1909afcacd.tar.gz postgresql-9426047021651d8d7b6595c1377aef1909afcacd.zip | |
Clean up bogosities in use of random(3) and srandom(3) --- do not assume
that RAND_MAX applies to them, since it doesn't. Instead add a
config.h parameter MAX_RANDOM_VALUE. This is currently set at 2^31-1
but could be auto-configured if that ever proves necessary. Also fix
some outright bugs like calling srand() where srandom() is appropriate.
Diffstat (limited to 'src/backend/utils/adt/float.c')
| -rw-r--r-- | src/backend/utils/adt/float.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/backend/utils/adt/float.c b/src/backend/utils/adt/float.c index 54735191a9f..7c06e31ed84 100644 --- a/src/backend/utils/adt/float.c +++ b/src/backend/utils/adt/float.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/utils/adt/float.c,v 1.67 2000/08/01 18:29:35 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/utils/adt/float.c,v 1.68 2000/08/07 00:51:14 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -1315,7 +1315,7 @@ drandom(PG_FUNCTION_ARGS) float8 result; /* result 0.0-1.0 */ - result = ((double) random()) / RAND_MAX; + result = ((double) random()) / ((double) MAX_RANDOM_VALUE); PG_RETURN_FLOAT8(result); } @@ -1328,7 +1328,7 @@ Datum setseed(PG_FUNCTION_ARGS) { float8 seed = PG_GETARG_FLOAT8(0); - int iseed = (seed * RAND_MAX); + int iseed = (int) (seed * MAX_RANDOM_VALUE); srandom((unsigned int) iseed); |