Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:118347
MIME-Version: 1.0
References: <157e4f6b6e96376e61194b9a358c87b02880ec28.camel@sandfox.me>
In-Reply-To: <157e4f6b6e96376e61194b9a358c87b02880ec28.camel@sandfox.me>
Date: Fri, 5 Aug 2022 01:14:43 +0900
Message-ID: <CABTDVhCkd00V5ScxMThsv8P3PC7nrRF6e3YQn-GCzUHs8FnumQ@mail.gmail.com>
To: Anton Smirnov <sandfox@sandfox.me>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000061291e05e56ca6ca"
Subject: Re: [PHP-DEV] xoshiro** edge case (all zeros)
From: zeriyoshi@gmail.com (Go Kudo)

--00000000000061291e05e56ca6ca
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

2022=E5=B9=B48=E6=9C=884=E6=97=A5(=E6=9C=A8) 17:10 Anton Smirnov <sandfox@s=
andfox.me>:

> Hi!
>
> Randomness again. Sorry if I just missed some relevant discussion
>
> xoshiro** has a known edge case: all-zero seed
>
> <?php
>
> $engine =3D new \Random\Engine\Xoshiro256StarStar(str_repeat("\0", 32));
>
> while (true) {
>     echo hex2bin($engine->generate()), PHP_EOL; // 0000000000000000
> }
>
> It should be documented and/or handled
>
> It's only for a string seed, int seed is not affected
>
> --
> Anton
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: https://www.php.net/unsub.php
>
>
Hi.

Thanks for the report! This is dangerous behavior and we will attempt to
fix it for 8.2beta3.

Best regards
Go Kudo

--00000000000061291e05e56ca6ca--