<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>