3 ms·
This is the offending code: https://github.com/bmmayer/ephchat/blob/ec375c7974ea825f887ff4ca1cca2349018a187e/php/functions.php#L4-L20 https://github.com/bmmayer
by sarciszewski 12y ago
This is the offending code:
https://github.com/bmmayer/ephchat/blob/ec375c7974ea825f887ff4ca1cca2349018a187e/php/functions.php#L4-L20 https://github.com/bmmayer/ephchat/blob/ec375c7974ea825f887f...
This is the proper way to do it (h/t @tptacek):
http://sockpuppet.org/blog/2014/02/25/safely-generate-random-numbers/ http://sockpuppet.org/blog/2014/02/25/safely-generate-random...
mt_srand() + rand() is just hilarious.
The md5(uniquid()) thing is a common randomness anti-pattern in PHP projects that needs to die in a fire.
In PHP, a very brief example of the code to achieve the proper way of generating randomness looks like the snippet I posted in the issue.