Skip to content

Use standard CKR_TOKEN_NOT_INITIALIZED error#413

Merged
simo5 merged 1 commit intolatchset:mainfrom
simo5:token_err
Feb 3, 2026
Merged

Use standard CKR_TOKEN_NOT_INITIALIZED error#413
simo5 merged 1 commit intolatchset:mainfrom
simo5:token_err

Conversation

@simo5
Copy link
Member

@simo5 simo5 commented Feb 2, 2026

Description

Replace the custom vendor error KRR_TOKEN_NOT_INITIALIZED with the standard PKCS#11 error CKR_TOKEN_NOT_INITIALIZED.

This resolves a FIXME and improves compliance with the PKCS#11 standard by using a standard error code where appropriate.

Checklist

  • Test suite updated with functionality tests
  • Test suite updated with negative tests
  • Rustdoc string were added or updated
  • CHANGELOG and/or other documentation added or updated
  • This is not a code change

Reviewer's checklist:

  • Any issues marked for closing are fully addressed
  • There is a test suite reasonably covering new functionality or modifications
  • This feature/change has adequate documentation added
  • A changelog entry is added if the change is significant
  • Code conform to coding style that today cannot yet be enforced via the check style test
  • Commits have short titles and sensible text
  • Doc string are properly updated

@simo5 simo5 requested a review from Jakuje February 2, 2026 20:23
Copy link
Contributor

@Jakuje Jakuje left a comment

Choose a reason for hiding this comment

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

code-wise looks good. Not sure whats up with the CI ...

@simo5
Copy link
Member Author

simo5 commented Feb 3, 2026

code-wise looks good. Not sure whats up with the CI ...

GitHub had a multi-hour incident with Actions yesterday.

Replace the custom vendor error `KRR_TOKEN_NOT_INITIALIZED` with the standard
PKCS#11 error `CKR_TOKEN_NOT_INITIALIZED`.

This resolves a FIXME and improves compliance with the PKCS#11 standard by
using a standard error code where appropriate.

Co-authored-by: Gemini <gemini@google.com>
Signed-off-by: Simo Sorce <simo@redhat.com>
@simo5 simo5 merged commit ee93407 into latchset:main Feb 3, 2026
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants