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 mem leak related to do_os_response() function in ts_lua plugin #2621

Merged
merged 1 commit into from Oct 5, 2017

Conversation

Projects
None yet
2 participants
@shukitchan
Contributor

shukitchan commented Oct 5, 2017

The exit for this block is missing a few things needed to release the resources
Those are things we are already doing when exiting the function in other execution path

@shukitchan shukitchan added this to the 8.0.0 milestone Oct 5, 2017

@shukitchan shukitchan self-assigned this Oct 5, 2017

@shukitchan shukitchan requested review from zwoop and bryancall Oct 5, 2017

@shukitchan shukitchan changed the title from Fix mem leak related to do_os_response() function to Fix mem leak related to do_os_response() function in ts_lua plugin Oct 5, 2017

@PSUdaemon

LGTM

@shukitchan shukitchan merged commit 66d5caf into apache:master Oct 5, 2017

7 checks passed

Jenkins Clang-Analyzer Build finished.
Details
Jenkins FreeBSD Build finished.
Details
Jenkins ICC Build finished.
Details
Jenkins Linux Build finished.
Details
Jenkins RAT Build finished.
Details
Jenkins autest Build finished.
Details
Jenkins clang-format Build finished.
Details
@PSUdaemon

This comment has been minimized.

Show comment
Hide comment
@PSUdaemon

PSUdaemon Oct 6, 2017

Contributor

@zwoop we should backport this to 7.1.

Contributor

PSUdaemon commented Oct 6, 2017

@zwoop we should backport this to 7.1.

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