-
-
Notifications
You must be signed in to change notification settings - Fork 11.1k
Closed
Labels
branch: 3.5Applies to openssl-3.5Applies to openssl-3.5branch: masterApplies to master branchApplies to master branchhold: discussionThe community needs to establish a consensus how to move forward with the issue or PRThe community needs to establish a consensus how to move forward with the issue or PRinvestgation neededIssue to research a problem, from which subsequent actionable issues can be createdIssue to research a problem, from which subsequent actionable issues can be createdseverity: fips changeThe pull request changes FIPS provider sourcesThe pull request changes FIPS provider sourcestriaged: bugThe issue/pr is/fixes a bugThe issue/pr is/fixes a bug
Description
The FIPS provider has a number of pairwise consistency checks in place. When these fail it is not consistent about how it handles them. Specifically, some failures cause the module to enter a failure state whereas other return an error (& possibly adding something to the error stack).
What behaviour do we want after a failed PCT?
Whatever we choose needs to be implemented across the board consistently.
Bear in mind that a PCT is mandated on key import, so that an attacker who manages to give the module an invalid key will cause such an error which could cause the module to enter the error state and cease functioning completely.
It's likely one of NIST's standards will provide options.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
branch: 3.5Applies to openssl-3.5Applies to openssl-3.5branch: masterApplies to master branchApplies to master branchhold: discussionThe community needs to establish a consensus how to move forward with the issue or PRThe community needs to establish a consensus how to move forward with the issue or PRinvestgation neededIssue to research a problem, from which subsequent actionable issues can be createdIssue to research a problem, from which subsequent actionable issues can be createdseverity: fips changeThe pull request changes FIPS provider sourcesThe pull request changes FIPS provider sourcestriaged: bugThe issue/pr is/fixes a bugThe issue/pr is/fixes a bug