Skip to content

Fix memory leak on Randomizer::__construct() call twice #9091

New issue

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 12 commits into from
Jul 23, 2022
Merged

Fix memory leak on Randomizer::__construct() call twice #9091

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
e9a447f
random: fix __construct() call twice
zeriyoshi Jul 22, 2022
4adebfd
update NEWS
zeriyoshi Jul 22, 2022
a095e45
[CI skip] fix typo
zeriyoshi Jul 22, 2022
fc643aa
Merge branch 'master' into randomizer_constructor_twice
zeriyoshi Jul 22, 2022
4afc9db
fix: replace all strlen -> sizeof - 1
zeriyoshi Jul 22, 2022
e9b9118
Update ext/random/tests/03_randomizer/construct_twice.phpt
zeriyoshi Jul 23, 2022
8d38061
Revert "fix: replace all strlen -> sizeof - 1"
zeriyoshi Jul 23, 2022
31b7f9a
random: revert strlen -> sizeof - 1
zeriyoshi Jul 23, 2022
cb5f7ae
random: test: fix test readability
zeriyoshi Jul 23, 2022
e26f709
Merge remote-tracking branch 'upstream/master' into randomizer_constr…
zeriyoshi Jul 23, 2022
da07c17
random: revert sizeof - 1 in randomizer.c
zeriyoshi Jul 23, 2022
8708360
Update NEWS
zeriyoshi Jul 23, 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
5 changes: 5 additions & 0 deletions ext/random/randomizer.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,11 @@ PHP_METHOD(Random_Randomizer, __construct)
Z_PARAM_OBJ_OF_CLASS_OR_NULL(engine_object, random_ce_Random_Engine);
ZEND_PARSE_PARAMETERS_END();

if (randomizer->algo) {
zend_throw_exception_ex(spl_ce_BadMethodCallException, 0, "Cannot call constructor twice");
RETURN_THROWS();
}

/* Create default RNG instance */
if (!engine_object) {
engine_object = random_ce_Random_Engine_Secure->create_object(random_ce_Random_Engine_Secure);
Expand Down
40 changes: 40 additions & 0 deletions ext/random/tests/03_randomizer/construct_twice.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
--TEST--
Random: Randomizer: Disallow manually calling __construct
--FILE--
<?php

final class UserEngine implements \Random\Engine
{
public function generate(): string
{
return \random_byte(4); /* 32-bit */
}
}

try {
(new \Random\Randomizer())->__construct();
} catch (\BadMethodCallException $e) {
echo $e->getMessage() . PHP_EOL;
}

try {
$r = new \Random\Randomizer(new \Random\Engine\Xoshiro256StarStar());
$r->__construct(new \Random\Engine\PcgOneseq128XslRr64());
} catch (\BadMethodCallException $e) {
echo $e->getMessage() . PHP_EOL;
}

try {
$r = new \Random\Randomizer(new \UserEngine());
$r->__construct(new \UserEngine());
} catch (\BadMethodCallException $e) {
echo $e->getMessage() . PHP_EOL;
}

die('success');
?>
--EXPECT--
Cannot call constructor twice
Cannot call constructor twice
Cannot call constructor twice
success