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

wolfSSL/CyaSSL: Fix memory leak in Curl_cyassl_random #2784

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@cariepointer
Contributor

cariepointer commented Jul 24, 2018

RNG structure must be freed by call to FreeRng after its use in
Curl_cyassl_random. This call fixes Valgrind failures when running the
test suite with wolfSSL.

wolfSSL/CyaSSL: Fix memory leak in Curl_cyassl_random
RNG structure must be freed by call to FreeRng after its use in
Curl_cyassl_random. This call fixes Valgrind failures when running the
test suite with wolfSSL.
@bagder

bagder approved these changes Jul 25, 2018

@bagder

This comment has been minimized.

Show comment
Hide comment
@bagder

bagder Jul 26, 2018

Member

Thanks!

Member

bagder commented Jul 26, 2018

Thanks!

xquery added a commit to xquery/curl that referenced this pull request Aug 9, 2018

wolfSSL/CyaSSL: Fix memory leak in Curl_cyassl_random
RNG structure must be freed by call to FreeRng after its use in
Curl_cyassl_random. This call fixes Valgrind failures when running the
test suite with wolfSSL.

Closes #2784

falconindy added a commit to falconindy/curl that referenced this pull request Sep 10, 2018

wolfSSL/CyaSSL: Fix memory leak in Curl_cyassl_random
RNG structure must be freed by call to FreeRng after its use in
Curl_cyassl_random. This call fixes Valgrind failures when running the
test suite with wolfSSL.

Closes #2784
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment