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.

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

Add iop key agrmnt tests #9735

Open
wants to merge 5 commits into
base: development
Choose a base branch
from

Conversation

paul-elliott-arm
Copy link
Member

Description

Negative tests for Interruptible Key Agreement (probably not in final form yet, but testing via CI)

PR checklist

Please remove the segment/s on either side of the | symbol as appropriate, and add any relevant link/s to the end of the line.
If the provided content is part of the present PR remove the # symbol.

  • changelog provided | not required because: Part of interruptible ECC work
  • development PR provided # | not required because: This is the development PR
  • framework PR provided Mbed-TLS/mbedtls-framework# | not required (no framework interaction)
  • 3.6 PR provided # | not required because: New feature for 4.0
  • 2.28 PR provided # | not required because: New feature for 4.0
  • tests provided | not required because:

@paul-elliott-arm paul-elliott-arm added needs-work DO-NOT-MERGE size-s Estimated task size: small (~2d) labels Oct 28, 2024
@paul-elliott-arm paul-elliott-arm self-assigned this Oct 28, 2024
@paul-elliott-arm paul-elliott-arm added needs-ci Needs to pass CI tests component-test Test framework and CI scripts labels Oct 28, 2024
Incorrect usage of psa_key_id_t breaks ability to regenerate psa_sim
headers properly, should have been mbedtls_svc_key_id_t in the first
place, so replace with that.

Signed-off-by: Paul Elliott <[email protected]>
Non generated code was unfortnuately added to psa_sim_crypto_server.c,
fix that by adding it to the generation scripts.

Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: Paul Elliott <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component-test Test framework and CI scripts DO-NOT-MERGE needs-ci Needs to pass CI tests needs-work size-s Estimated task size: small (~2d)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant