[tika] 07/13: TIKA-3110 -- revert throwing IOException if a parser tries to skip past the end of an inputstream...make TikaInputStream live up to its Java contract at the risk of infinite loops in dependencies that weren't written with security in mind.

[ta...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

tallison pushed a commit to branch branch_1x
in repository https://gitbox.apache.org/repos/asf/tika.git

commit aad237c717672b0369ec1696e74075aa4fb3e383
Author: tallison <ta...@apache.org>
AuthorDate: Fri Jun 12 15:33:44 2020 -0400

    TIKA-3110 -- revert throwing IOException if a parser tries to skip past the end of an inputstream...make TikaInputStream live up to its Java contract at the risk of infinite loops in dependencies that weren't written with security in mind.
---
 tika-core/src/main/java/org/apache/tika/io/TikaInputStream.java | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/tika-core/src/main/java/org/apache/tika/io/TikaInputStream.java b/tika-core/src/main/java/org/apache/tika/io/TikaInputStream.java
index a0275a8..fcd6cd6 100644
--- a/tika-core/src/main/java/org/apache/tika/io/TikaInputStream.java
+++ b/tika-core/src/main/java/org/apache/tika/io/TikaInputStream.java
@@ -713,9 +713,6 @@ public class TikaInputStream extends TaggedInputStream {
             skipBuffer = new byte[4096];
         }
         long n = IOUtils.skip(super.in, ln, skipBuffer);
-        if (n != ln) {
-            throw new IOException("tried to skip "+ln + " but actually skipped: "+n);
-        }
         position += n;
         return n;
     }