-
-
Notifications
You must be signed in to change notification settings - Fork 15.8k
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
SslHandlerTest#testTruncatedPacket ByteBuf leak #6057
Comments
@normanmaurer - FYI |
Closed
Scottmitch
added a commit
to Scottmitch/netty
that referenced
this issue
Dec 1, 2016
Motivation: We are now more careful to flush alerts that are generated when errors occur. We should also be more careful in unit tests to release any buffers that may be queued due to potential errors wich result in alerts. Modifications: - When SslHandlerTest uses EmbeddedChannel we should always call finishAndReleaseAll Results: Fixes netty#6057
This was referenced Dec 1, 2016
normanmaurer
pushed a commit
that referenced
this issue
Dec 1, 2016
Motivation: We are now more careful to flush alerts that are generated when errors occur. We should also be more careful in unit tests to release any buffers that may be queued due to potential errors wich result in alerts. Modifications: - When SslHandlerTest uses EmbeddedChannel we should always call finishAndReleaseAll Results: Fixes #6057
liuzhengyang
pushed a commit
to liuzhengyang/netty
that referenced
this issue
Sep 10, 2017
Motivation: We are now more careful to flush alerts that are generated when errors occur. We should also be more careful in unit tests to release any buffers that may be queued due to potential errors wich result in alerts. Modifications: - When SslHandlerTest uses EmbeddedChannel we should always call finishAndReleaseAll Results: Fixes netty#6057
pulllock
pushed a commit
to pulllock/netty
that referenced
this issue
Oct 19, 2023
Motivation: We are now more careful to flush alerts that are generated when errors occur. We should also be more careful in unit tests to release any buffers that may be queued due to potential errors wich result in alerts. Modifications: - When SslHandlerTest uses EmbeddedChannel we should always call finishAndReleaseAll Results: Fixes netty#6057
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Leak uncovered while building PR #6054
https://garage.netty.io/teamcity/viewLog.html?tab=buildLog&logTab=tree&filter=debug&expand=all&buildId=13958#_focus=69576
The text was updated successfully, but these errors were encountered: