<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body>
<div dir="ltr">
<div></div>
<div>
<div dir="ltr">BTW, after then ensures it looks like a good candidate for a system-assert for not-null for all of those fields, right?</div>
<div dir="ltr"><br>
</div>
<div dir="ltr">Gruss</div>
<div dir="ltr">Bernd</div>
<div id="ms-outlook-mobile-signature">
<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> core-libs-dev <core-libs-dev-retn@openjdk.org> im Auftrag von Andrey Turbanov <aturbanov@openjdk.org><br>
<b>Gesendet:</b> Thursday, September 1, 2022 8:24:58 AM<br>
<b>An:</b> core-libs-dev@openjdk.org <core-libs-dev@openjdk.org>; security-dev@openjdk.org <security-dev@openjdk.org><br>
<b>Betreff:</b> RFR: 8293197: Avoid double racy reads from non-volatile fields in SharedSecrets</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">After integration of [JDK-8259021](https://bugs.openjdk.org/browse/JDK-8259021) a few more fields were added to `SharedSecrets` class. It make sense to update their reading code to use the same idiom.<br>
<br>
-------------<br>
<br>
Commit messages:<br>
- [PATCH] SharedSecrets should avoid double racy reads from non-volatile fields<br>
- [PATCH] SharedSecrets should avoid double racy reads from non-volatile fields<br>
<br>
Changes: <a href="https://git.openjdk.org/jdk/pull/10076/files">https://git.openjdk.org/jdk/pull/10076/files</a><br>
Webrev: <a href="https://webrevs.openjdk.org/?repo=jdk&pr=10076&range=00">https://webrevs.openjdk.org/?repo=jdk&pr=10076&range=00</a><br>
Issue: <a href="https://bugs.openjdk.org/browse/JDK-8293197">https://bugs.openjdk.org/browse/JDK-8293197</a><br>
Stats: 16 lines in 1 file changed: 8 ins; 0 del; 8 mod<br>
Patch: <a href="https://git.openjdk.org/jdk/pull/10076.diff">https://git.openjdk.org/jdk/pull/10076.diff</a><br>
Fetch: git fetch <a href="https://git.openjdk.org/jdk">https://git.openjdk.org/jdk</a> pull/10076/head:pull/10076<br>
<br>
PR: <a href="https://git.openjdk.org/jdk/pull/10076">https://git.openjdk.org/jdk/pull/10076</a><br>
</div>
</span></font></div>
</body>
</html>