Skip to content
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

Fix for WOLFSSL_NO_REALLOC build crash #8371

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

lealem47
Copy link
Contributor

Description

Fix for make check crash introduced in #8369

Testing

./configure --enable-all CFLAGS=-DWOLFSSL_NO_REALLOC && make check

Checklist

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

@lealem47 lealem47 self-assigned this Jan 23, 2025
@lealem47 lealem47 changed the title Fix for NO_REALLOC build crash Fix for WOLFSSL_NO_REALLOC build crash Jan 23, 2025
@lealem47
Copy link
Contributor Author

Jenkins Retest this Please

Failure in scripts/openssl.test with ./configure --enable-sp-math-all --enable-sp CFLAGS="-DALT_ECC_SIZE" --enable-sp-asm --enable-opensslextra

New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : 0000
    Session-ID: 
    Session-ID-ctx: 
    Master-Key: 
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    Start Time: 1737675320
    Timeout   : 7200 (sec)
    Verify return code: 0 (ok)
    Extended master secret: no
---
client failed! Suite = DHE-RSA-AES128-SHA256 version = 3
in cleanup
killing server: OpenSSL_RSA (61717)
killing server: wolfSSL_RSA (51923)
killing server: OpenSSL_ECDH[E]-ECDSA (62939)
killing server: wolfSSL_ECDH[E]-ECDSA (51923)
FAIL scripts/openssl.test (exit status: 1)

@dgarske dgarske merged commit f1e06e1 into wolfSSL:master Jan 24, 2025
155 of 156 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.

2 participants