[jira] [Comment Edited] (EXEC-92) StreamPumper - Check For EOF

["BELUGA BEHR (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/EXEC-92?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14364409#comment-14364409 ] 

BELUGA BEHR edited comment on EXEC-92 at 3/17/15 1:55 AM:
----------------------------------------------------------

Like I said, I'm just mimicking IOUtils, so if a check for zero-length is required, then that should be addressed with that project too.

I'm not sure there's anything to "protect" against.  Any implementation of an OutputStream is going to do some sort of "for (0..length)" and if length is zero, nothing happens.

BufferedInputStream is an example of this:

http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/6-b14/java/io/InputStream.java#InputStream.read%28byte%5B%5D%2Cint%2Cint%29


was (Author: belugabehr):
Like I said, I'm just mimicking IOUtils, so if a check for zero-length is required, then that should be addressed with that project too.

I'm not sure there's anything to "protect" against.  Any implementation of an OutputStream is going to do some sort of "for (0..length)" and if length is zero, nothing happens.

BufferedInputStream is an example of this:

http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/6-b14/java/io/FilterInputStream.java#FilterInputStream.read%28byte%5B%5D%2Cint%2Cint%29

> StreamPumper - Check For EOF
> ----------------------------
>
>                 Key: EXEC-92
>                 URL: https://issues.apache.org/jira/browse/EXEC-92
>             Project: Commons Exec
>          Issue Type: Bug
>    Affects Versions: 1.3
>            Reporter: BELUGA BEHR
>             Fix For: 1.4
>
>         Attachments: StreamPumper.java.patch
>
>
> When the StreamPumper is copying data from the InputStream to the OutputStream, it would be safer to check for EOF instead of a read size greater than zero to protect against InputStream implementations that don't respect the contract as defined in Java core JavaDoc.
> Patch Included.  I took code from the copy method of the IOUtils class.
> {code:title=StreamPumper.java|borderStyle=solid}
> while ((length = is.read(buf)) > 0)
> {
>     os.write(buf, 0, length);
> }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)