diff options
| author | Nick Coghlan <ncoghlan@gmail.com> | 2022-10-30 22:01:30 +1000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-10-30 22:01:30 +1000 |
| commit | 05e48865be69e1e5824f6915b588ff054717bb42 (patch) | |
| tree | 716ac0f9e038f575b4212db6d4a7b6a76b484d59 /Python/pylifecycle.c | |
| parent | 76f989dc3e668d15b3ec9a90bf6530276530acac (diff) | |
| download | cpython-05e48865be69e1e5824f6915b588ff054717bb42.tar.gz cpython-05e48865be69e1e5824f6915b588ff054717bb42.zip | |
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/pylifecycle.c')
| -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 334abfb191f..d26ae74a0f1 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); } |