Skip to contentSkip to conversation

Crypto library fixes for v6.16-rc3

GIT PULL Merged Eric Biggers 3 participants 6 messages Jun 18, 2025 – Jul 7, 2025 Crypto API

lore
Share & tools
lore
#0 lore raw

The following changes since commit 19272b37aa4f83ca52bdf9c16d5d81bdd1354494:

Linux 6.16-rc1 (2025-06-08 13:44:43 -0700)

are available in the Git repository at:

https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git tags/libcrypto-for-linus

for you to fetch changes up to 9d4204a8106fe7dc80e3f2e440c8f2ba1ba47319:

lib/crypto/poly1305: Fix arm64's poly1305_blocks_arch() (2025-06-16 12:51:34 -0700)

----------------------------------------------------------------

- Fix a regression in the arm64 Poly1305 code
- Fix a couple compiler warnings

----------------------------------------------------------------
Eric Biggers (1):

      lib/crypto/poly1305: Fix arm64's poly1305_blocks_arch()

Kees Cook (1):
lib/crypto: Annotate crypto strings with nonstring

Nathan Chancellor (1):
lib/crypto/curve25519-hacl64: Disable KASAN with clang-17 and older

 arch/arm64/lib/crypto/poly1305-glue.c |  4 +--
 lib/crypto/Makefile                   |  4 +++
 lib/crypto/aescfb.c                   |  8 +++---
 lib/crypto/aesgcm.c                   | 46 +++++++++++++++++------------------
 4 files changed, 33 insertions(+), 29 deletions(-)
merge bot Merged
#1 lore raw
The pull request you sent on Wed, 18 Jun 2025 12:49:58 -0700:

The pull request you sent on Wed, 18 Jun 2025 12:49:58 -0700:

Quoted 1 line

https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git tags/libcrypto-for-linus

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/61f4769affffc398499250ccacf0b86d5b654399

Thank you!

Signature · 2 lines
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html
#2 lore raw
In reply to Eric Biggers
Quoted 8 lines · Eric Biggers

The following changes since commit 19272b37aa4f83ca52bdf9c16d5d81bdd1354494:

Linux 6.16-rc1 (2025-06-08 13:44:43 -0700)

are available in the Git repository at:

https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git tags/libcrypto-for-linus

Quoted 7 lines

for you to fetch changes up to 9d4204a8106fe7dc80e3f2e440c8f2ba1ba47319:

lib/crypto/poly1305: Fix arm64's poly1305_blocks_arch() (2025-06-16 12:51:34 -0700)

----------------------------------------------------------------

- Fix a regression in the arm64 Poly1305 code

Some more tests too, perhaps? :)

This was a bit of a scary one, since poly1305 was returning an inconsistent result, not total garbage. Meaning most of the tests passed, but fortunately the migrate tests read data written by userspace with a different library.

Quoted 17 lines

- Fix a couple compiler warnings

----------------------------------------------------------------
Eric Biggers (1):

      lib/crypto/poly1305: Fix arm64's poly1305_blocks_arch()

Kees Cook (1):
lib/crypto: Annotate crypto strings with nonstring

Nathan Chancellor (1):
lib/crypto/curve25519-hacl64: Disable KASAN with clang-17 and older

 arch/arm64/lib/crypto/poly1305-glue.c |  4 +--
 lib/crypto/Makefile                   |  4 +++
 lib/crypto/aescfb.c                   |  8 +++---
 lib/crypto/aesgcm.c                   | 46 +++++++++++++++++------------------
 4 files changed, 33 insertions(+), 29 deletions(-)
#3 lore raw
Quoted 15 lines · Kent Overstreet
Quoted 14 lines · Eric Biggers

The following changes since commit 19272b37aa4f83ca52bdf9c16d5d81bdd1354494:

Linux 6.16-rc1 (2025-06-08 13:44:43 -0700)

are available in the Git repository at:

https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git tags/libcrypto-for-linus

for you to fetch changes up to 9d4204a8106fe7dc80e3f2e440c8f2ba1ba47319:

lib/crypto/poly1305: Fix arm64's poly1305_blocks_arch() (2025-06-16 12:51:34 -0700)

----------------------------------------------------------------

Quoted 8 lines
Quoted 1 line

- Fix a regression in the arm64 Poly1305 code

Some more tests too, perhaps? :)

This was a bit of a scary one, since poly1305 was returning an inconsistent result, not total garbage. Meaning most of the tests passed, but fortunately the migrate tests read data written by userspace with a different library.

Yep, I have a KUnit test for Poly1305 planned. Actually, I already wrote a preliminary one and used it to test this patch. I just haven't sent it out for review quite yet, since so far it's just a one-off test that isn't too complete, and I'm not satisfied with it quite yet. I'd like to reuse the hash-test-template.h I'm adding for SHA-2
(https://lore.kernel.org/linux-crypto/[email protected]/)
which would result in a more complete test. I'd also like to include tests for some of the overflow cases that are specific to Poly1305.

So we're kind of still in an early stage where we're defining what the KUnit testing for lib/crypto/ is going to look like. I am working on it, though!

- Eric

#4 lore raw
Quoted 34 lines · Eric Biggers
Quoted 15 lines · Kent Overstreet
Quoted 14 lines · Eric Biggers

The following changes since commit 19272b37aa4f83ca52bdf9c16d5d81bdd1354494:

Linux 6.16-rc1 (2025-06-08 13:44:43 -0700)

are available in the Git repository at:

https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git tags/libcrypto-for-linus

for you to fetch changes up to 9d4204a8106fe7dc80e3f2e440c8f2ba1ba47319:

lib/crypto/poly1305: Fix arm64's poly1305_blocks_arch() (2025-06-16 12:51:34 -0700)

----------------------------------------------------------------

Quoted 8 lines
Quoted 1 line

- Fix a regression in the arm64 Poly1305 code

Some more tests too, perhaps? :)

This was a bit of a scary one, since poly1305 was returning an inconsistent result, not total garbage. Meaning most of the tests passed, but fortunately the migrate tests read data written by userspace with a different library.

Yep, I have a KUnit test for Poly1305 planned. Actually, I already wrote a preliminary one and used it to test this patch. I just haven't sent it out for review quite yet, since so far it's just a one-off test that isn't too complete, and I'm not satisfied with it quite yet. I'd like to reuse the hash-test-template.h I'm adding for SHA-2
(https://lore.kernel.org/linux-crypto/[email protected]/)
which would result in a more complete test. I'd also like to include tests for some of the overflow cases that are specific to Poly1305.

Quoted 2 lines

So we're kind of still in an early stage where we're defining what the KUnit testing for lib/crypto/ is going to look like. I am working on it, though!

I do a little dance any time someeone says something like that about testing. Bravo :)

(And now, back to sifting through the rest of the rc1 breakage.)

#5 lore raw
Quoted 35 lines · Kent Overstreet
Quoted 34 lines · Eric Biggers
Quoted 15 lines · Kent Overstreet
Quoted 14 lines · Eric Biggers

The following changes since commit 19272b37aa4f83ca52bdf9c16d5d81bdd1354494:

Linux 6.16-rc1 (2025-06-08 13:44:43 -0700)

are available in the Git repository at:

https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git tags/libcrypto-for-linus

for you to fetch changes up to 9d4204a8106fe7dc80e3f2e440c8f2ba1ba47319:

lib/crypto/poly1305: Fix arm64's poly1305_blocks_arch() (2025-06-16 12:51:34 -0700)

----------------------------------------------------------------

Quoted 8 lines
Quoted 1 line

- Fix a regression in the arm64 Poly1305 code

Some more tests too, perhaps? :)

This was a bit of a scary one, since poly1305 was returning an inconsistent result, not total garbage. Meaning most of the tests passed, but fortunately the migrate tests read data written by userspace with a different library.

Yep, I have a KUnit test for Poly1305 planned. Actually, I already wrote a preliminary one and used it to test this patch. I just haven't sent it out for review quite yet, since so far it's just a one-off test that isn't too complete, and I'm not satisfied with it quite yet. I'd like to reuse the hash-test-template.h I'm adding for SHA-2
(https://lore.kernel.org/linux-crypto/[email protected]/)
which would result in a more complete test. I'd also like to include tests for some of the overflow cases that are specific to Poly1305.

Quoted 7 lines
Quoted 2 lines

So we're kind of still in an early stage where we're defining what the KUnit testing for lib/crypto/ is going to look like. I am working on it, though!

I do a little dance any time someeone says something like that about testing. Bravo :)

(And now, back to sifting through the rest of the rc1 breakage.)

FYI, the following patch adds a KUnit test suite for Poly1305:
https://lore.kernel.org/linux-crypto/[email protected]/

- Eric