[Lldb-commits] [PATCH] D18459: Fix FILE * leak in Python API
Zachary Turner via lldb-commits
lldb-commits at lists.llvm.org
Thu Mar 24 15:23:34 PDT 2016
zturner added inline comments.
Comment at: scripts/Python/python-typemaps.swig:532
@@ -531,3 +524,1 @@
> The problem is that here, we save the `FILE*` (`$1`) and let the `File` (`file`) go out of scope. So the `File` gets destructed (but it's after calling `file.Clear()`, so the close doesn't happen). But we still hold onto the `FILE*`, and we pass it, as `$1`, to the API call (not seen here, but it comes right after this block in the generated swig cpp code).
I think SWIG explicitly has a mechanism to handle this. Let me find it.
More information about the lldb-commits