diff options
author | Victor Stinner <victor.stinner@gmail.com> | 2012-10-02 00:55:07 +0200 |
---|---|---|
committer | Victor Stinner <victor.stinner@gmail.com> | 2012-10-02 00:55:07 +0200 |
commit | bc68926a83e681a8ae10b80493830144c4485f19 (patch) | |
tree | 0063d5da6a12494b7f80fbe6e33f8f716db2dca7 /Python/dynload_shlib.c | |
parent | 2c5268124f1e3fcc86a813c2c09077912f3d38c4 (diff) | |
download | cpython-bc68926a83e681a8ae10b80493830144c4485f19.tar.gz |
Close #15766: Catch exceptions while raising the ImportError in imp.load_dynamic()
Diffstat (limited to 'Python/dynload_shlib.c')
-rw-r--r-- | Python/dynload_shlib.c | 25 |
1 files changed, 18 insertions, 7 deletions
diff --git a/Python/dynload_shlib.c b/Python/dynload_shlib.c index 0ca65c7877..6df6199ae8 100644 --- a/Python/dynload_shlib.c +++ b/Python/dynload_shlib.c @@ -129,19 +129,30 @@ dl_funcptr _PyImport_GetDynLoadFunc(const char *shortname, handle = dlopen(pathname, dlopenflags); if (handle == NULL) { - PyObject *mod_name = NULL; - PyObject *path = NULL; - PyObject *error_ob = NULL; + PyObject *mod_name; + PyObject *path; + PyObject *error_ob; const char *error = dlerror(); if (error == NULL) error = "unknown dlopen() error"; error_ob = PyUnicode_FromString(error); - path = PyUnicode_FromString(pathname); + if (error_ob == NULL) + return NULL; mod_name = PyUnicode_FromString(shortname); + if (mod_name == NULL) { + Py_DECREF(error_ob); + return NULL; + } + path = PyUnicode_FromString(pathname); + if (path == NULL) { + Py_DECREF(error_ob); + Py_DECREF(mod_name); + return NULL; + } PyErr_SetImportError(error_ob, mod_name, path); - Py_XDECREF(error_ob); - Py_XDECREF(path); - Py_XDECREF(mod_name); + Py_DECREF(error_ob); + Py_DECREF(mod_name); + Py_DECREF(path); return NULL; } if (fp != NULL && nhandles < 128) |