X64 AESNI assembler routines corrupt XMM6-XMM15 which the Win64 calling convention expects to be preserved about mormot HOT 8 OPEN

Please switch to mORMot 2, which already includes the fix.

from mormot.

Thanks, Arnaud. I figured that was likely the case.

We're only using a selected few pieces of mORMot v1 so I'll look at what is involved in switching those to mORMot v2. I've coded up the required fixes in case anyone else wants them.

from mormot.

You are welcome to make a pull request here, of course!

from mormot.

Thanks, Arnaud. I figured that was likely the case.

We're only using a selected few pieces of mORMot v1 so I'll look at what is involved in switching those to mORMot v2. I've coded up the required fixes in case anyone else wants them.

@zunzster May I ask what the fix is? Is it possible to make a pull request or just show us the fixed code? We also use mORMot V1 for few selected pieces, it will be a great help for us if the legacy V1 has fixes for bugs.

from mormot.

The fix exists in mormot 2.

from mormot.

@synopse Is it possible to port it back to v1? synopse/mORMot2@4708b03

from mormot.

I've forked Mormot and pushed up my SynCrypto.pas changes for inspection. I've adapted my changes to be even closer to the Mormot2 changes but I'm 16-byte aligning the stack so I can use movaps for a slight speed boost.

I still have the original changes I made before seeing the v2 changes - that's the {$IFNDEF LINUX} section where I was following the Intel SHA1 convention from the example further down that file.

zunzster@8d6eb99

from mormot.

Thanks a lot for sharing!
You can make a Pull Request!

I am not sure there is any benefit with using the MORMOT2 version of the code, for these procedures.

from mormot.