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 2014/05/04 06:44:24 UTC

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

Author: bodewig
Date: Sun May  4 04:44:24 2014
New Revision: 1592309

URL: http://svn.apache.org/r1592309
Log:
COMPRESS-280 add explicit argument verification

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

Modified: commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveInputStream.java
URL: http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveInputStream.java?rev=1592309&r1=1592308&r2=1592309&view=diff
==============================================================================
--- commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveInputStream.java (original)
+++ commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveInputStream.java Sun May  4 04:44:24 2014
@@ -199,6 +199,9 @@ public class TarArchiveInputStream exten
      */
     @Override
     public long skip(final long n) throws IOException {
+        if (n <= 0) {
+            return 0;
+        }
 
         final long available = entrySize - entryOffset;
         final long skipped = is.skip(Math.min(n, available));