Skip to content

Conversation

@gojimmypi
Copy link
Contributor

Description

Adds a wolfSSL debug message when WOLFSSL_MAX_ERROR_SZ is too small. The string comparison will otherwise fail:

        errStr = wc_GetErrorString(i);
        wc_ErrorString(i, out);
        if (XSTRCMP(errStr, out) != 0) {
            WOLFSSL_MSG("errStr does not match output");
            return WC_TEST_RET_ENC_I(-i);
        }

As the root cause is not immediately intuitive, this message may help. Perhaps a hard compile-time error instead?

See fixed length strings returned in wc_GetErrorString().

Fixes zd# n/a

Testing

How did you test?

Minimal testing on embedded device.

Checklist

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

@gojimmypi gojimmypi self-assigned this Sep 24, 2025
@gojimmypi
Copy link
Contributor Author

Jenkins retest this please

For java.io.IOException: Unexpected EOF Remote call to wolf-linux-cloud-node-[n]

@dgarske dgarske assigned gojimmypi and unassigned gojimmypi and wolfSSL-Bot Oct 2, 2025
@dgarske dgarske removed the request for review from wolfSSL-Bot October 9, 2025 00:14
@gojimmypi gojimmypi requested a review from dgarske October 9, 2025 00:29
@dgarske
Copy link
Contributor

dgarske commented Oct 9, 2025

Jenkins retest this please. FIPS v2 apt issue still

@dgarske dgarske merged commit f070ae1 into wolfSSL:master Oct 9, 2025
266 of 271 checks passed
@gojimmypi gojimmypi deleted the pr-max-error-sz branch October 10, 2025 23:10
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