search

osquery / osquery-go

Go bindings for osquery
MIT License
386 stars 78 forks source link

windows extension memleak related to go-winio #116

Closed jshen14 closed 10 months ago

jshen14 commented 11 months ago

Hi,

I notice there is memleak related to go-winio on windows only. Is anyone else noticing similar issue? Am I missing anything when using the lib?

this is found for commit: v0.0.0-20220706183148-4e1f83012b42

and mem leak is not found in latest commit at this comment: v0.0.0-20231006172600-d6f325f636a9

jshen14 commented 11 months ago

after debugging, I suspect it might be due to the used thrift did not close conn object

in latest thrift, I do see it close conn object

https://github.com/apache/thrift/blob/master/lib/go/thrift/simple_server.go#L209-L217