Skip to content

Fix for STM32 hash status check logic (also fix NO_AES_192 and NO_AES_256) #8732

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

dgarske
Copy link
Contributor

@dgarske dgarske commented May 2, 2025

Description

  • Fix for STM32 Hashing status bit checking logic. Fixes ZD 19783.
    The digest calculation was indicating "not busy" before digest result (DCIS) was finished. This did not show up on most systems because the computation is usually done by the time we read the computed digest.

  • Fixes for NO_AES_192 / NO_AES_256 and added test case.
    Note: Most STM32 AES hardware engines do not support 192-bit.

  • Fixed bad BUILD_ logic for ADH-AES256-GCM-SHA384.

Testing

Tested on STM32U585 with IAR

Checklist

  • [ X ] added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@dgarske dgarske self-assigned this May 2, 2025
@dgarske
Copy link
Contributor Author

dgarske commented May 5, 2025

Retest this please

Cloning the remote Git repository
ERROR: Checkout failed

dgarske added 3 commits May 5, 2025 14:36
… calculation was indicating "not busy" before digest result (DCIS) was finished. This did not show up on most systems because the computation is usually done by the time it reads.
@dgarske dgarske force-pushed the stm32_hash_status branch from 923ba97 to d87a267 Compare May 5, 2025 21:50
@dgarske dgarske force-pushed the stm32_hash_status branch from d87a267 to 2199021 Compare May 5, 2025 22:55
@dgarske
Copy link
Contributor Author

dgarske commented May 6, 2025

Retest this please

@dgarske dgarske requested a review from SparkiDev May 6, 2025 21:26
@dgarske dgarske assigned SparkiDev and unassigned dgarske May 6, 2025
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