
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

  <head>


    <meta http-equiv="content-type" content="text/html; charset=UTF-8">

  </head>

  <body bgcolor="#ffffff" text="#000000">

    <br>

    Can someone please help reviewing this following fix?<br>

    <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8049312">https://bugs.openjdk.java.net/browse/JDK-8049312</a><br>

    Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~valeriep/8049312/webrev.00/">http://cr.openjdk.java.net/~valeriep/8049312/webrev.00/</a><br>

    <br>

    The must-fix change is in <code></code>

    src/share/classes/com/sun/crypto/provider/CipherCore.java which is

    to correct the data size calculation based on "unitBytes". For

    example, for CFB24, our current impl assumes the given data will be

    multiples of 3 bytes. When the given data isn't multiples of 3, it

    will continue but then the result is incorrect.<br>

    <br>

    To make the code more robust, I think we should explicitly check and

    error out when the given data doesn't have the correct size. Thus, I

    have added the input-length check to the various mode

    implementations. Along the way, I also fixed javadoc typos, removed

    redundancies, etc.<br>

    <br>

    Thanks,<br>

    Valerie<br>

    <br>

  </body>

</html>