
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

</head>

<body>

<div dir="ltr">

<div></div>

<div data-ogsc="" style="">

<div>

<div>Hello Valerie.</div>

<div><br>

</div>

<div style="direction: ltr;">In MacKAT 121 you would get a NPE if the catch prints the skip message, probably needs an additional return; guard?</div>

<div><br>

</div>

<div style="direction: ltr;">The BAOS default length change in parse() was not immediately clear to me? (Maybe next s. Base64?)</div>

<div><br>

</div>

<div style="direction: ltr;">BTW It is good to see that you also add truncated SHA512 variants. It's not mentioned in commit message or RFE.</div>

<div><br>

</div>

<div style="direction: ltr;">hTH</div>

<div style="direction: ltr;">Bernd</div>

<div><br>

</div>

</div>

<div><br>

</div>

<div class="ms-outlook-ios-signature">

<div><br>

</div>

<div style="direction: ltr;">-- </div>

<div style="direction: ltr;">http://bernd.eckenfels.net</div>

</div>

</div>

</div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>Von:</b> security-dev <security-dev-bounces@openjdk.java.net> im Auftrag von Valerie Peng <valerie.peng@oracle.com><br>

<b>Gesendet:</b> Wednesday, March 18, 2020 11:57:37 PM<br>

<b>An:</b> OpenJDK Dev list <security-dev@openjdk.java.net><br>

<b>Betreff:</b> [15] RFR 8172680: Support SHA-3 based Hmac algorithms</font>

<div> </div>

</div>

<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">

<div class="PlainText"><br>

Anyone has time to help review this straight forward RFE? It's to add <br>

SHA-3 support to Hmac.<br>

<br>

RFE: <a href="https://bugs.openjdk.java.net/browse/JDK-8172680">https://bugs.openjdk.java.net/browse/JDK-8172680</a><br>

<br>

Webrev: <a href="http://cr.openjdk.java.net/~valeriep/8172680/webrev.00/">http://cr.openjdk.java.net/~valeriep/8172680/webrev.00/</a><br>

<br>

Mach5 run is clean.<br>

<br>

Thanks,<br>

Valerie<br>

</div>

</span></font></div>

</body>

</html>