Fixed incorrect management key algorithm used for FIPS in TryChangeManagementKey

[DennisDyallo]

Description

The PivSession.ChangeManagementKey method was incorrectly assuming TripleDes as management key algorithm. This is now fixed.

Fixes: #160

Type of change

• Refactor (non-breaking change which improves code quality or performance)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How has this been tested?

Had both FIPS and normal 5.7 keys. Ran the TryChangeManagementKey method and reproduced and fixed the bug.

Test configuration:

• OS version: Windows 11
• Firmware version: 5.7.2, 5.4.3
• Yubikey model¹:
  Normal 5 Multiprotocol, FIPS

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have run dotnet format to format my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

Footnotes

1. See Yubikey models (Multi-protocol, Security Key, FIPS, Bio, YubiHSM, YubiHSM FIPS) ↩

[AdamVe]

The PR looks good, please check my comments.

[github-actions]

Test Results: Windows

    2 files      2 suites   4s ⏱️
3 703 tests 3 703 ✅ 0 💤 0 ❌
3 705 runs  3 705 ✅ 0 💤 0 ❌

Results for commit 9de0750.

[github-actions]

Test Results: Ubuntu

    2 files      2 suites   10s ⏱️
3 695 tests 3 695 ✅ 0 💤 0 ❌
3 697 runs  3 697 ✅ 0 💤 0 ❌

Results for commit 9de0750.

[github-actions]

Test Results: MacOS

    2 files      2 suites   4s ⏱️
3 695 tests 3 695 ✅ 0 💤 0 ❌
3 697 runs  3 697 ✅ 0 💤 0 ❌

Results for commit 9de0750.

[github-actions]

Package	Line Rate	Branch Rate	Complexity	Health
Yubico.Core	43%	32%	4221	➖
Yubico.YubiKey	50%	47%	19155	➖
Summary	49% (32222 / 66084)	44% (8191 / 18579)	23376	➖

Minimum allowed line rate is 40%