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

Posted to commits@felix.apache.org by ri...@apache.org on 2011/11/10 05:42:57 UTC

svn commit: r1200145 - /felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java

Author: rickhall
Date: Thu Nov 10 04:42:57 2011
New Revision: 1200145

URL: http://svn.apache.org/viewvc?rev=1200145&view=rev
Log:
Check for null when listing directories to delete from bundle cache. (FELIX-3211)

Modified:
    felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java

Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java?rev=1200145&r1=1200144&r2=1200145&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java Thu Nov 10 04:42:57 2011
@@ -426,9 +426,12 @@ public class BundleCache
         if (getSecureAction().isFileDirectory(target))
         {
             File[] files = getSecureAction().listDirectory(target);
-            for (int i = 0; i < files.length; i++)
+            if (files != null)
             {
-                deleteDirectoryTreeRecursive(files[i]);
+                for (int i = 0; i < files.length; i++)
+                {
+                    deleteDirectoryTreeRecursive(files[i]);
+                }
             }
         }