Skip to content

ext/zip: fix memory leak when encryption is passed as userland array … - #20363

Closed
devnexen wants to merge 1 commit into
php:PHP-8.3from
devnexen:fix_options_withencryption_leak
Closed

devnexen wants to merge 1 commit into
php:PHP-8.3from
devnexen:fix_options_withencryption_leak

Conversation

@devnexen

@devnexen devnexen commented Nov 2, 2025

Copy link
Copy Markdown
Member

…option.

Similar issue fixed in GH-19936.

@devnexen
devnexen marked this pull request as ready for review November 2, 2025 17:13

@ndossche ndossche left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes me wonder if we should introduce wrapper functions for these

@devnexen

devnexen commented Nov 2, 2025

Copy link
Copy Markdown
Member Author

not a bad idea

@devnexen devnexen closed this in 55f7303 Nov 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants