You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ri...@apache.org on 2009/04/11 02:15:33 UTC
svn commit: r764116 -
/qpid/branches/0.5-release/qpid/java/common/src/main/java/org/apache/qpid/util/FileUtils.java
Author: ritchiem
Date: Sat Apr 11 00:15:33 2009
New Revision: 764116
URL: http://svn.apache.org/viewvc?rev=764116&view=rev
Log:
QPID-1803 : Fixed potential NPE in FileUtils.delete
merged from trunk r764076
Modified:
qpid/branches/0.5-release/qpid/java/common/src/main/java/org/apache/qpid/util/FileUtils.java
Modified: qpid/branches/0.5-release/qpid/java/common/src/main/java/org/apache/qpid/util/FileUtils.java
URL: http://svn.apache.org/viewvc/qpid/branches/0.5-release/qpid/java/common/src/main/java/org/apache/qpid/util/FileUtils.java?rev=764116&r1=764115&r2=764116&view=diff
==============================================================================
--- qpid/branches/0.5-release/qpid/java/common/src/main/java/org/apache/qpid/util/FileUtils.java (original)
+++ qpid/branches/0.5-release/qpid/java/common/src/main/java/org/apache/qpid/util/FileUtils.java Sat Apr 11 00:15:33 2009
@@ -250,9 +250,17 @@
{
if (recursive)
{
- for (File subFile : file.listFiles())
+ File[] files = file.listFiles();
+
+ // This can occur if the file is deleted outside the JVM
+ if (files == null)
+ {
+ return false;
+ }
+
+ for (int i = 0; i < files.length; i++)
{
- success = delete(subFile, true) && success;
+ success = delete(files[i], true) && success;
}
return success && file.delete();
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org