From d6deed89f75480dc770488a9310f69a71dec3e75 Mon Sep 17 00:00:00 2001 From: Martin Ritchie Date: Mon, 2 Mar 2009 15:12:47 +0000 Subject: Added check for when file reference becomes null, occurs if the file is externally deleted. Also noted success return value was never propogated correctly, Updated : Test needed. git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@749330 13f79535-47bb-0310-9956-ffa450edef68 --- java/common/src/main/java/org/apache/qpid/util/FileUtils.java | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'java/common') diff --git a/java/common/src/main/java/org/apache/qpid/util/FileUtils.java b/java/common/src/main/java/org/apache/qpid/util/FileUtils.java index 3e13259ee3..3d43b9d511 100644 --- a/java/common/src/main/java/org/apache/qpid/util/FileUtils.java +++ b/java/common/src/main/java/org/apache/qpid/util/FileUtils.java @@ -250,12 +250,17 @@ public class FileUtils { if (recursive) { - for (File subFile : file.listFiles()) + try{ + for (File subFile : file.listFiles()) + { + success = delete(subFile, true) & success ; + } + }catch (NullPointerException npe) { - success = delete(subFile, true) & success ; + success = false; } - return file.delete(); + return success && file.delete(); } return false; -- cgit v1.2.1