8279066: entries.remove(entry) is useless in PKCS12KeyStore

Reviewed-by: mullan
openjdk · Dec 21, 2021 · fb623f1 · fb623f1 · openjdk-notifier · Dec 21, 2021
1 parent 803cb8a
commit fb623f1
Showing 1 changed file with 0 additions and 5 deletions.
diff --git a/src/java.base/share/classes/sun/security/pkcs12/PKCS12KeyStore.java b/src/java.base/share/classes/sun/security/pkcs12/PKCS12KeyStore.java
@@ -2265,11 +2265,6 @@ public synchronized void engineLoad(InputStream stream, char[] password)
                 /* Update existing KeyEntry in entries table */
                 if (chain.size() > 0) {
                     entry.chain = chain.toArray(new Certificate[chain.size()]);
-                } else {
-                    // Remove private key entries where there is no associated
-                    // certs. Most likely the keystore is loaded with a null
-                    // password.
-                    entries.remove(entry);
                 }
             }
         }