diff options
| author | Nick Coghlan <ncoghlan@gmail.com> | 2022-10-30 12:01:30 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-10-30 12:01:30 (GMT) |
| commit | 05e48865be69e1e5824f6915b588ff054717bb42 (patch) | |
| tree | 716ac0f9e038f575b4212db6d4a7b6a76b484d59 /Python | |
| parent | 76f989dc3e668d15b3ec9a90bf6530276530acac (diff) | |
| download | cpython-05e48865be69e1e5824f6915b588ff054717bb42.zip cpython-05e48865be69e1e5824f6915b588ff054717bb42.tar.gz cpython-05e48865be69e1e5824f6915b588ff054717bb42.tar.bz2 | |
gh-96853: Restore test coverage for Py_Initialize(Ex) (GH-98212)
* As most of `test_embed` now uses `Py_InitializeFromConfig`, add
a specific test case to cover `Py_Initialize` (and `Py_InitializeEx`)
* Rename `_testembed` init helper to clarify the API used
* Add a `PyConfig_Clear` call in `Py_InitializeEx` to make
the code more obviously correct (it already didn't leak as
none of the dynamically allocated config fields were being
populated, but it's clearer if the wrappers follow the
documented API usage guidelines)
Diffstat (limited to 'Python')
| -rw-r--r-- | Python/pylifecycle.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/Python/pylifecycle.c b/Python/pylifecycle.c index 334abfb..d26ae74 100644 --- a/Python/pylifecycle.c +++ b/Python/pylifecycle.c @@ -1313,6 +1313,7 @@ Py_InitializeEx(int install_sigs) config.install_signal_handlers = install_sigs; status = Py_InitializeFromConfig(&config); + PyConfig_Clear(&config); if (_PyStatus_EXCEPTION(status)) { Py_ExitStatusException(status); } |