thrift-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (THRIFT-4624) Refc binary leak in binary and compact protocols
Date Fri, 19 Oct 2018 12:07:00 GMT

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

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

jeking3 commented on issue #1585: THRIFT-4624: Fix refc binary leak
URL: https://github.com/apache/thrift/pull/1585#issuecomment-431341038
 
 
   If you want to pare down the PR to just the changes in `lib/erl/src/thrift_processor.erl`
and it passes the CI build, I can merge that.  I'm not sure the clone changes are needed per
comments.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Refc binary leak in binary and compact protocols
> ------------------------------------------------
>
>                 Key: THRIFT-4624
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4624
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Erlang - Library
>            Reporter: something
>            Priority: Major
>
> Pattern-matching on large (>64B) Erlang binaries merely produces slices of objects
on the Refc heap. Therefore Thrift binary and compact protocols should clone all binaries
they send to upper levels, otherwise there's a chance that transport-level messages will be
never freed.
> Patch is underway.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message