You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2017/10/17 13:20:43 UTC

svn commit: r1812401 - /tomcat/trunk/java/org/apache/tomcat/util/scan/AbstractInputStreamJar.java

Author: markt
Date: Tue Oct 17 13:20:42 2017
New Revision: 1812401

URL: http://svn.apache.org/viewvc?rev=1812401&view=rev
Log:
Correct refactored reset logic

Modified:
    tomcat/trunk/java/org/apache/tomcat/util/scan/AbstractInputStreamJar.java

Modified: tomcat/trunk/java/org/apache/tomcat/util/scan/AbstractInputStreamJar.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/scan/AbstractInputStreamJar.java?rev=1812401&r1=1812400&r2=1812401&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/scan/AbstractInputStreamJar.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/scan/AbstractInputStreamJar.java Tue Oct 17 13:20:42 2017
@@ -189,8 +189,10 @@ public abstract class AbstractInputStrea
 
 
     private void gotoEntry(String name) throws IOException {
+        boolean needsReset = true;
         if (multiRelease == null) {
             reset();
+            needsReset = false;
         }
 
         // Need to convert requested name to multi-release name (if one exists)
@@ -207,6 +209,9 @@ public abstract class AbstractInputStrea
         if (entry != null && name.equals(entry.getName())) {
             return;
         }
+        if (needsReset) {
+            reset();
+        }
 
         JarEntry jarEntry = jarInputStream.getNextJarEntry();
         while (jarEntry != null) {



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org