summaryrefslogtreecommitdiffstats
path: root/Python/pathconfig.c
diff options
context:
space:
mode:
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>2022-11-03 18:09:15 (GMT)
committerGitHub <noreply@github.com>2022-11-03 18:09:15 (GMT)
commit41a9f49bc5d52b44386fe44ed6a1983bd2cd65cc (patch)
treefc4a6c434c52661d35122967d6208adcc3fea659 /Python/pathconfig.c
parenteb023a84d92661bcde9dfe1641576774710f6c64 (diff)
downloadcpython-41a9f49bc5d52b44386fe44ed6a1983bd2cd65cc.zip
cpython-41a9f49bc5d52b44386fe44ed6a1983bd2cd65cc.tar.gz
cpython-41a9f49bc5d52b44386fe44ed6a1983bd2cd65cc.tar.bz2
gh-98978: Fix Py_SetPythonHome(NULL) (GH-99066)
Fix use-after-free in Py_SetPythonHome(NULL), Py_SetProgramName(NULL) and _Py_SetProgramFullPath(NULL) function calls. Issue reported by Benedikt Reinartz. (cherry picked from commit b07f546ea3a574bc3016fb023c157c65a47f4849) Co-authored-by: Victor Stinner <vstinner@python.org>
Diffstat (limited to 'Python/pathconfig.c')
-rw-r--r--Python/pathconfig.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/Python/pathconfig.c b/Python/pathconfig.c
index 69b7e10..be0f97c 100644
--- a/Python/pathconfig.c
+++ b/Python/pathconfig.c
@@ -261,6 +261,8 @@ Py_SetPythonHome(const wchar_t *home)
_PyMem_SetDefaultAllocator(PYMEM_DOMAIN_RAW, &old_alloc);
PyMem_RawFree(_Py_path_config.home);
+ _Py_path_config.home = NULL;
+
if (has_value) {
_Py_path_config.home = _PyMem_RawWcsdup(home);
}
@@ -282,6 +284,8 @@ Py_SetProgramName(const wchar_t *program_name)
_PyMem_SetDefaultAllocator(PYMEM_DOMAIN_RAW, &old_alloc);
PyMem_RawFree(_Py_path_config.program_name);
+ _Py_path_config.program_name = NULL;
+
if (has_value) {
_Py_path_config.program_name = _PyMem_RawWcsdup(program_name);
}
@@ -302,6 +306,8 @@ _Py_SetProgramFullPath(const wchar_t *program_full_path)
_PyMem_SetDefaultAllocator(PYMEM_DOMAIN_RAW, &old_alloc);
PyMem_RawFree(_Py_path_config.program_full_path);
+ _Py_path_config.program_full_path = NULL;
+
if (has_value) {
_Py_path_config.program_full_path = _PyMem_RawWcsdup(program_full_path);
}