Skip to content

Testing: Fix HMAC unit test issue - #11492

Open
padelsbach wants to merge 1 commit into
wolfSSL:masterfrom
padelsbach:tls-bounds-test-failure
Open

padelsbach wants to merge 1 commit into
wolfSSL:masterfrom
padelsbach:tls-bounds-test-failure

Conversation

@padelsbach

Copy link
Copy Markdown
Contributor

Description

Fix for this issue, observed locally:

ERROR - tests/api/test_tls_bounds.c line 842 failed with:
    expected: ssl->hmac(ssl, digest, in, 50, 0, application_data, 1, PEER_ORDER) == (BAD_FUNC_ARG)
    result:   0 != -173

with the following build config:

./configure --disable-shared \
    --enable-swdev --enable-cryptocb \
    CPPFLAGS=-DWOLF_CRYPTO_CB_ONLY_SHA256

Note that CI does not build with this config, so it is not observed there.

Testing

Local testing

Checklist

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

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.

2 participants