Minor fix to named group selection. JDK-8171279-XDH-TLS-branch-2
authorapetcher
Fri, 17 Aug 2018 10:44:25 -0400
branchJDK-8171279-XDH-TLS-branch-2
changeset 56860 9ce739c1ae20
parent 56859 6f9f7f4100c7
child 56861 3c83db004067
Minor fix to named group selection.
src/java.base/share/classes/sun/security/ssl/SupportedGroupsExtension.java
src/java.base/share/classes/sun/security/ssl/X509Authentication.java
--- a/src/java.base/share/classes/sun/security/ssl/SupportedGroupsExtension.java	Fri Aug 17 10:35:01 2018 -0400
+++ b/src/java.base/share/classes/sun/security/ssl/SupportedGroupsExtension.java	Fri Aug 17 10:44:25 2018 -0400
@@ -805,7 +805,7 @@
         }
 
         // check whether the group is supported by the underlying providers
-        private static boolean isAvailableGroup(NamedGroup namedGroup) {
+        public static boolean isAvailableGroup(NamedGroup namedGroup) {
 
             Optional<NamedGroupFunctions> ngf = namedGroup.getFunctions();
             if (ngf.isEmpty()) {
--- a/src/java.base/share/classes/sun/security/ssl/X509Authentication.java	Fri Aug 17 10:35:01 2018 -0400
+++ b/src/java.base/share/classes/sun/security/ssl/X509Authentication.java	Fri Aug 17 10:44:25 2018 -0400
@@ -290,7 +290,7 @@
                         ((ECPublicKey)serverPublicKey).getParams();
                 NamedGroup namedGroup = NamedGroup.valueOf(params);
                 if ((namedGroup == null) ||
-                    (!SupportedGroups.isSupported(namedGroup))) {
+                    (!SupportedGroups.isAvailableGroup(namedGroup))) {
 
                     if (SSLLogger.isOn && SSLLogger.isOn("ssl")) {
                         SSLLogger.warning(