Re: kern/48119 (/dev/random kernel crash)

[Taylor R Campbell <riastradh%NetBSD.org@localhost>]

The following reply was made to PR kern/48119; it has been noted by GNATS.

From: Taylor R Campbell <riastradh%NetBSD.org@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: Thor Lancelot Simon <tls%NetBSD.org@localhost>
Subject: Re: kern/48119 (/dev/random kernel crash)
Date: Tue, 3 Sep 2013 13:17:04 +0000

 If someone has tested it and it works, the patch looks OK to me
 (remove the printf after testing before commit, of course), as a
 provisional measure until we split cprng_strong into two routines.
 When shoe-horning -1 into a size_t, I'd write an explicit cast in both
 cases to make it clear that we're doing something bogus, and perhaps
 add a comment to the effect that returning -1 is and provisional until
 we fix the API.
 
 Any comments, Thor?