# HG changeset patch
# User igerasim
# Date 1533713527 25200
# Node ID ed17b461784df205b8f81387aa04062de054a918
# Parent  8d9cb2b095e532e396d65796b847699fdd8458c5
8208754: The fix for JDK-8194534 needs updates
Reviewed-by: alanb, igerasim, rhalade, mullan

diff -r 8d9cb2b095e5 -r ed17b461784d src/share/classes/java/util/jar/JarFile.java
--- a/src/share/classes/java/util/jar/JarFile.java	Wed Aug 01 11:16:50 2018 -0700
+++ b/src/share/classes/java/util/jar/JarFile.java	Wed Aug 08 00:32:07 2018 -0700
@@ -549,7 +549,7 @@
         return false;
     }
 
-    private synchronized void ensureInitialization() {
+    synchronized void ensureInitialization() {
         try {
             maybeInstantiateVerifier();
         } catch (IOException e) {
diff -r 8d9cb2b095e5 -r ed17b461784d src/share/classes/java/util/jar/JavaUtilJarAccessImpl.java
--- a/src/share/classes/java/util/jar/JavaUtilJarAccessImpl.java	Wed Aug 01 11:16:50 2018 -0700
+++ b/src/share/classes/java/util/jar/JavaUtilJarAccessImpl.java	Wed Aug 08 00:32:07 2018 -0700
@@ -65,4 +65,7 @@
         return man.getTrustedAttributes(name);
     }
 
+    public void ensureInitialization(JarFile jar) {
+        jar.ensureInitialization();
+    }
 }
diff -r 8d9cb2b095e5 -r ed17b461784d src/share/classes/java/util/jar/Manifest.java
--- a/src/share/classes/java/util/jar/Manifest.java	Wed Aug 01 11:16:50 2018 -0700
+++ b/src/share/classes/java/util/jar/Manifest.java	Wed Aug 08 00:32:07 2018 -0700
@@ -150,6 +150,10 @@
      *      does not exist in SF files of all signers).
      */
     Attributes getTrustedAttributes(String name) {
+        // Note: Before the verification of MANIFEST.MF/.SF/.RSA files is done,
+        // jv.isTrustedManifestEntry() isn't able to detect MANIFEST.MF change.
+        // Users of this method should call SharedSecrets.javaUtilJarAccess()
+        // .ensureInitialization() first.
         Attributes result = getAttributes(name);
         if (result != null && jv != null && ! jv.isTrustedManifestEntry(name)) {
             throw new SecurityException("Untrusted manifest entry: " + name);
diff -r 8d9cb2b095e5 -r ed17b461784d src/share/classes/sun/misc/JavaUtilJarAccess.java
--- a/src/share/classes/sun/misc/JavaUtilJarAccess.java	Wed Aug 01 11:16:50 2018 -0700
+++ b/src/share/classes/sun/misc/JavaUtilJarAccess.java	Wed Aug 08 00:32:07 2018 -0700
@@ -44,4 +44,5 @@
     public void setEagerValidation(JarFile jar, boolean eager);
     public List getManifestDigests(JarFile jar);
     public Attributes getTrustedAttributes(Manifest man, String name);
+    public void ensureInitialization(JarFile jar);
 }
diff -r 8d9cb2b095e5 -r ed17b461784d src/share/classes/sun/misc/URLClassPath.java
--- a/src/share/classes/sun/misc/URLClassPath.java	Wed Aug 01 11:16:50 2018 -0700
+++ b/src/share/classes/sun/misc/URLClassPath.java	Wed Aug 08 00:32:07 2018 -0700
@@ -760,8 +760,10 @@
                     { return jar.getInputStream(entry); }
                 public int getContentLength()
                     { return (int)entry.getSize(); }
-                public Manifest getManifest() throws IOException
-                    { return jar.getManifest(); };
+                public Manifest getManifest() throws IOException {
+                    SharedSecrets.javaUtilJarAccess().ensureInitialization(jar);
+                    return jar.getManifest();
+                }
                 public Certificate[] getCertificates()
                     { return entry.getCertificates(); };
                 public CodeSigner[] getCodeSigners()