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.

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

Jump to bottom

[UNDERTOW-2252] Potential fix for the SslConduit.dataToUnwrap buffer leak #1452

Merged
merged 1 commit into from
Mar 25, 2023
Merged

[UNDERTOW-2252] Potential fix for the SslConduit.dataToUnwrap buffer leak #1452

merged 1 commit into from
Mar 25, 2023

Conversation

fl4via
Copy link
Member

@fl4via fl4via commented Mar 25, 2023 •
edited

@fl4via fl4via added the under verification Currently being verified (running tests, reviewing) before posting a review to contributor label Mar 25, 2023
@fl4via
[UNDERTOW-2252] Potential fix for the SslConduit.dataToUnwrap buffer …
a9fe3d0 
…leak

Signed-off-by: Flavia Rainone <frainone@redhat.com>
@fl4via fl4via force-pushed the INVESTIGATE_BUFFER_LEAK_SSLCONDUIT branch from df9b4b1 to a9fe3d0 Compare March 25, 2023 12:37
@fl4via fl4via added bug fix Contains bug fix(es) next release This PR will be merged before next release or has already been merged (for payload double check) waiting CI check Ready to be merged but waiting for CI check labels Mar 25, 2023
@fl4via fl4via changed the title Investigate the buffer leak, is this possibly the cause of the failure? [UNDERTOW-2252] Potential fix for the SslConduit.dataToUnwrap buffer leak Mar 25, 2023
@fl4via fl4via removed under verification Currently being verified (running tests, reviewing) before posting a review to contributor waiting CI check Ready to be merged but waiting for CI check labels Mar 25, 2023
@fl4via fl4via merged commit 86bf19d into undertow-io:master Mar 25, 2023
@fl4via fl4via deleted the INVESTIGATE_BUFFER_LEAK_SSLCONDUIT branch March 25, 2023 13:33
@fl4via fl4via removed the next release This PR will be merged before next release or has already been merged (for payload double check) label Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug fix Contains bug fix(es)
Projects
None yet
Milestone
No milestone
1 participant
@fl4via