[ https://issues.apache.org/jira/browse/THRIFT-3086?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14482381#comment-14482381 ]
ASF GitHub Bot commented on THRIFT-3086: ---------------------------------------- GitHub user jeking3 opened a pull request: https://github.com/apache/thrift/pull/429 [THRIFT-3086] fix a few minor valgrind identified issues You can merge this pull request into a Git repository by running: $ git pull https://github.com/jeking3/thrift bugfix/THRIFT-3086-minor-valgrind-issues Alternatively you can review and apply these changes as the patch at: https://github.com/apache/thrift/pull/429.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #429 ---- commit 86889762402795b78e4c1c9381d5b699e74c5fae Author: Jim King <jim.k...@simplivity.com> Date: 2015-04-07T01:38:06Z [THRIFT-3086] fix a few minor valgrind identified issues ---- > C++ Valgrind Error Cleanup > -------------------------- > > Key: THRIFT-3086 > URL: https://issues.apache.org/jira/browse/THRIFT-3086 > Project: Thrift > Issue Type: Bug > Components: C++ - Library > Affects Versions: 0.9.2 > Reporter: James E. King, III > Priority: Minor > > valgrind identified a few issues in the C++ library: > 1. In TFileTransport, inconsistent use of malloc then delete[] > 2. In ZLibTest, leaking allocations > 3. In RecursiveTest, leaking allocation > These are minor, but a clean valgrind run is a happy valgrind run. -- This message was sent by Atlassian JIRA (v6.3.4#6332)