[BUG] LiteFileStream SetReadStreamPosition causes incorrect seeking

[CSDSP]

Version
Current master (commit 4f77be4).

Describe the bug
LiteFileStream.SetReadStreamPosition, part of how seeking is implemented, has a few bugs.
First of all,

if (newPosition < 0 && newPosition > Length)
{
    throw new ArgumentOutOfRangeException();
}

This is clearly wrong, probably meant to be an or instead of an and. The effect is you can seek to positions below 0 or greater than length, and you get exceptions other than ArgumentOutOfRangeException.

Secondly, and more importantly for my purposes, is the block below:

// calculate new chunk position
 _currentChunkIndex = (int)_streamPosition / MAX_CHUNK_SIZE;
_positionInChunk = (int)_streamPosition % MAX_CHUNK_SIZE;

This assumes that all chunks aside from the last one are exactly MAX_CHUNK_SIZE bytes. However, that is not the case.
If it is meant to be the case, then the bug is with writing, because various files in my database (uploaded with LiteFileStorage.Upload) have some randomly much smaller chunks.

[CSDSP]

On the write size, MemoryStream.Read isn't guaranteed to return the maximum available number of bytes. Currently, a chunk is written for every buffer read, so whenever it happens to read less then max chunk size bytes, it still writes a chunk. This could be fixed, though it would leave the read issue in place for all current databases.

[alexbereznikov]

@CSDSP @SpaceCheetah May I ask how this issue shows itself? We have some new db corruption issues after upgrading to v5.0.19 so just wanted to know whether your fix could help us :)

[SpaceCheetah]

@alexbereznikov It shows up when you try to seek in a file read stream. Just downloading directly to a file or copying to a memory stream works fine, but if you try to seek there's a chance depending on the file for it to seek to the wrong position. I noticed it when I was getting exceptions where it was trying to read more data from a chunk than actually existed.

Seeking to the beginning still works, seeking anywhere else in the file can fail if any chunks are smaller than max length, which for me is fairly common.

[bnuzhouwei]

The Seek get error, so the file can not be suport enableRangeProcessing in MVC file result. How to fix it.!

[bnuzhouwei]

It also cause that The LiteFileStream can't be use for read documents with stream by ITextSharp or other document libs.
Hope this fix as soon as possible