On Fri, 9 Oct 2020 03:20:40 GMT, Valerie Peng <valer...@openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Xuelei comments
>
> src/java.base/share/classes/com/sun/crypto/provider/GaloisCounterMode.java 
> line 635:
> 
>> 633:         dst.put(block, 0, tagLenBytes);
>> 634:
>> 635:         return (processed + tagLenBytes);
> 
> Is it supposed to return "all data processed + tag len"? Normally, we return 
> the output size produced by this
> particular call instead of all accumulated.

Yes, that should be len + tagLenBytes.  We obviously don't check that in our 
tests

-------------

PR: https://git.openjdk.java.net/jdk/pull/411

Reply via email to