From davidn@labs.usn.blaze.net.au  Mon Jul  7 03:16:07 1997
Received: from labs.usn.blaze.net.au (labs.usn.blaze.net.au [203.17.53.30])
          by hub.freebsd.org (8.8.5/8.8.5) with ESMTP id DAA06586
          for <FreeBSD-gnats-submit@freebsd.org>; Mon, 7 Jul 1997 03:16:04 -0700 (PDT)
Received: (from davidn@localhost)
	by labs.usn.blaze.net.au (8.8.5/8.8.5) id UAA19462;
	Mon, 7 Jul 1997 20:15:53 +1000 (EST)
Message-Id: <199707071015.UAA19462@labs.usn.blaze.net.au>
Date: Mon, 7 Jul 1997 20:15:53 +1000 (EST)
From: David Nugent <davidn@labs.usn.blaze.net.au>
To: FreeBSD-gnats-submit@freebsd.org
Subject: use of srandomdev() in src/eBones/lib/libkrb/des_rw.c
X-Send-Pr-Version: 3.2

>Number:         4046
>Category:       bin
>Synopsis:       srandomdev() in stdlib.h now returns void, but return value used
>Confidential:   no
>Severity:       critical
>Priority:       high
>Responsible:    steve
>State:          closed
>Quarter:
>Keywords:
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Mon Jul  7 03:20:01 PDT 1997
>Closed-Date:    Sun Aug 10 20:15:26 PDT 1997
>Last-Modified:  Sun Aug 10 20:16:26 PDT 1997
>Originator:     davidn@blaze.net.au
>Release:        FreeBSD 3.0-CURRENT i386
>Organization:
Unique Computing, Melbourne, Australia
>Environment:

	3.0 current, after revision 1.9 of stdlib.h.
	Build eBones distribution.

>Description:

	return type of srandomdev() changed from int to void, but the
	return value was being used in src/eBones/lib/libkrb/des_rw.c.

>How-To-Repeat:

	Make world, include eBones.

>Fix:

--- des_rw.c.orig	Mon Jul  7 20:12:08 1997
+++ des_rw.c	Mon Jul  7 20:15:14 1997
@@ -230,8 +230,7 @@
 		 */
 		if(!seeded) {
 			seeded = 1;
-			if (srandomdev() < 0)
-				srandom((unsigned long)(time(NULL) ^ getpid()));
+			srandomdev();
 		}
 
 		for(i = 0 ; i < 8 ; i+= sizeof(long)) {


>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->closed 
State-Changed-By: steve 
State-Changed-When: Sun Aug 10 20:15:26 PDT 1997 
State-Changed-Why:  
Fixed by Andrey Chernov <ache@FreeBSD.ORG> in revision 
1.9 of des_rw.c. 
>Unformatted:
