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