isa/sudo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge sudo_module_register_loghandler and sudo_module_set_default_loghandler.

Browse Source 
We now create the LogHandler class for each interpreter in
python_plugin_init() instead of just once in sudo_module_init().
This fixes the crash seen in Py_EndInterpreter() with Python 3.12
and significantly reduces the number of leaked objects tracked by
MemorySanitizer.
This commit is contained in:
Todd C. Miller
2023-07-24 11:07:45 -06:00
parent 56b829f66b
commit 09b9632699
4 changed files with 28 additions and 51 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
plugins/python/python_plugin_common.c
View File

@@ -542,8 +542,9 @@ python_plugin_init(struct PluginContext *plugin_ctx, char * const plugin_options
goto cleanup;
}
if (sudo_module_set_default_loghandler() < 0)
if (sudo_module_set_default_loghandler() != SUDO_RC_OK) {
goto cleanup;
}
if (_python_plugin_set_path(plugin_ctx, _lookup_value(plugin_options, "ModulePath")) != SUDO_RC_OK) {
goto cleanup;