Skip to content

Conversation

@jorisdral
Copy link
Collaborator

@jorisdral jorisdral commented Nov 10, 2025

This resolves a part of #33. See the commit message for more information.

@jorisdral jorisdral self-assigned this Nov 10, 2025
@jorisdral jorisdral force-pushed the jdral/fix-encrypt-test branch from 0632d98 to 2271d62 Compare November 10, 2025 14:19
We now use upper bounds on output buffer sizes as provided by the Botan C FFI to
allocate buffers, rather than querying the encryption/decryption fuctions with a
null pointer (see randombit/botan#5154), which are
arguably suffering from a bug. Using the upper bounds was already advised by the
Botan C FFI anyway.
@jorisdral jorisdral force-pushed the jdral/fix-encrypt-test branch from 2271d62 to f0a4964 Compare November 10, 2025 14:30
@jorisdral jorisdral changed the title Fix a bug in Test.Botan.Low.PubKey.Encrypt Fix a bug in public-key encrypt and decrypt functions Nov 10, 2025
@jorisdral jorisdral added botan-low Related to the botan-low package bug Something isn't working labels Nov 10, 2025
@jorisdral jorisdral marked this pull request as ready for review November 10, 2025 14:33
@jorisdral jorisdral added this pull request to the merge queue Nov 10, 2025
Merged via the queue into main with commit 22355cd Nov 10, 2025
34 checks passed
@jorisdral jorisdral deleted the jdral/fix-encrypt-test branch November 10, 2025 14:52
@jorisdral jorisdral mentioned this pull request Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

botan-low Related to the botan-low package bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants