Page MenuHomePhorge

Remove checks for random_bytes and openssl_random_pseudo_bytes
ClosedPublic

Authored by aklapper on Mon, Jun 9, 12:28.
Tags
None
Referenced Files
F5440643: D26071.1750065670.diff
Sun, Jun 15, 09:21
F5440466: D26071.1750065516.diff
Sun, Jun 15, 09:18
F5440040: D26071.1750065155.diff
Sun, Jun 15, 09:12
F5407993: D26071.1750041568.diff
Sun, Jun 15, 02:39
F5383891: D26071.1750024648.diff
Sat, Jun 14, 21:57
F5378752: D26071.1750020997.diff
Sat, Jun 14, 20:56
F5354848: D26071.1750003815.diff
Sat, Jun 14, 16:10
F5339471: D26071.1749990064.diff
Sat, Jun 14, 12:21

Details

Summary

Per https://www.php.net/manual/en/function.random-bytes.php, random_bytes() has been available since PHP 7.0.
(And per https://www.php.net/manual/en/function.openssl-random-pseudo-bytes.php, openssl_random_pseudo_bytes() has been available since PHP 5.3.0 but that's irrelevant here.)
Phorge requires PHP 7.2 since rP7d8c84a7bdc8fb43674341b97c36c9d8ae1d894a so also remove these pre-PHP 7.2 checks in Arcanist.

Test Plan

Read the code.

Diff Detail

Repository
rARC Arcanist
Branch
php530openssl (branched from master)
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/filesystem/Filesystem.php:447XHP45PHP Compatibility
Unit
Tests Passed
Build Status
Buildable 2075
Build 2075: arc lint + arc unit

Event Timeline

aklapper requested review of this revision.Mon, Jun 9, 12:28

Also bump PHP version check in support/init/init-script.php

Bump PHP version also in .arclint

To the CentOS 7 user surfing to this patch: please kill your server with as much violence as possible, since your server is a security hole with software around.

This revision is now accepted and ready to land.Thu, Jun 12, 07:18