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

Changes from 1 commit
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
Prev Previous commit
Next Next commit
Merge branch 'master' into randomizer_constructor_twice
  • Loading branch information
@zeriyoshi
zeriyoshi authored Jul 22, 2022
commit fc643aac76d5ee18b1ff1e8f0483ca953e17da60
2 changes: 0 additions & 2 deletions NEWS
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,6 @@ PHP NEWS

- Random:
. Added new random extension. (Go Kudo)
. Fixed bug GH-9066 (signed integer overflow). (zeriyoshi)
. Fixed bug GH-9089 (Memory leak when calling Randomizer::__construct() on an existing object). (zeriyoshi)

- SPL:
. Widen iterator_to_array() and iterator_count()'s $iterator parameter to
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.