From: millert <millert@openbsd.org>
Date: Tue, 4 Apr 2000 14:27:00 +0000 (+0000)
Subject: Fix the leak for real (that's what I get for hacking when i can't sleep).
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=cbf1311a43332bbc1e044a9386aa1b762b213a64;p=openbsd

Fix the leak for real (that's what I get for hacking when i can't sleep).
---

diff --git a/lib/libc/stdlib/random.c b/lib/libc/stdlib/random.c
index 2b97c5a54a2..5ce7c90ee9c 100644
--- a/lib/libc/stdlib/random.c
+++ b/lib/libc/stdlib/random.c
@@ -32,7 +32,7 @@
  */
 
 #if defined(LIBC_SCCS) && !defined(lint)
-static char *rcsid = "$OpenBSD: random.c,v 1.8 2000/04/04 13:38:24 millert Exp $";
+static char *rcsid = "$OpenBSD: random.c,v 1.9 2000/04/04 14:27:00 millert Exp $";
 #endif /* LIBC_SCCS and not lint */
 
 #include <sys/types.h>
@@ -245,18 +245,20 @@ 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) {
+	if ((fd = open("/dev/arandom", O_RDONLY, 0)) != -1 &&
+	    read(fd, (void *) state, len) == (ssize_t) len) {
+		close(fd);
+	} else {
 		struct timeval tv;
 		u_int junk;
 
 		/* XXX - this could be better */
 		gettimeofday(&tv, NULL);
 		srandom(getpid() ^ tv.tv_sec ^ tv.tv_usec ^ junk);
+		if (fd != -1)
+			close(fd);
 		return;
 	}
-	if (fd != -1)
-		close(fd);
 
 	if (rand_type != TYPE_0) {
 		fptr = &state[rand_sep];