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

Posted to commits@commons.apache.org by bo...@apache.org on 2015/08/18 19:51:22 UTC

svn commit: r1696471 - /commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java

Author: bodewig
Date: Tue Aug 18 17:51:21 2015
New Revision: 1696471

URL: http://svn.apache.org/r1696471
Log:
newer version of findbugs has detected a potential NPE

Modified:
    commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java

Modified: commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java
URL: http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java?rev=1696471&r1=1696470&r2=1696471&view=diff
==============================================================================
--- commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java (original)
+++ commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java Tue Aug 18 17:51:21 2015
@@ -865,9 +865,9 @@ public class TarArchiveEntry implements
         }
 
         String[]   list = file.list();
-        TarArchiveEntry[] result = new TarArchiveEntry[list.length];
+        TarArchiveEntry[] result = new TarArchiveEntry[list == null ? 0 : list.length];
 
-        for (int i = 0; i < list.length; ++i) {
+        for (int i = 0; i < result.length; ++i) {
             result[i] = new TarArchiveEntry(new File(file, list[i]));
         }