Return empty lists instead of null to prevent null pointer exception happening in Collections.sort()

This commit is contained in:
Eamonn Saunders 2015-05-29 14:05:09 -04:00
parent a949403cb2
commit 20ef8d397f

View File

@ -178,7 +178,7 @@ import org.sleuthkit.datamodel.VolumeSystem;
} catch (TskCoreException tce) { } catch (TskCoreException tce) {
logger.log(Level.WARNING, "Couldn't get a list of Unallocated Files, failed at sending out the visitor ", tce); //NON-NLS logger.log(Level.WARNING, "Couldn't get a list of Unallocated Files, failed at sending out the visitor ", tce); //NON-NLS
} }
return null; return Collections.emptyList();
} }
/** /**
@ -372,7 +372,7 @@ import org.sleuthkit.datamodel.VolumeSystem;
* return the single instance of unallocated space. * return the single instance of unallocated space.
* *
* @param lf the LayoutFile the visitor encountered * @param lf the LayoutFile the visitor encountered
* @return A list<LayoutFile> of size 1, returns null if it fails * @return A list<LayoutFile> of size 1
*/ */
@Override @Override
public List<LayoutFile> visit(final org.sleuthkit.datamodel.LayoutFile lf) { public List<LayoutFile> visit(final org.sleuthkit.datamodel.LayoutFile lf) {
@ -389,7 +389,7 @@ import org.sleuthkit.datamodel.VolumeSystem;
* *
* @param fs the FileSystem the visitor encountered * @param fs the FileSystem the visitor encountered
* @return A list<LayoutFile> containing the layout files from * @return A list<LayoutFile> containing the layout files from
* subsequent Visits(), returns null if it fails * subsequent Visits(), or an empty list
*/ */
@Override @Override
public List<LayoutFile> visit(FileSystem fs) { public List<LayoutFile> visit(FileSystem fs) {
@ -402,7 +402,7 @@ import org.sleuthkit.datamodel.VolumeSystem;
} catch (TskCoreException tce) { } catch (TskCoreException tce) {
logger.log(Level.WARNING, "Couldn't get a list of Unallocated Files, failed at visiting FileSystem " + fs.getId(), tce); //NON-NLS logger.log(Level.WARNING, "Couldn't get a list of Unallocated Files, failed at visiting FileSystem " + fs.getId(), tce); //NON-NLS
} }
return null; return Collections.emptyList();
} }
/** /**
@ -410,12 +410,12 @@ import org.sleuthkit.datamodel.VolumeSystem;
* *
* @param vd VirtualDirectory the visitor encountered * @param vd VirtualDirectory the visitor encountered
* @return A list<LayoutFile> containing all the LayoutFile in ld, * @return A list<LayoutFile> containing all the LayoutFile in ld,
* returns null if it fails * or an empty list.
*/ */
@Override @Override
public List<LayoutFile> visit(VirtualDirectory vd) { public List<LayoutFile> visit(VirtualDirectory vd) {
try { try {
List<LayoutFile> lflst = new ArrayList<LayoutFile>(); List<LayoutFile> lflst = new ArrayList<>();
for (Content layout : vd.getChildren()) { for (Content layout : vd.getChildren()) {
lflst.add((LayoutFile) layout); lflst.add((LayoutFile) layout);
} }
@ -423,7 +423,7 @@ import org.sleuthkit.datamodel.VolumeSystem;
} catch (TskCoreException tce) { } catch (TskCoreException tce) {
logger.log(Level.WARNING, "Could not get list of Layout Files, failed at visiting Layout Directory", tce); //NON-NLS logger.log(Level.WARNING, "Could not get list of Layout Files, failed at visiting Layout Directory", tce); //NON-NLS
} }
return null; return Collections.emptyList();
} }
/** /**
@ -432,7 +432,7 @@ import org.sleuthkit.datamodel.VolumeSystem;
* *
* @param dir the directory this visitor encountered * @param dir the directory this visitor encountered
* @return A list<LayoutFile> containing LayoutFiles encountered during * @return A list<LayoutFile> containing LayoutFiles encountered during
* subsequent Visits(), returns null if it fails * subsequent Visits(), or an empty list.
*/ */
@Override @Override
public List<LayoutFile> visit(Directory dir) { public List<LayoutFile> visit(Directory dir) {
@ -445,12 +445,12 @@ import org.sleuthkit.datamodel.VolumeSystem;
} catch (TskCoreException tce) { } catch (TskCoreException tce) {
logger.log(Level.WARNING, "Couldn't get a list of Unallocated Files, failed at visiting Directory " + dir.getId(), tce); //NON-NLS logger.log(Level.WARNING, "Couldn't get a list of Unallocated Files, failed at visiting Directory " + dir.getId(), tce); //NON-NLS
} }
return null; return Collections.emptyList();
} }
@Override @Override
protected List<LayoutFile> defaultVisit(Content cntnt) { protected List<LayoutFile> defaultVisit(Content cntnt) {
return null; return Collections.emptyList();
} }
} }