Fix possible NPE in FileUtils.
Change-Id: I503f91e266c71e2370a5807d171e2254c334f7cb
This commit is contained in:
parent
6bca9ac43d
commit
86da47e8d7
1 changed files with 8 additions and 2 deletions
|
@ -25,8 +25,11 @@ import java.io.FilenameFilter;
|
||||||
public class FileUtils {
|
public class FileUtils {
|
||||||
public static boolean deleteRecursively(final File path) {
|
public static boolean deleteRecursively(final File path) {
|
||||||
if (path.isDirectory()) {
|
if (path.isDirectory()) {
|
||||||
for (final File child : path.listFiles()) {
|
final File[] files = path.listFiles();
|
||||||
deleteRecursively(child);
|
if (files != null) {
|
||||||
|
for (final File child : files) {
|
||||||
|
deleteRecursively(child);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return path.delete();
|
return path.delete();
|
||||||
|
@ -37,6 +40,9 @@ public class FileUtils {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
final File[] files = dir.listFiles(fileNameFilter);
|
final File[] files = dir.listFiles(fileNameFilter);
|
||||||
|
if (files == null) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
boolean hasDeletedAllFiles = true;
|
boolean hasDeletedAllFiles = true;
|
||||||
for (final File file : files) {
|
for (final File file : files) {
|
||||||
if (!deleteRecursively(file)) {
|
if (!deleteRecursively(file)) {
|
||||||
|
|
Loading…
Reference in a new issue