IQ.Lvbs/pycapnp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix memory leak in _DynamicCapabilityClient._send_helper() (#398)

Browse Source 
The heap-allocated Request object was never freed after send().
  Each RPC call leaked ~72 bytes, growing linearly with usage.

  Added del request after request.send() to free the Request once
  the RemotePromise has been constructed.
This commit is contained in:
ed-tsn
2026-07-03 09:08:47 +02:00
committed by GitHub
parent 6aa1849da6
commit e2aaef8b6c
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
capnp/lib/capnp.pyx
View File

@@ -2320,7 +2320,9 @@ cdef class _DynamicCapabilityClient:
self._set_fields(request, name, args, kwargs) self._set_fields(request, name, args, kwargs)
return _RemotePromise()._init(request.send(), self) cdef _RemotePromise result = _RemotePromise()._init(request.send(), self)
del request
return result
cpdef _request_helper(self, name, firstSegmentWordSize, args, kwargs): cpdef _request_helper(self, name, firstSegmentWordSize, args, kwargs):
# if word_count is None: # if word_count is None: