Skip to content

Fix missing header includes and conditional compilation issue for PSoC6 port.#9776

Merged
dgarske merged 1 commit intowolfSSL:masterfrom
Pushyanth-Infineon:fix_psoc6_sha_includes
Feb 17, 2026
Merged

Fix missing header includes and conditional compilation issue for PSoC6 port.#9776
dgarske merged 1 commit intowolfSSL:masterfrom
Pushyanth-Infineon:fix_psoc6_sha_includes

Conversation

@Pushyanth-Infineon
Copy link
Copy Markdown
Contributor

Fix missing header includes and conditional compilation issue in PSoC6 crypto hardware acceleration port.
Guard the aes->left = 0 assignment to enable when WOLFSSL_AES_CFB is defined.

Description

1. Added Required SHA Header Includes

Added conditional includes based on enabled hash algorithms to fix the compilation issues

#ifndef NO_SHA256
#include "wolfssl/wolfcrypt/sha256.h"
#endif

#ifdef WOLFSSL_SHA3
#include "wolfssl/wolfcrypt/sha3.h" 
#endif

#if defined(WOLFSSL_SHA512) || defined(WOLFSSL_SHA384)
#include "wolfssl/wolfcrypt/sha512.h"
#endif

2. Guarded aes->left Assignment

Wrapped the aes->left = 0 assignment with the appropriate conditional:

#if defined(WOLFSSL_AES_CFB)
    aes->left   = 0;
#endif

Testing

Tested with CY8CPROTO-062-4343W board on ModusToolbox™ IDE with tests present in wolfcrypt/test/test.c:wolfcrypt_test()
PSoC6 PDL library: mtb-pdl-cat1

Checklist

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

@wolfSSL-Bot
Copy link
Copy Markdown

Can one of the admins verify this patch?

@dgarske
Copy link
Copy Markdown
Member

dgarske commented Feb 16, 2026

Okay to test. Contributor agreement on file.

Comment thread wolfcrypt/src/port/cypress/psoc6_crypto.c
…6 crypto hardware acceleration port.

Guard the `aes->left = 0` assignment to enable when WOLFSSL_AES_CFB is defined.
@dgarske
Copy link
Copy Markdown
Member

dgarske commented Feb 16, 2026

Jenkins retest this please. Generic config agent offline exception

@dgarske dgarske merged commit 5960a36 into wolfSSL:master Feb 17, 2026
569 of 570 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.

3 participants