From 540eda599c9c4c613ad5b7cb757a8483d6bbae0b Mon Sep 17 00:00:00 2001
From: millert <>
Date: Tue, 4 Apr 2000 13:38:24 +0000
Subject: [PATCH] Fix an fd leak if the read from /dev/arandom fails.  Pointed
 out by Markus Friedl.

---
 src/lib/libc/stdlib/random.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/lib/libc/stdlib/random.c b/src/lib/libc/stdlib/random.c
index 7c6e5f7e..2b97c5a5 100644
--- a/src/lib/libc/stdlib/random.c
+++ b/src/lib/libc/stdlib/random.c
@@ -32,7 +32,7 @@
  */
 
 #if defined(LIBC_SCCS) && !defined(lint)
-static char *rcsid = "$OpenBSD: random.c,v 1.7 2000/04/03 23:23:48 millert Exp $";
+static char *rcsid = "$OpenBSD: random.c,v 1.8 2000/04/04 13:38:24 millert Exp $";
 #endif /* LIBC_SCCS and not lint */
 
 #include <sys/types.h>
@@ -245,10 +245,8 @@ srandomdev()
 	else
 		len = rand_deg * sizeof(state[0]);
 
-	if ((fd = open("/dev/arandom", O_RDONLY, 0)) != -1 &&
-	    read(fd, (void *) state, len) == (ssize_t) len) {
-		close(fd);
-	} else {
+	if ((fd = open("/dev/arandom", O_RDONLY, 0)) == -1 ||
+	    read(fd, (void *) state, len) != (ssize_t) len) {
 		struct timeval tv;
 		u_int junk;
 
@@ -257,6 +255,8 @@ srandomdev()
 		srandom(getpid() ^ tv.tv_sec ^ tv.tv_usec ^ junk);
 		return;
 	}
+	if (fd != -1)
+		close(fd);
 
 	if (rand_type != TYPE_0) {
 		fptr = &state[rand_sep];