diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2008-03-10 12:39:23 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2008-03-10 12:39:23 +0000 |
| commit | 23c356ccecaf10665777c05fac414466b7d5793d (patch) | |
| tree | 81e263f9f7dcc54883d5ff6e898acbe78de3ca65 /src/backend/utils/adt/float.c | |
| parent | bfce56eea45b1369b7bb2150a150d1ac109f5073 (diff) | |
| download | postgresql-23c356ccecaf10665777c05fac414466b7d5793d.tar.gz postgresql-23c356ccecaf10665777c05fac414466b7d5793d.zip | |
Document and enforce that the usable range of setseed() arguments is
-1 to 1, not 0 to 1. The actual behavior for values within this range
does not change. Kris Jurka
Diffstat (limited to 'src/backend/utils/adt/float.c')
| -rw-r--r-- | src/backend/utils/adt/float.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/backend/utils/adt/float.c b/src/backend/utils/adt/float.c index e7d0cdd70dd..2f1e262ea7a 100644 --- a/src/backend/utils/adt/float.c +++ b/src/backend/utils/adt/float.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/adt/float.c,v 1.153 2008/01/01 19:45:52 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/utils/adt/float.c,v 1.154 2008/03/10 12:39:22 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -1684,8 +1684,12 @@ Datum setseed(PG_FUNCTION_ARGS) { float8 seed = PG_GETARG_FLOAT8(0); - int iseed = (int) (seed * MAX_RANDOM_VALUE); + int iseed; + if (seed < -1 || seed > 1) + elog(ERROR, "setseed parameter %f out of range [-1,1]", seed); + + iseed = (int) (seed * MAX_RANDOM_VALUE); srandom((unsigned int) iseed); PG_RETURN_VOID(); |