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

Fallback if C_GetInterface returns 0x54 #622

Merged
merged 1 commit into from
Mar 5, 2024

Conversation

space88man
Copy link
Contributor

@space88man space88man commented Feb 22, 2024

A vendor module (Thales Luna libCryptoki2_64.so 10.5.0 works with v6 / v7 hardware) exposes the symbol C_GetInterface but may return 0x54 if the HSM firmware is too old.

This PR allows fallback to C_GetFunctionList in this scenario.

Addresses #621

@coveralls
Copy link

coveralls commented Feb 22, 2024

Coverage Status

coverage: 69.473% (-0.004%) from 69.477%
when pulling 932127e on space88man:fallback-0x54
into 7756404 on p11-glue:master.

Copy link
Contributor

@ZoltanFridrich ZoltanFridrich left a comment

Choose a reason for hiding this comment

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

The change looks good. Thank you.

p11-kit/modules.c Outdated Show resolved Hide resolved
Addresses p11-glue#621

Signed-off-by: S-P Chan <shihping.chan@gmail.com>
Copy link
Contributor

@ZoltanFridrich ZoltanFridrich left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks!

@ZoltanFridrich ZoltanFridrich merged commit e8abb00 into p11-glue:master Mar 5, 2024
14 checks passed
@ZoltanFridrich ZoltanFridrich added this to the 0.25.4 milestone Jul 1, 2024
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.

3 participants