-fix: issue boostorg#239 exec_file does not close the FILE handle. Note: Using FILE* is a bad choice here because of possible exceptions, but Py_RunFile is a C function. This fix works, because Py_RunFile - as a C function - does not throw exceptions.

Hajo Kirchhoff · stefanseefeld · commit f028aa407699 · 2021-08-10T12:25:40.000-04:00
diff --git a/src/exec.cpp b/src/exec.cpp
@@ -129,6 +129,7 @@ object BOOST_PYTHON_DECL exec_file(char const *filename, object global, object l
                 f,
                 Py_file_input,
 		global.ptr(), local.ptr());
+  fclose(fs);
   if (!result) throw_error_already_set();
   return object(detail::new_reference(result));
 }

Original file line number	Diff line number	Diff line change
`@@ -129,6 +129,7 @@ object BOOST_PYTHON_DECL exec_file(char const *filename, object global, object l`
`129`	`129`	`f,`
`130`	`130`	`Py_file_input,`
`131`	`131`	`global.ptr(), local.ptr());`
	`132`	`+ fclose(fs);`
`132`	`133`	`if (!result) throw_error_already_set();`
`133`	`134`	`return object(detail::new_reference(result));`
`134`	`135`	`}`