[jira] [Resolved] (VFS-624) LocalFileRandomAccessContent returning wrong value when reading 0xFF

["Gary D. Gregory (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/VFS-624?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gary D. Gregory resolved VFS-624.
---------------------------------
    Fix Version/s: 2.7.0
       Resolution: Fixed

> LocalFileRandomAccessContent returning wrong value when reading 0xFF
> --------------------------------------------------------------------
>
>                 Key: VFS-624
>                 URL: https://issues.apache.org/jira/browse/VFS-624
>             Project: Commons VFS
>          Issue Type: Bug
>    Affects Versions: 2.0, 2.1
>            Reporter: David Goitia Amigo
>            Priority: Critical
>              Labels: easyfix
>             Fix For: 2.7.0
>
>   Original Estimate: 0h
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> The LocalFileRandomAccessContent InputStream used for reading a byte as an int is calling the RandomAccessFile method readByte(byte), so it returns -1 in case of EOFException, but also in case of reading 0xFF in a file.
> This also violates the expected behaviour of the read() method of the InputStream as can return negative values as result of the casting, while should only return an int value beetween 0 and 255, or -1 in case of eof.
> This problem is also inherited by the default read(byte[]) and read(byte[], int off, int length).
> The solution is to change the implementation to:
>  raf.read();
> Pull request: https://github.com/apache/commons-vfs/pull/17



--
This message was sent by Atlassian Jira
(v8.3.4#803005)