You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by ga...@apache.org on 2010/01/26 07:14:53 UTC

svn commit: r903093 - /geronimo/server/trunk/plugins/jetty8/geronimo-jetty8/src/main/java/org/apache/geronimo/jetty8/handler/GeronimoWebAppContext.java

Author: gawor
Date: Tue Jan 26 06:14:52 2010
New Revision: 903093

URL: http://svn.apache.org/viewvc?rev=903093&view=rev
Log:
help prevent NPE in java.util.zip.Inflater on Jetty

Modified:
    geronimo/server/trunk/plugins/jetty8/geronimo-jetty8/src/main/java/org/apache/geronimo/jetty8/handler/GeronimoWebAppContext.java

Modified: geronimo/server/trunk/plugins/jetty8/geronimo-jetty8/src/main/java/org/apache/geronimo/jetty8/handler/GeronimoWebAppContext.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/jetty8/geronimo-jetty8/src/main/java/org/apache/geronimo/jetty8/handler/GeronimoWebAppContext.java?rev=903093&r1=903092&r2=903093&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/jetty8/geronimo-jetty8/src/main/java/org/apache/geronimo/jetty8/handler/GeronimoWebAppContext.java (original)
+++ geronimo/server/trunk/plugins/jetty8/geronimo-jetty8/src/main/java/org/apache/geronimo/jetty8/handler/GeronimoWebAppContext.java Tue Jan 26 06:14:52 2010
@@ -139,5 +139,14 @@
         protected BasicURLResource(URL url) {
             super(url, null);
         }
+        
+        /* 
+         * Always return true as we are pretty sure the resource does exist. This prevents 
+         * NPE as described at https://bugs.eclipse.org/bugs/show_bug.cgi?id=193269
+         */
+        @Override
+        public boolean exists() {
+            return true;
+        }
     }
 }