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

mbedtls: fix memory leak in error path #1417

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@lstefani
Contributor

lstefani commented Apr 13, 2017

Add missing our_ssl_sessionid free call in mbed_connect_step3().

Update mbedtls.c
Add missing our_ssl_sessionid free call in mbed_connect_step3().
@mention-bot

This comment has been minimized.

Show comment
Hide comment
@mention-bot

mention-bot Apr 13, 2017

@lstefani, thanks for your PR! By analyzing the history of the files in this pull request, we identified @sasq64, @bagder and @jay to be potential reviewers.

mention-bot commented Apr 13, 2017

@lstefani, thanks for your PR! By analyzing the history of the files in this pull request, we identified @sasq64, @bagder and @jay to be potential reviewers.

@bagder bagder changed the title from Update mbedtls.c to mbedtls: fix memory leak in error path Apr 13, 2017

@bagder bagder closed this in fc34782 Apr 15, 2017

@bagder

This comment has been minimized.

Show comment
Hide comment
@bagder

bagder Apr 15, 2017

Member

Thanks!

(I edited the patch to use a first line according to our style and I set your real name properly as author.)

Member

bagder commented Apr 15, 2017

Thanks!

(I edited the patch to use a first line according to our style and I set your real name properly as author.)

@lock lock bot locked as resolved and limited conversation to collaborators May 21, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.