You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ta...@apache.org on 2018/05/04 13:28:09 UTC
svn commit: r1830916 -
/myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/view/facelets/impl/DefaultFaceletFactory.java
Author: tandraschko
Date: Fri May 4 13:28:09 2018
New Revision: 1830916
URL: http://svn.apache.org/viewvc?rev=1830916&view=rev
Log:
MYFACES-4228 File Descriptor leak in DefaultFaceletFactory
Modified:
myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/view/facelets/impl/DefaultFaceletFactory.java
Modified: myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/view/facelets/impl/DefaultFaceletFactory.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/view/facelets/impl/DefaultFaceletFactory.java?rev=1830916&r1=1830915&r2=1830916&view=diff
==============================================================================
--- myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/view/facelets/impl/DefaultFaceletFactory.java (original)
+++ myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/view/facelets/impl/DefaultFaceletFactory.java Fri May 4 13:28:09 2018
@@ -333,9 +333,10 @@ public final class DefaultFaceletFactory
{
// Should check for file modification
+ URLConnection conn = null;
try
{
- URLConnection conn = facelet.getSource().openConnection();
+ conn = facelet.getSource().openConnection();
long lastModified = ResourceLoaderUtils.getResourceLastModified(conn);
return lastModified == 0 || lastModified > target;
@@ -344,6 +345,20 @@ public final class DefaultFaceletFactory
{
throw new FaceletException("Error Checking Last Modified for " + facelet.getAlias(), e);
}
+ finally
+ {
+ if (conn != null)
+ {
+ try
+ {
+ conn.getInputStream().close();
+ }
+ catch (Exception e)
+ {
+ // Ignored
+ }
+ }
+ }
}
return false;