Skip to content

Test data for failure caused by empty digest algorithm - #302

Open
bjwtaylor wants to merge 4 commits into
Mbed-TLS:mainfrom
bjwtaylor:empty-digest-failure
Open

Test data for failure caused by empty digest algorithm#302
bjwtaylor wants to merge 4 commits into
Mbed-TLS:mainfrom
bjwtaylor:empty-digest-failure

Conversation

@bjwtaylor

@bjwtaylor bjwtaylor commented May 1, 2026

Copy link
Copy Markdown

Description

Test data for failure caused by empty digest algorithm. contributes Mbed-TLS/mbedtls#10480.

Parameterised build https://ci.trustedfirmware.org/view/Mbed-TLS/job/mbed-tls-restricted-pr-test-parametrized/176/

This PR is part of a two part set which must be merged in the following order:

  1. Test data for failure caused by empty digest algorithm #302
  2. Resolve failure caused by empty digest algorithm mbedtls#10746

PR checklist

  • TF-PSA-Crypto development PR not required because: No changes
  • TF-PSA-Crypto 1.1 PR not required because: No changes
  • mbedtls development PR provided Resolve failure caused by empty digest algorithm mbedtls#10746
  • mbedtls 4.1 PR provided # | not required because: TBC
  • mbedtls 3.6 PR provided # | not required because: TBC

@bjwtaylor bjwtaylor changed the title Empty digest failure Test data for failure caused by empty digest algorithm May 1, 2026
@bjwtaylor
bjwtaylor force-pushed the empty-digest-failure branch 2 times, most recently from b1b62f1 to b3bc2ec Compare May 7, 2026 07:39
@bjwtaylor
bjwtaylor marked this pull request as ready for review May 7, 2026 13:50
@bjwtaylor bjwtaylor added needs-review Every commit must be reviewed by at least two team members. needs-reviewer This PR needs someone to pick it up for review priority-low Low priority - this may not receive review soon size-xs Estimated task size: extra small (a few hours at most) size-s Estimated task size: small (~2d) and removed size-xs Estimated task size: extra small (a few hours at most) labels May 7, 2026
@bjwtaylor
bjwtaylor requested a review from valeriosetti June 4, 2026 09:07
Ben Taylor added 3 commits July 1, 2026 09:50
Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
@bjwtaylor
bjwtaylor force-pushed the empty-digest-failure branch from b3bc2ec to 97ef810 Compare July 1, 2026 08:50

@valeriosetti valeriosetti left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usually each data file in this folder has an associated make rule in the Makefile. This helps in case in the future the file has to be re-generated for some reason.
I don't know how you get these files, but do you think it should be possible to do this?

@valeriosetti valeriosetti left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given the description provided in Mbed-TLS/mbedtls#10480 should we want to have something similar to pkcs7_empty_digest.der but with some CRT in it?
Probably this doesn't change anything compared to pkcs7_empty_digest.der, but it would be something close to some real use case. Wdyt?

Sorry perhaps that's already covered from pkcs7_data_cert_signed_sha256_empty_digest.der 🤔

Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
@bjwtaylor

Copy link
Copy Markdown
Author

@valeriosetti, I've added a section to the makefile.

@bjwtaylor
bjwtaylor requested a review from valeriosetti July 6, 2026 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review Every commit must be reviewed by at least two team members. needs-reviewer This PR needs someone to pick it up for review priority-low Low priority - this may not receive review soon size-s Estimated task size: small (~2d)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants