8000 Improve error reporting in random extension by TimWolla · Pull Request #9071 · php/php-src · GitHub
[go: up one dir, main page]
Skip to content

Improve error reporting in random extension #9071

New issue
8000

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Jul 25, 2022
Merged

Improve error reporting in random extension #9071

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
8daa5fe
Use `php_random_bytes_throw()` in Secure engine's generate()
TimWolla Jul 20, 2022
b7bdf1a
Use `php_random_int_throw()` in Secure engine's range()
TimWolla Jul 20, 2022
f93daeb
Throw exception when a user engine returns an empty string
TimWolla Jul 20, 2022
fdfa2f3
Throw exception when the range selector fails to get acceptable numbe…
TimWolla Jul 20, 2022
12b07ce
Improve user_unsafe test
TimWolla Jul 20, 2022
3e18441
Fix user_unsafe test
TimWolla Jul 20, 2022
8adb67a
Remove duplication in user_unsafe.phpt
TimWolla Jul 20, 2022
3090379
Catch `Throwable` in user_unsafe.phpt
TimWolla Jul 21, 2022
39b996a
Throw an error if an engine returns an empty string
TimWolla Jul 24, 2022
734c79c
Throw an Error if range fails to find an acceptable number in 50 atte…
TimWolla Jul 24, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use php_random_int_throw() in Secure engine's range() 
This exposes the underlying exception, improving debugging:

    Exception: Cannot open source device in php-src/test.php:17
    Stack trace:
    #0 php-src/test.php(17): Random\Randomizer->getInt(1, 3)
    #1 {main}

    Next RuntimeException: Random number generation failed in php-src/test.php:17
    Stack trace:
    #0 php-src/test.php(17): Random\Randomizer->getInt(1, 3)
    #1 {main}
  • Loading branch information
@TimWolla
TimWolla committed Jul 20, 2022
commit b7bdf1aca647f9bf834a4224ef693131f25386bc
4 changes: 2 additions & 2 deletions ext/random/engine_secure.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,9 @@ static uint64_t generate(php_random_status *status)

static zend_long range(php_random_status *status, zend_long min, zend_long max)
{
zend_long result;
zend_long result = 0;

if (php_random_int_silent(min, max, &result) == FAILURE) {
if (php_random_int_throw(min, max, &result) == FAILURE) {
status->last_unsafe = true;
}

Expand Down
0