<div>Hi,<br></div><div><br></div><div>Thanks very much!<br></div><div><br></div><div>This version:<br></div><div><br></div><div>and</div><div><br></div><p style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-stretch: normal; line-height: normal;"><span class="colour" style="color:rgb(0, 0, 0)"><span class="font" style="font-family:Menlo"><span class="size" style="font-size:11px"><span style="font-variant-ligatures: no-common-ligatures">Date: Sat Dec 11 11:58:58 2021 CommitHash: 97b4903b4</span></span></span></span><br></p><div><br></div><div>and ran this test:<br></div><div><br></div><pre style="outline: none; box-sizing: content-box; font-family: monospace, monospace; font-size: 14px; margin-top: 0px; margin-bottom: 1rem; overflow: auto; white-space: normal; word-wrap: break-word; line-height: normal; caret-color: rgb(51, 51, 51); color: rgb(51, 51, 51); font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;"><div>Installer ss<br></div><div>project: 'Registers';<br></div><div>install: 'Registers-Core'.<br></div><div>Installer ss<br></div><div>project: 'Cryptography';<br></div><div>addPackage: 'CryptographyHashing';<br></div><div>addPackage: 'CryptographyHashingTests';<br></div><div>install.<br></div><div>(Smalltalk classNamed: #SHA256WithSHA2PluginTest) run: #testInputs<br></div></pre><div>and <br></div><div><br></div><div>Linux64x64 had no crashes, but a few expected failures.<br></div><div>Linux64ARMv8 had no crashes but some unexpected test failures<br></div><div>Linux32ArmV6 had no crashes but some unexpected test failures.<br></div><div><br></div><div>So thanks, this bug seems squashed.<br></div><div><br></div><div>cheers<br></div><div><br></div><div>bruce<br></div><div><br></div><div><br></div><div><br></div><div  class="ik_mail_quote"><div>On 2021-12-11T21:02:23.000+01:00, Eliot Miranda <eliot.miranda@gmail.com> wrote:</div><blockquote class="ws-ng-quote"><div class="ws-ng-mail-style--480342eJzz9HPyjwAABGYBgQ"><div dir="ltr"><div class="gmail_default-480342eJzz9HPyjwAABGYBgQ" style="font-size:small;">Hi Both,</div><div class="gmail_default-480342eJzz9HPyjwAABGYBgQ" style="font-size:small;"><br></div><div class="gmail_default-480342eJzz9HPyjwAABGYBgQ" style="font-size:small;">     see commit 97b4903b4b88f22c1bd11760f107852c63f9db40</div>Author: Eliot Miranda <<a class="defaultMailLink" href="mailto:eliot.miranda@gmail.com" target="_blank" rel="noopener noreferrer">eliot.miranda@gmail.com</a>><br>Date:   Sat Dec 11 11:58:58 2021 -0800<br><br>    src/plugins/SHA2Plugin/SHA2Plugin.c as per CryptographyPlugins-eem.24<br><br>    Fix crashes in primitiveSHA256ProcessBufferUpdatingHash when compiling with<br>    Clang on x86_64 due to SSE instructions which require 128-bit stack alignment.<div><br></div><div><div class="gmail_default-480342eJzz9HPyjwAABGYBgQ" style="font-size:small;">I need to know about any other such crashes pronto.</div><br></div></div><br><div class="gmail_quote-480342eJzz9HPyjwAABGYBgQ"><div dir="ltr" class="gmail_attr-480342eJzz9HPyjwAABGYBgQ">On Fri, Dec 10, 2021 at 4:43 AM Levente Uzonyi <<a class="defaultMailLink" href="mailto:leves@caesar.elte.hu" target="_blank" rel="noopener noreferrer">leves@caesar.elte.hu</a>> wrote:<br></div><blockquote class="gmail_quote-480342eJzz9HPyjwAABGYBgQ" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex;">Hi Marcel,<br>
<br>
The SHA2 plugin (primitiveSHA256ProcessBufferUpdatingHash) still<br>
crashes with that VM on 64-bit linux.<br>
The plugin code works with earlier versions, so it's either a VM <br>
change of the past 6-9 months, a code generator bug or a compiler bug IMO.<br>
<br>
To reproduce the crash, evaluate the following:<br>
<br>
Installer ss<br>
        project: 'Registers';<br>
        install: 'Registers-Core'.<br>
Installer ss<br>
        project: 'Cryptography';<br>
        addPackage: 'CryptographyHashing';<br>
        addPackage: 'CryptographyHashingTests';<br>
        install.<br>
(Smalltalk classNamed: #SHA256WithSHA2PluginTest) run: #testInputs<br>
<br>
Interestingly another test (SHA512WithSHA2PluginTest) using a very <br>
similar primitive but with DoubleWords works fine. So perhaps it's an <br>
alignment issue.<br>
<br>
Assert and debug VMs do not have that issue, so it's not that easy to <br>
debug it. What I found was that buffer's value at <br>
<a class="defaultMailLink" href="https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/99f1116c0f7a4ba9a0bf88cf7deb276325aa2d79/src/plugins/SHA2Plugin/SHA2Plugin.c#L277" target="_blank" rel="noopener noreferrer" data-ik="ik-secure">https://github.com/OpenSmalltalk/opensmalltalk-vm/blob/99f1116c0f7a4ba9a0bf88cf7deb276325aa2d79/src/plugins/SHA2Plugin/SHA2Plugin.c#L277</a><br>
was not the expected pointer, so the subsequent copying into it resulted <br>
in segmentation fault.<br>
<br>
<br>
Levente<br>
<br>
<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature-480342eJzz9HPyjwAABGYBgQ"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate;"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div>
<hr></div></blockquote></div>