[jira] [Commented] (FLINK-3269) Return value from DataInputStream#read() should be checked in PythonPlanReceiver

["Chesnay Schepler (JIRA)" <ji...@apache.org>]

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

Chesnay Schepler commented on FLINK-3269:
-----------------------------------------

looks like i messed up while rebasing, will fix it tomorrow.

> Return value from DataInputStream#read() should be checked in PythonPlanReceiver
> --------------------------------------------------------------------------------
>
>                 Key: FLINK-3269
>                 URL: https://issues.apache.org/jira/browse/FLINK-3269
>             Project: Flink
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Chesnay Schepler
>            Priority: Minor
>
> There're several places where the return value from DataInputStream#read() isn't checked.
> Here is an example from BytesDeserializer :
> {code}
>       byte[] buffer = new byte[size];
>       input.read(buffer);
>       return buffer;
> {code}
> Buffer of size bytes is allocated.
> However, the read operation may not fill size bytes.
> The return value should be checked.



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