I was thinking of backporting both arc4random() and Linux' getrandom() changes actually, objections? They are the best entropy/random source available for these platform, and as Bill said, no API constraints...
On Tue, Apr 3, 2018 at 9:22 PM, Eric Covener <cove...@gmail.com> wrote: > I think that's right, the thing it's correcting is trunk-only (I > checked 1.7 had no grep -ri arc4random_buf) > > On Tue, Apr 3, 2018 at 3:19 PM, William A Rowe Jr <wr...@rowe-clan.net> wrote: >> Eric, >> >> Confirming; this is a trunk-only change? I don't see 1.7/1.6 commits. >> >> On Wed, Jan 10, 2018 at 8:53 AM, <cove...@apache.org> wrote: >>> Author: covener >>> Date: Wed Jan 10 14:53:43 2018 >>> New Revision: 1820755 >>> >>> URL: http://svn.apache.org/viewvc?rev=1820755&view=rev >>> Log: >>> fix ifdef for arc4random >>> >>> r1814239 added: >>> AC_CHECK_FUNCS(arc4random_buf) >>> >>> Which only defines HAVE_ARC4RANDOM_BUF >>> >>> >>> >>> >>> Modified: >>> apr/apr/trunk/misc/unix/rand.c >>> >>> Modified: apr/apr/trunk/misc/unix/rand.c >>> URL: >>> http://svn.apache.org/viewvc/apr/apr/trunk/misc/unix/rand.c?rev=1820755&r1=1820754&r2=1820755&view=diff >>> ============================================================================== >>> --- apr/apr/trunk/misc/unix/rand.c (original) >>> +++ apr/apr/trunk/misc/unix/rand.c Wed Jan 10 14:53:43 2018 >>> @@ -224,7 +224,7 @@ APR_DECLARE(apr_status_t) apr_generate_r >>> length -= rc; >>> } while (length > 0); >>> >>> -#elif defined(HAVE_ARC4RANDOM) >>> +#elif defined(HAVE_ARC4RANDOM_BUF) >>> >>> arc4random_buf(buf, length); >>> >>> >>> > > > > -- > Eric Covener > cove...@gmail.com