
<html>

  <head>


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

  </head>

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

    <p>Anyone have spare cycles for reviewing this? <br>

    </p>

    <p>The current PKCS11 JNI code for handling native mechanism and its

      parameters is a bit too all over the place. To make the tracing

      and verification easier, I consolidated the memory deallocation to

      the freeCKMechanismPtr(...) method in p11_util.c (some was in

      p11_keymgmt.c). <br>

    </p>

    <p>Also, fixed the j<u>XXX</u>ToCK<u>XXXX</u> methods in

      p11_convert.c to allocate the memory inside each method and return

      NULL upon error and make sure allocated memories are free'd upon

      any failure.<br>

    </p>

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

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

    <p>Mach5 run is clean.</p>

    Thanks,<br>

    Valerie<br>

  </body>

</html>