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));