Skip to content

[Backport 3.6] PSA its function uid0 errorcode fix - #10920

Open
yiwu0b11 wants to merge 2 commits into
Mbed-TLS:mbedtls-3.6from
yiwu0b11:psa-its-uid0-errorcode-fix-3.6
Open

[Backport 3.6] PSA its function uid0 errorcode fix#10920
yiwu0b11 wants to merge 2 commits into
Mbed-TLS:mbedtls-3.6from
yiwu0b11:psa-its-uid0-errorcode-fix-3.6

Conversation

@yiwu0b11

@yiwu0b11 yiwu0b11 commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes: Mbed-TLS/TF-PSA-Crypto#826
Backport of: Mbed-TLS/TF-PSA-Crypto#835

Fix psa_its_set(), psa_its_get(), psa_its_get_info(), psa_its_remove()to return PSA_ERROR_INVALID_ARGUMENT when called with UID 0, to match the PSA Storage API requirement.

Update the test and function description.

PR checklist

@yiwu0b11
yiwu0b11 changed the base branch from development to mbedtls-3.6 August 11, 2026 16:38
@yiwu0b11 yiwu0b11 changed the title [Backport 3.6] Psa its uid0 errorcode fix [Backport 3.6] PSA its function uid0 errorcode fix Aug 11, 2026
@yiwu0b11
yiwu0b11 marked this pull request as ready for review August 12, 2026 14:57
function changed: psa_its_set, _get, _get_info, _remove

Signed-off-by: Yi Wu <yi.wu2@arm.com>
@yiwu0b11
yiwu0b11 force-pushed the psa-its-uid0-errorcode-fix-3.6 branch from 1d1fbb3 to 9a23f70 Compare August 12, 2026 15:00
@gilles-peskine-arm gilles-peskine-arm 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-medium Medium priority - this can be reviewed as time permits size-xs Estimated task size: extra small (a few hours at most) labels Aug 12, 2026

@gilles-peskine-arm gilles-peskine-arm 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.

LGTM

Signed-off-by: Yi Wu <yi.wu2@arm.com>

@gilles-peskine-arm gilles-peskine-arm 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.

LGTM

@valeriosetti
valeriosetti requested a review from bjwtaylor August 13, 2026 08:41
@valeriosetti valeriosetti removed the needs-reviewer This PR needs someone to pick it up for review label Aug 13, 2026
@github-project-automation github-project-automation Bot moved this from In Development to Has Approval in Non-roadmap pull requests Aug 14, 2026
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, priority-medium Medium priority - this can be reviewed as time permits size-xs Estimated task size: extra small (a few hours at most)

Projects

Status: Has Approval

Development

Successfully merging this pull request may close these issues.

Test inconsistent with documentation

4 participants