Merge "Don't throw exceptions from BCPrivate provider." am: d022e6ef8f am: ed07aeb495 am: 0c003a58d2

Original change: https://android-review.googlesource.com/c/platform/external/bouncycastle/+/2382712

Change-Id: Ic6d7d82323cd436e61fc90aa01c20f541425fb94
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

3 files changed, 69 insertions, 21 deletions

diff --git a/bcprov/src/main/java/org/bouncycastle/jcajce/util/BCJcaJceHelper.java b/bcprov/src/main/java/org/bouncycastle/jcajce/util/BCJcaJceHelper.java
index 9a41d31c..13205ec2 100644
--- a/bcprov/src/main/java/org/bouncycastle/jcajce/util/BCJcaJceHelper.java
+++ b/bcprov/src/main/java/org/bouncycastle/jcajce/util/BCJcaJceHelper.java
@@ -48,13 +48,21 @@ public class BCJcaJceHelper
     //
     // If code is using a BCJcajceHelper to ensure it gets its implementation from BC, then
     // also search in the privately provided algorithms if not found in the main set.
+    //
+    // If any error occurs while searching the private Provider, typically a
+    // NoSuchAlgorithmException being thrown, then the original NoSuchAlgorithmException
+    // from the BC Provider is thrown for consistency.
     @Override
     public Cipher createCipher(String algorithm)
         throws NoSuchAlgorithmException, NoSuchPaddingException {
         try {
             return super.createCipher(algorithm);
-        } catch (NoSuchAlgorithmException e) {
-            return Cipher.getInstance(algorithm, getPrivateProvider());
+        } catch (NoSuchAlgorithmException originalException) {
+            try {
+                return Cipher.getInstance(algorithm, getPrivateProvider());
+            } catch (Throwable throwable) {
+                throw originalException;
+            }
         }
     }
 
@@ -63,8 +71,12 @@ public class BCJcaJceHelper
         throws NoSuchAlgorithmException {
         try {
             return super.createSecretKeyFactory(algorithm);
-        } catch (NoSuchAlgorithmException e) {
-            return SecretKeyFactory.getInstance(algorithm, getPrivateProvider());
+        } catch (NoSuchAlgorithmException originalException) {
+            try {
+                return SecretKeyFactory.getInstance(algorithm, getPrivateProvider());
+            } catch (Throwable throwable) {
+                throw originalException;
+            }
         }
     }
 
@@ -72,8 +84,12 @@ public class BCJcaJceHelper
     public Mac createMac(String algorithm) throws NoSuchAlgorithmException {
         try {
             return super.createMac(algorithm);
-        } catch (NoSuchAlgorithmException e) {
-            return Mac.getInstance(algorithm, getPrivateProvider());
+        } catch (NoSuchAlgorithmException originalException) {
+            try {
+                return Mac.getInstance(algorithm, getPrivateProvider());
+            } catch (Throwable throwable) {
+                throw originalException;
+            }
         }
     }
 
@@ -81,7 +97,7 @@ public class BCJcaJceHelper
         if (provider instanceof BouncyCastleProvider) {
             return ((BouncyCastleProvider) provider).getPrivateProvider();
         }
-        throw new IllegalStateException(); // XXX
+        throw new IllegalStateException("Internal error in BCJcaJceHelper");
     }
     // END Android-added: Look up algorithms in private provider if not found in main Provider.
 }
diff --git a/repackaged/bcprov/src/main/java/com/android/org/bouncycastle/jcajce/util/BCJcaJceHelper.java b/repackaged/bcprov/src/main/java/com/android/org/bouncycastle/jcajce/util/BCJcaJceHelper.java
index b444878f..69ab946c 100644
--- a/repackaged/bcprov/src/main/java/com/android/org/bouncycastle/jcajce/util/BCJcaJceHelper.java
+++ b/repackaged/bcprov/src/main/java/com/android/org/bouncycastle/jcajce/util/BCJcaJceHelper.java
@@ -50,13 +50,21 @@ public class BCJcaJceHelper
     //
     // If code is using a BCJcajceHelper to ensure it gets its implementation from BC, then
     // also search in the privately provided algorithms if not found in the main set.
+    //
+    // If any error occurs while searching the private Provider, typically a
+    // NoSuchAlgorithmException being thrown, then the original NoSuchAlgorithmException
+    // from the BC Provider is thrown for consistency.
     @Override
     public Cipher createCipher(String algorithm)
         throws NoSuchAlgorithmException, NoSuchPaddingException {
         try {
             return super.createCipher(algorithm);
-        } catch (NoSuchAlgorithmException e) {
-            return Cipher.getInstance(algorithm, getPrivateProvider());
+        } catch (NoSuchAlgorithmException originalException) {
+            try {
+                return Cipher.getInstance(algorithm, getPrivateProvider());
+            } catch (Throwable throwable) {
+                throw originalException;
+            }
         }
     }
 
@@ -65,8 +73,12 @@ public class BCJcaJceHelper
         throws NoSuchAlgorithmException {
         try {
             return super.createSecretKeyFactory(algorithm);
-        } catch (NoSuchAlgorithmException e) {
-            return SecretKeyFactory.getInstance(algorithm, getPrivateProvider());
+        } catch (NoSuchAlgorithmException originalException) {
+            try {
+                return SecretKeyFactory.getInstance(algorithm, getPrivateProvider());
+            } catch (Throwable throwable) {
+                throw originalException;
+            }
         }
     }
 
@@ -74,8 +86,12 @@ public class BCJcaJceHelper
     public Mac createMac(String algorithm) throws NoSuchAlgorithmException {
         try {
             return super.createMac(algorithm);
-        } catch (NoSuchAlgorithmException e) {
-            return Mac.getInstance(algorithm, getPrivateProvider());
+        } catch (NoSuchAlgorithmException originalException) {
+            try {
+                return Mac.getInstance(algorithm, getPrivateProvider());
+            } catch (Throwable throwable) {
+                throw originalException;
+            }
         }
     }
 
@@ -83,7 +99,7 @@ public class BCJcaJceHelper
         if (provider instanceof BouncyCastleProvider) {
             return ((BouncyCastleProvider) provider).getPrivateProvider();
         }
-        throw new IllegalStateException(); // XXX
+        throw new IllegalStateException("Internal error in BCJcaJceHelper");
     }
     // END Android-added: Look up algorithms in private provider if not found in main Provider.
 }
diff --git a/repackaged_platform/bcprov/src/main/java/com/android/internal/org/bouncycastle/jcajce/util/BCJcaJceHelper.java b/repackaged_platform/bcprov/src/main/java/com/android/internal/org/bouncycastle/jcajce/util/BCJcaJceHelper.java
index 7b7c6cb3..507d225c 100644
--- a/repackaged_platform/bcprov/src/main/java/com/android/internal/org/bouncycastle/jcajce/util/BCJcaJceHelper.java
+++ b/repackaged_platform/bcprov/src/main/java/com/android/internal/org/bouncycastle/jcajce/util/BCJcaJceHelper.java
@@ -50,13 +50,21 @@ public class BCJcaJceHelper
     //
     // If code is using a BCJcajceHelper to ensure it gets its implementation from BC, then
     // also search in the privately provided algorithms if not found in the main set.
+    //
+    // If any error occurs while searching the private Provider, typically a
+    // NoSuchAlgorithmException being thrown, then the original NoSuchAlgorithmException
+    // from the BC Provider is thrown for consistency.
     @Override
     public Cipher createCipher(String algorithm)
         throws NoSuchAlgorithmException, NoSuchPaddingException {
         try {
             return super.createCipher(algorithm);
-        } catch (NoSuchAlgorithmException e) {
-            return Cipher.getInstance(algorithm, getPrivateProvider());
+        } catch (NoSuchAlgorithmException originalException) {
+            try {
+                return Cipher.getInstance(algorithm, getPrivateProvider());
+            } catch (Throwable throwable) {
+                throw originalException;
+            }
         }
     }
 
@@ -65,8 +73,12 @@ public class BCJcaJceHelper
         throws NoSuchAlgorithmException {
         try {
             return super.createSecretKeyFactory(algorithm);
-        } catch (NoSuchAlgorithmException e) {
-            return SecretKeyFactory.getInstance(algorithm, getPrivateProvider());
+        } catch (NoSuchAlgorithmException originalException) {
+            try {
+                return SecretKeyFactory.getInstance(algorithm, getPrivateProvider());
+            } catch (Throwable throwable) {
+                throw originalException;
+            }
         }
     }
 
@@ -74,8 +86,12 @@ public class BCJcaJceHelper
     public Mac createMac(String algorithm) throws NoSuchAlgorithmException {
         try {
             return super.createMac(algorithm);
-        } catch (NoSuchAlgorithmException e) {
-            return Mac.getInstance(algorithm, getPrivateProvider());
+        } catch (NoSuchAlgorithmException originalException) {
+            try {
+                return Mac.getInstance(algorithm, getPrivateProvider());
+            } catch (Throwable throwable) {
+                throw originalException;
+            }
         }
     }
 
@@ -83,7 +99,7 @@ public class BCJcaJceHelper
         if (provider instanceof BouncyCastleProvider) {
             return ((BouncyCastleProvider) provider).getPrivateProvider();
         }
-        throw new IllegalStateException(); // XXX
+        throw new IllegalStateException("Internal error in BCJcaJceHelper");
     }
     // END Android-added: Look up algorithms in private provider if not found in main Provider.
 }