[ 
https://issues.apache.org/jira/browse/THRIFT-2936?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14903176#comment-14903176
 ] 

ASF GitHub Bot commented on THRIFT-2936:
----------------------------------------

GitHub user jeking3 opened a pull request:

    https://github.com/apache/thrift/pull/619

    THRIFT-2936: fix crash in memory handling in cleanupOpenSSL that was 
introduced in 0.9.3

    fix crash in memory handling if executable calls cleanupOpenSSL more than 
once

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jeking3/thrift bugfix/THRIFT-2936

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/thrift/pull/619.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 #619
    
----
commit 10453446b3a4dd72b90200ef61eac1c00ec12188
Author: James E. King, III <jim.k...@simplivity.com>
Date:   2015-09-22T18:33:42Z

    THRIFT-2936: fix crash in memory handling if executable calls 
cleanupOpenSSL more than once

----


> Minor memory leak in SSL
> ------------------------
>
>                 Key: THRIFT-2936
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2936
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Library
>    Affects Versions: 0.9.3
>         Environment: Ubuntu 14.04.1 LTS
>            Reporter: Cristian Klein
>            Assignee: James E. King, III
>            Priority: Minor
>              Labels: easyfix, newbie, patch
>             Fix For: 0.9.3
>
>         Attachments: 0001-THRIFT-2936-Fix-minor-memory-leak-in-SSL.patch
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Valgrind shows that Thrift clients that use SSL leak 64 bytes in two chunks. 
> This is because the list of available compression methods is not freed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to