https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=194204
Mateusz Guzik <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #2 from Mateusz Guzik <[email protected]> --- I'm confused with this patch. OpenBSD implementation returns EIO for too big buffers, although apparently this was not the behaviour from the start. Previously it filled up to 256 bytes, but also indicated how many bytes were returned. Your patch fills up to 256 bytes and does not tell the caller how many bytes it got, which is a big no-no. We should stick to what is in OpenBSD, so please update the patch to return EIO when needed. memset you put there is buggy. explicit_zero or equivalent is needed and if we don't have that in the kernel one will need to be ported as well. Cannot comment on arc4rand usefulness for this purpose though. -- You are receiving this mail because: You are the assignee for the bug. _______________________________________________ [email protected] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-bugs To unsubscribe, send any mail to "[email protected]"
