Skip to content

JAVA-3075: Possible memory leak in FrameEncode#encode #1713

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
Open
wants to merge 1 commit into
base: 4.x
Choose a base branch
from
Open

JAVA-3075: Possible memory leak in FrameEncode#encode #1713

wants to merge 1 commit into from
+133 −8

Conversation

@hhughes
Copy link
Contributor

@hhughes hhughes commented Aug 21, 2023

FrameEncoder.java

  • release encoded frame when throwing FrameTooLongException

FrameEncoderTest.java

  • test successful frame encode does not leak
  • test failing frame encode does not leak
@hhughes hhughes mentioned this pull request Aug 21, 2023
Closed
@hhughes
JAVA-3075: Possible memory leak in FrameEncode#encode
4b88a5e 
FrameEncoder.java - release encoded frame when throwing FrameTooLongException FrameEncoderTest.java - test successful frame encode does not leak - test failing frame encode does not leak
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@hhughes