tree 81012f89690f716f0e73c61b4a4baa49c1b82292
parent cac40fc37d673e6c60d05b6715ed7655608be50d
author Thomas Wolf <twolf@apache.org> 1779044340 +0200
committer Matthias Sohn <matthias.sohn@sap.com> 1779230174 +0200

TemporaryBuffer.LocalFile: use Files.newInputStream()

On Unix, FileInputStream.skip() uses lseek under the hood and allows
skipping beyond EOF, which makes WorkingTreeIterator.computeLength() go
into an endless loop.

Files.newInputStream() returns a sun.nio.ch.ChannelInputStream, which
for files does not skip beyond EOF, avoiding this endless loop.

At the point WorkingTreeIterator.computeLength() is called, the input
stream is either a CR/LF-processing stream, which extend InputStream,
which actually uses reads to implement skipping and thus doesn't skip
beyond EOF, or a BlockInputStream, which also does not skip beyond the
content length, or a stream obtained from TemporaryBuffer.LocalFile.

Bug: jgit-261
Change-Id: I2e1bfe04c308220a1c2d819e6414a6ff7b2f9c34