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

Memory leak in case *outlen is zero #970

Merged
merged 1 commit into from Aug 19, 2016

Conversation

Projects
None yet
3 participants
@mirekfranc
Copy link
Contributor

commented Aug 19, 2016

This was found during static analysis. In case of memory being allocated by Curl_base64_encode, but output length being zero, the memory pointed to by *outptr is never freed.

@mention-bot

This comment has been minimized.

Copy link

commented Aug 19, 2016

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

@mirekfranc mirekfranc force-pushed the mirekfranc:master branch from 4f021fc to b988fc5 Aug 19, 2016

@bagder bagder merged commit 0796a99 into curl:master Aug 19, 2016

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@bagder

This comment has been minimized.

Copy link
Member

commented Aug 19, 2016

thanks!

@lock lock bot locked as resolved and limited conversation to collaborators Jan 19, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.