Code review request: 7106277 Brokenness in the seqNumberOverflow of MAC

[Bradford Wetmore]

Looks good.  As you may remember, my personal preference is to use lots 
of parens to clearly show what you intended, but up to you as it's 
pretty clear without

         return ((block != null) && (mac != null) &&
             (block[0] == (byte)0xFF) && (block[1] == (byte)0xFF) &&
             ...

I'd ask for a test case, but it might take a while to run.  ;)

Brad





On 10/28/2011 8:00 PM, Xuelei Fan wrote:
> Hi,
>
> Would you please review my fix for 7106277 (Brokenness in the
> seqNumberOverflow of MAC)?
>
> webrev: http://cr.openjdk.java.net/~xuelei/7106277/webrev.00/
>
> Thanks,
> Xuelei