153 lines
5.8 KiB
Diff
153 lines
5.8 KiB
Diff
# HG changeset patch
|
|
# User andrew
|
|
# Date 1467652889 -3600
|
|
# Mon Jul 04 18:21:29 2016 +0100
|
|
# Node ID a4541d1d8609cadb08d3e31b40b9184ff32dd6c3
|
|
# Parent bc6eab2038c603afb2eb2b4644f3b900c8fd0c46
|
|
PR3083, RH1346460: Regression in SSL debug output without an ECC provider
|
|
Summary: Return null rather than throwing an exception when there's no ECC provider.
|
|
|
|
diff -r bc6eab2038c6 -r a4541d1d8609 src/share/classes/sun/security/util/Debug.java
|
|
--- openjdk/jdk/src/share/classes/sun/security/util/Debug.java Mon Jul 04 17:08:12 2016 +0100
|
|
+++ openjdk/jdk/src/share/classes/sun/security/util/Debug.java Mon Jul 04 18:21:29 2016 +0100
|
|
@@ -73,6 +73,7 @@
|
|
System.err.println("certpath PKIX CertPathBuilder and");
|
|
System.err.println(" CertPathValidator debugging");
|
|
System.err.println("combiner SubjectDomainCombiner debugging");
|
|
+ System.err.println("ecc Elliptic Curve Cryptography debugging");
|
|
System.err.println("gssloginconfig");
|
|
System.err.println(" GSS LoginConfigImpl debugging");
|
|
System.err.println("configfile JAAS ConfigFile loading");
|
|
diff -r bc6eab2038c6 -r a4541d1d8609 src/share/classes/sun/security/util/ECUtil.java
|
|
--- openjdk/jdk/src/share/classes/sun/security/util/ECUtil.java Mon Jul 04 17:08:12 2016 +0100
|
|
+++ openjdk/jdk/src/share/classes/sun/security/util/ECUtil.java Mon Jul 04 18:21:29 2016 +0100
|
|
@@ -41,6 +41,9 @@
|
|
|
|
public class ECUtil {
|
|
|
|
+ /* Are we debugging ? */
|
|
+ private static final Debug debug = Debug.getInstance("ecc");
|
|
+
|
|
// Used by SunPKCS11 and SunJSSE.
|
|
public static ECPoint decodePoint(byte[] data, EllipticCurve curve)
|
|
throws IOException {
|
|
@@ -90,6 +93,10 @@
|
|
}
|
|
|
|
private static AlgorithmParameters getECParameters(Provider p) {
|
|
+ return getECParameters(p, false);
|
|
+ }
|
|
+
|
|
+ private static AlgorithmParameters getECParameters(Provider p, boolean throwException) {
|
|
try {
|
|
if (p != null) {
|
|
return AlgorithmParameters.getInstance("EC", p);
|
|
@@ -97,13 +104,21 @@
|
|
|
|
return AlgorithmParameters.getInstance("EC");
|
|
} catch (NoSuchAlgorithmException nsae) {
|
|
- throw new RuntimeException(nsae);
|
|
+ if (throwException) {
|
|
+ throw new RuntimeException(nsae);
|
|
+ } else {
|
|
+ // ECC provider is optional so just return null
|
|
+ if (debug != null) {
|
|
+ debug.println("Provider unavailable: " + nsae);
|
|
+ }
|
|
+ return null;
|
|
+ }
|
|
}
|
|
}
|
|
|
|
public static byte[] encodeECParameterSpec(Provider p,
|
|
ECParameterSpec spec) {
|
|
- AlgorithmParameters parameters = getECParameters(p);
|
|
+ AlgorithmParameters parameters = getECParameters(p, true);
|
|
|
|
try {
|
|
parameters.init(spec);
|
|
@@ -122,11 +137,16 @@
|
|
public static ECParameterSpec getECParameterSpec(Provider p,
|
|
ECParameterSpec spec) {
|
|
AlgorithmParameters parameters = getECParameters(p);
|
|
+ if (parameters == null)
|
|
+ return null;
|
|
|
|
try {
|
|
parameters.init(spec);
|
|
return parameters.getParameterSpec(ECParameterSpec.class);
|
|
} catch (InvalidParameterSpecException ipse) {
|
|
+ if (debug != null) {
|
|
+ debug.println("Invalid parameter specification: " + ipse);
|
|
+ }
|
|
return null;
|
|
}
|
|
}
|
|
@@ -135,34 +155,49 @@
|
|
byte[] params)
|
|
throws IOException {
|
|
AlgorithmParameters parameters = getECParameters(p);
|
|
+ if (parameters == null)
|
|
+ return null;
|
|
|
|
parameters.init(params);
|
|
|
|
try {
|
|
return parameters.getParameterSpec(ECParameterSpec.class);
|
|
} catch (InvalidParameterSpecException ipse) {
|
|
+ if (debug != null) {
|
|
+ debug.println("Invalid parameter specification: " + ipse);
|
|
+ }
|
|
return null;
|
|
}
|
|
}
|
|
|
|
public static ECParameterSpec getECParameterSpec(Provider p, String name) {
|
|
AlgorithmParameters parameters = getECParameters(p);
|
|
+ if (parameters == null)
|
|
+ return null;
|
|
|
|
try {
|
|
parameters.init(new ECGenParameterSpec(name));
|
|
return parameters.getParameterSpec(ECParameterSpec.class);
|
|
} catch (InvalidParameterSpecException ipse) {
|
|
+ if (debug != null) {
|
|
+ debug.println("Invalid parameter specification: " + ipse);
|
|
+ }
|
|
return null;
|
|
}
|
|
}
|
|
|
|
public static ECParameterSpec getECParameterSpec(Provider p, int keySize) {
|
|
AlgorithmParameters parameters = getECParameters(p);
|
|
+ if (parameters == null)
|
|
+ return null;
|
|
|
|
try {
|
|
parameters.init(new ECKeySizeParameterSpec(keySize));
|
|
return parameters.getParameterSpec(ECParameterSpec.class);
|
|
} catch (InvalidParameterSpecException ipse) {
|
|
+ if (debug != null) {
|
|
+ debug.println("Invalid parameter specification: " + ipse);
|
|
+ }
|
|
return null;
|
|
}
|
|
|
|
@@ -171,11 +206,16 @@
|
|
public static String getCurveName(Provider p, ECParameterSpec spec) {
|
|
ECGenParameterSpec nameSpec;
|
|
AlgorithmParameters parameters = getECParameters(p);
|
|
+ if (parameters == null)
|
|
+ return null;
|
|
|
|
try {
|
|
parameters.init(spec);
|
|
nameSpec = parameters.getParameterSpec(ECGenParameterSpec.class);
|
|
} catch (InvalidParameterSpecException ipse) {
|
|
+ if (debug != null) {
|
|
+ debug.println("Invalid parameter specification: " + ipse);
|
|
+ }
|
|
return null;
|
|
}
|
|
|