On Fri, 22 Oct 2021 18:24:22 GMT, Daniel Jeliński <d...@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 <djelins...@gmail.com> Committer: Xue-Lei Andrew Fan <xue...@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
