[
https://issues.apache.org/jira/browse/THRIFT-4225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16519939#comment-16519939
]
Brian Forbis commented on THRIFT-4225:
--------------------------------------
I believe this has been fixed by THRIFT-4564
> Error handling malformed arguments leaks memory, corrupts transport buffers
> causing next RPC to fail
> ----------------------------------------------------------------------------------------------------
>
> Key: THRIFT-4225
> URL: https://issues.apache.org/jira/browse/THRIFT-4225
> Project: Thrift
> Issue Type: Bug
> Components: Node.js - Compiler, Node.js - Library
> Affects Versions: 0.10.0
> Reporter: Paul Taylor
> Priority: Major
>
> This issue is related to
> [THRIFT-2997|https://issues.apache.org/jira/browse/THRIFT-2997]. I have a
> workaround
> [here|https://github.com/graphistry/rxjs-mapd/blob/bf88da285ebc32ee6528f596919c626efccaf7ee/src/client/bind.ts#L85],
> but it's only possible because I'm wrapping all thrift client calls anyway.
> If the arguments to a thrift client method are malformed, the transport
> throws an error while writing the arguments to its buffers. Unfortunately,
> the buffers aren't reset (as would happen in flush), and the next RPC call
> will fail if any values were already written. Additionally, the callback
> isn't dereferenced from the client's _reqs table, leaking memory.
> From what I understand, this problem would need to be addressed in both the
> node lib and compiler.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
