add possibility to use different hash algorithms in RSAES-OAEP #612
No reviewers
Labels
No labels
bug
duplicate
enhancement
feature
help wanted
improve API
invalid
needs documentation
not-planned
rtfm
stale
support
wishlist
No milestone
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
gh-libtom/libtomcrypt!612
Loading…
Reference in a new issue
No description provided.
Delete branch "rsaaes_oaep_hashes"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
The hash algorithms used in the MGF and to create the hash of the Label must not forcibly be the same. This change allows to use different algorithms.
Unfortunately this breaks API if you use
rsa_decrypt_key_ex(). Thersa_decrypt_key()macro is still the same.Checklist
Hi
I made the following changed to perl's CryptX module which should be fine but I get:
realloc(): invalid next size
Aborted (core dumped)
Ping @karel-m
ah, I made a mistake, can you please try again?
Yes, that was able to decrypt my test
Just need the encrypt now and I can test that too :-)
Can you give it a try?
These changes seem to work fine for me. I tested with the following combinations encrypting with a modified Perl Crypt::PK::RSA and decrypting the keys with xmlsec1 (using openssl library)
The data method is just the key used to encrypt the data in my XML files
@sjaeckel @karel-m anything else you need from me?
no, nothing required from your side.
I've to think about whether we should break the API or add a new API function with this extended functionality... and if we add a new one, how we call it ...
@sjaeckel any update/progress?
not really ... at least I've rebased on current develop.
@karel-m any comments or suggestions how we could handle the API break a bit better?
Thanks for keeping it fresh :-).
I was going to suggest a separate API call _ex to take a page out of
openSSL's book and you probably don't want to take a page out of openSSL's
book...
But, it's already _ex... :-)
Tim
Timothy Legge
@.***
@.***
On Mon, Aug 7, 2023 at 10:49 AM Steffen Jaeckel @.***>
wrote:
@karel-m could you please have a look at this PR as well?
1/ I do not like the variable name
lparam_hash_as it is hard to read/understand; I would preferlparam_hash_finalor something like that.2/ We are breaking compatibility of
rsa_encrypt_key_ex+rsa_decrypt_key_exbut I can live with that.Otherwise looks good.
@sjaeckel could you please rebase
rsaaes_oaep_hasheson top of the currentdevelop? (so that I can test it with my CryptX module)and
pkcs_1_oaep_encode+pkcs_1_oaep_decodeLooks good, I am for merging.