diff options
| author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2021-06-02 23:50:38 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-06-02 23:50:38 (GMT) |
| commit | 0e9af8cae314e4b0e770fe48d5f7b5f540c0b257 (patch) | |
| tree | e50333b0588c8d8573518713fe6d873d3d3ea2ae /Modules/getpath.c | |
| parent | e5e93e6145090a636e67766a53b758d7ac78e3ad (diff) | |
| download | cpython-0e9af8cae314e4b0e770fe48d5f7b5f540c0b257.zip cpython-0e9af8cae314e4b0e770fe48d5f7b5f540c0b257.tar.gz cpython-0e9af8cae314e4b0e770fe48d5f7b5f540c0b257.tar.bz2 | |
bpo-44285: getpath.c: Assert that env_file is NULL during an error check (GH-26486)
This was flagged by a static analyzer, but the logic of why this is NULL on error is hard to follow for humans as well.
(cherry picked from commit bdb56902a3bfe12b10f85a941d5dd0eae739f1a8)
Co-authored-by: stratakis <cstratak@redhat.com>
Diffstat (limited to 'Modules/getpath.c')
| -rw-r--r-- | Modules/getpath.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/Modules/getpath.c b/Modules/getpath.c index 44453f2..363d62a 100644 --- a/Modules/getpath.c +++ b/Modules/getpath.c @@ -1263,6 +1263,7 @@ calculate_read_pyenv(PyCalculatePath *calculate) status = calculate_open_pyenv(calculate, &env_file); if (_PyStatus_EXCEPTION(status)) { + assert(env_file == NULL); return status; } if (env_file == NULL) { |