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

Kenneth Knowles commented on BEAM-3140:
---------------------------------------

This issue has been migrated to https://github.com/apache/beam/issues/18512

> Unchecked return value from bis.read() in MutationGroupEncoder#decodeMutation
> -----------------------------------------------------------------------------
>
>                 Key: BEAM-3140
>                 URL: https://issues.apache.org/jira/browse/BEAM-3140
>             Project: Beam
>          Issue Type: Bug
>          Components: io-java-gcp
>            Reporter: Ted Yu
>            Priority: P3
>
> {code}
>   private Mutation decodeMutation(ByteArrayInputStream bis) throws 
> IOException {
>     Mutation.Op op = Mutation.Op.values()[bis.read()];
>     if (op == Mutation.Op.DELETE) {
>       return decodeDelete(bis);
> {code}
> The return value from bis.read() should be checked before it is used to 
> dereference values().



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to