Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dynamic salt length for PKCS#1 v2.1 signing and verification #8

Closed
pjbakker opened this issue Nov 15, 2012 · 1 comment
Closed

Dynamic salt length for PKCS#1 v2.1 signing and verification #8

pjbakker opened this issue Nov 15, 2012 · 1 comment
Labels
enhancement

Comments

@pjbakker
Copy link
Contributor

Thank you for adding PKCS #1 v2.1 to the library. I see that the salt length is effectively tied to the hash length with slen=hlen. Can you make the salt length a parameter when calling the sign and verify functions?

Verify automatically uses the salt that is in the signature itself.

For sign, a change is needed. I don't like adding a salt-len variable that is only used if the RSASSA-PSS is enable in the code.. I'm still looking for a nice and not so ugly solution to propagate the salt-len into that part of the code.

Suggestions are welcome.

Postponed to a next release.
@pjbakker
Copy link
Contributor Author

pjbakker commented Jun 3, 2013

Added support for custom labels in PolarSSL 1.2.6

@pjbakker pjbakker closed this as completed Jun 3, 2013
@edsiper edsiper mentioned this issue Apr 23, 2014
Closed
@fargly fargly mentioned this issue Sep 16, 2015
Closed
@qipeng090 qipeng090 mentioned this issue Dec 21, 2016
Closed
gilles-peskine-arm added a commit to gilles-peskine-arm/mbedtls that referenced this issue Sep 5, 2017
@gilles-peskine-arm
Merge pull request Mbed-TLS#8 from ARMmbed/msp430-build
e059bc3 
Msp430 build and test scripts
@chmorgan chmorgan mentioned this issue Aug 10, 2018
Closed
Patater added a commit to Patater/mbedtls that referenced this issue Jan 15, 2019
@Patater
Merge pull request Mbed-TLS#8 from NirSonnenschein/fix_check-names.sh
005401e 
modify check-names.sh and list-macros.sh to work with PSA constants
@mcli mcli mentioned this issue Feb 2, 2019
Closed
@RonEld RonEld mentioned this issue Feb 7, 2019
Open
4 tasks
hanno-becker pushed a commit to hanno-becker/mbedtls that referenced this issue Mar 25, 2020
Merge pull request Mbed-TLS#8 from hanno-arm/tls13-prototype_doc
bc6e35f 
Enhance documentation of cTLS and TLS 1.3 compatibility mode configuration options
loafer-mka pushed a commit to loafer-mka/mbedtls that referenced this issue Jun 22, 2021
reformat and check stack buffer sizes pkwrite.c and pk.h (pkcs Mbed-T…
d397fa0 
…LS#8)
@daverodgman daverodgman mentioned this issue Nov 1, 2022
Merged
@yuhaoth yuhaoth mentioned this issue Apr 21, 2023
Closed
@SangsubLim SangsubLim mentioned this issue Jun 20, 2023
Closed
bensze01 pushed a commit to bensze01/mbedtls that referenced this issue Jul 3, 2024
@tom-cosgrove-arm
Merge pull request Mbed-TLS#8 from gilles-peskine-arm/psasim-publish
ceab979 
Publish PSA framework simulator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@pjbakker