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

win32: fix a potential memory leak in Curl_load_library #938

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@avakar

avakar commented Aug 1, 2016

If a call to GetSystemDirectory fails, the path pointer that was
previously allocated would be leaked. This makes sure that path is
always freed.

Martin Vejnár
win32: fix a potential memory leak in Curl_load_library
If a call to GetSystemDirectory fails, the `path` pointer that was
previously allocated would be leaked. This makes sure that `path` is
always freed.
@mention-bot

This comment has been minimized.

Show comment
Hide comment
@mention-bot

mention-bot Aug 1, 2016

@avakar, thanks for your PR! By analyzing the annotation information on this pull request, we identified @captain-caveman2k and @jay to be potential reviewers

mention-bot commented Aug 1, 2016

@avakar, thanks for your PR! By analyzing the annotation information on this pull request, we identified @captain-caveman2k and @jay to be potential reviewers

@bagder bagder closed this in 608b11a Aug 1, 2016

@bagder

This comment has been minimized.

Show comment
Hide comment
@bagder

bagder Aug 1, 2016

Member

Lovely, thanks!

Member

bagder commented Aug 1, 2016

Lovely, thanks!

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