From bb1c08f6beedc5a89ad7713c914f734e29c76d74 Mon Sep 17 00:00:00 2001 From: millert <> Date: Wed, 21 Mar 2012 12:36:49 +0000 Subject: Fix a bug where random() always returns 0 when srandom() is seeded with 0. Use 1 and not 0 as the first element of the state array, similar to what glibc does. OK nicm@ --- src/lib/libc/stdlib/random.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/lib/libc/stdlib/random.c b/src/lib/libc/stdlib/random.c index 48e892042b..5a9a7c3313 100644 --- a/src/lib/libc/stdlib/random.c +++ b/src/lib/libc/stdlib/random.c @@ -1,4 +1,4 @@ -/* $OpenBSD: random.c,v 1.15 2005/11/30 07:51:02 otto Exp $ */ +/* $OpenBSD: random.c,v 1.16 2012/03/21 12:36:49 millert Exp $ */ /* * Copyright (c) 1983 Regents of the University of California. * All rights reserved. @@ -196,7 +196,8 @@ srandom(unsigned int x) if (rand_type == TYPE_0) state[0] = x; else { - state[0] = x; + /* A seed of 0 would result in state[] always being zero. */ + state[0] = x ? x : 1; for (i = 1; i < rand_deg; i++) { /* * Implement the following, without overflowing 31 bits: -- cgit v1.2.3-55-g6feb