You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mm...@apache.org on 2006/10/18 18:17:12 UTC

svn commit: r465289 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/config/FacesConfigurator.java

Author: mmarinschek
Date: Wed Oct 18 09:17:11 2006
New Revision: 465289

URL: http://svn.apache.org/viewvc?view=rev&rev=465289
Log:
fixed MYFACES-1428: Application cannot be started with Snapshot 1.1.5 of 27th Sept./1st Oct.

Modified:
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/config/FacesConfigurator.java

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/config/FacesConfigurator.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/config/FacesConfigurator.java?view=diff&rev=465289&r1=465288&r2=465289
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/config/FacesConfigurator.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/config/FacesConfigurator.java Wed Oct 18 09:17:11 2006
@@ -242,9 +242,18 @@
             int beginIndex = index+versionInfo.getPackageName().length()+1;
 
             if(beginIndex > fileName.length()-1)
+            {
+                log.debug("beginIndex out of bounds. fileName: "+fileName);
                 return false;
+            }
 
             int endIndex = fileName.length()-JAR_EXTENSION.length();
+
+            if(endIndex<0 || endIndex<=beginIndex)
+            {
+                log.debug("endIndex out of bounds. fileName: "+fileName);
+                return false;
+            }
 
             String newVersion = fileName.substring(beginIndex, endIndex);