RFR+RFA [11u] 8214339: SSLSocketImpl erroneously wraps SocketException

Langer, Christoph christoph.langer at sap.com
Tue Feb 19 15:33:39 UTC 2019


Looks good.

> -----Original Message-----
> From: jdk-updates-dev <jdk-updates-dev-bounces at openjdk.java.net> On
> Behalf Of Aleksey Shipilev
> Sent: Dienstag, 19. Februar 2019 15:28
> To: jdk-updates-dev at openjdk.java.net
> Subject: RFR+RFA [11u] 8214339: SSLSocketImpl erroneously wraps
> SocketException
> 
> Please review and approve the backport to 11u for this bug.
> 
>   Original Bug:
>       URL: https://bugs.openjdk.java.net/browse/JDK-8214339
>       Reporter: Webbug Group
>       Assignee: Xue-Lei Fan
>       Priority: P3
>       Components: security-libs
> 
>   Original Fix:
>         12: JDK-8214339,
> http://hg.openjdk.java.net/jdk/jdk12/rev/9041178a0b69, 66 day(s) ago
> 
>   Backports and Forwardports:
>         13: 13, JDK-8215663,
> http://hg.openjdk.java.net/jdk/jdk/rev/9041178a0b69, 62 day(s) ago
>         11: MISSING
>          8: Not affected
> 
> The patch does not apply cleanly to 11u, because it has trivial formatting
> conflict:
> 
> $ cat
> src/java.base/share/classes/sun/security/ssl/PreSharedKeyExtension.java.r
> ej
> --- PreSharedKeyExtension.java
> +++ PreSharedKeyExtension.java
> @@ -171,7 +171,7 @@
> 
> -            for(PskIdentity curId : identities) {
> +            for (PskIdentity curId : identities) {
>                  idEncodedLength += curId.getEncodedLength();
> 
> @@ -194,7 +194,7 @@
>              Record.putInt16(m, idsEncodedLength);
> -            for(PskIdentity curId : identities) {
> +            for (PskIdentity curId : identities) {
>                  curId.writeEncoded(m);
> 
> That code is already in this form in 11u.
> 
> Full webrev for 11u:
>   http://cr.openjdk.java.net/~shade/8214339/webrev.11u.01/
> 
> Testing: new test, entire jdk_security test suite; verified new test fails
> without the patch
> 
> Thanks,
> -Aleksey



More information about the jdk-updates-dev mailing list