Re: PHP 7.1.0 to 7.2.0beta2 mt_rand() modulo bias bug
| From: | Tom Worster | Date: | Fri, 08 Sep 2017 11:56:23 +0000 |
| Subject: | Re: PHP 7.1.0 to 7.2.0beta2 mt_rand() modulo bias bug | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-100468@lists.php.net to get a copy of this message | ||
From: Nikita Popov <nikita.ppv@gmail.com>
Sorry for the long delay. I've just applied https://github.com/php/php-src/commit/fd07302024bc47082b13b32217147fd39d1e9e61 to the 7.2 branch. Davey, Joe, do we want to take action here for 7.1? It's a pretty severe bias, but fixing it is going to change seed sequences. I think at this point we're too far in the 7.1 cycle to apply this kind of change.I think it is very unlikely that anyone has PHP software that relies on predictable output given a 64-bit seed. And, yes, the bias is bad so I would not worry about fixing it asap. Tom