Integrated: 8275811 Incorrect instance to dispose
Daniel Jeliński
duke at openjdk.java.net
Tue Nov 16 20:37:45 UTC 2021
On Fri, 22 Oct 2021 18:24:22 GMT, Daniel Jeliński <duke at openjdk.java.net> wrote:
> The current code that changes cipher suites disposes the new suite instead of the old one, which usually silently fails. This patch fixes the code to dispose the old instance instead.
>
> DTLS appears to be unaffected: DTLSOutputRecord keeps 2 ciphers and correctly [disposes the old one](https://github.com/openjdk/jdk/blob/739769c8fc4b496f08a92225a12d07414537b6c0/src/java.base/share/classes/sun/security/ssl/DTLSOutputRecord.java#L106), and DTLSInputRecord [doesn't dispose anything](https://github.com/openjdk/jdk/blob/4b9303b77b43d890ebacbec38b4ac5db7e171886/src/java.base/share/classes/sun/security/ssl/DTLSInputRecord.java#L57)
This pull request has now been integrated.
Changeset: cddc6ce4
Author: Daniel Jelinski <djelinski1 at gmail.com>
Committer: Xue-Lei Andrew Fan <xuelei at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/cddc6ce44695cba4614c3405eb2b194d7c76489b
Stats: 35 lines in 4 files changed: 32 ins; 0 del; 3 mod
8275811: Incorrect instance to dispose
Reviewed-by: xuelei
-------------
PR: https://git.openjdk.java.net/jdk/pull/6084
More information about the security-dev
mailing list