Skip to content
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.

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

msi: Replace OpenSSL with MinGW's latest one to fix memory leak #387

Merged
merged 2 commits into from
May 10, 2022

Conversation

ashie
Copy link
Member

@ashie ashie commented May 9, 2022

Fix #374

Signed-off-by: Takuro Ashie [email protected]

@ashie ashie force-pushed the replace-openssl-in-rubyinstaller branch from 319b52a to 54542bb Compare May 10, 2022 07:41
@ashie ashie marked this pull request as ready for review May 10, 2022 07:45
@ashie ashie requested a review from kenhys May 10, 2022 07:45
Copy link
Contributor

@kenhys kenhys left a comment

Choose a reason for hiding this comment

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

Could you amend a commit message to include a summary of #374 for the record?

ashie added 2 commits May 10, 2022 17:06
MinGW'S OpenSSL 1.1.1.o-1 or before has a memory leak bug in
`X509_STORE_set_default_paths`. Because Fluentd's out_forward plugin
calls it on every communication, memory leaks are accumulated so that
it will causes out of memory.

Fix #374

Signed-off-by: Takuro Ashie <[email protected]>
@ashie ashie force-pushed the replace-openssl-in-rubyinstaller branch from 54542bb to 4712eb0 Compare May 10, 2022 08:21
@ashie
Copy link
Member Author

ashie commented May 10, 2022

Done: 0a4aa1b

@ashie ashie merged commit 91955c5 into master May 10, 2022
@ashie ashie deleted the replace-openssl-in-rubyinstaller branch May 10, 2022 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Memory leak when using TLS on Windows
2 participants