[Python-checkins] gh-107916: Save the error code before decoding the filename in PyErr_SetFromErrnoWithFilename() etc (GH-107929)
serhiy-storchaka
webhook-mailer at python.org
Mon Aug 21 07:16:34 EDT 2023
https://github.com/python/cpython/commit/80bdebdd8593f007a2232ec04a7729bba6ebf12c
commit: 80bdebdd8593f007a2232ec04a7729bba6ebf12c
branch: main
author: Serhiy Storchaka <storchaka at gmail.com>
committer: serhiy-storchaka <storchaka at gmail.com>
date: 2023-08-21T14:16:31+03:00
summary:
gh-107916: Save the error code before decoding the filename in PyErr_SetFromErrnoWithFilename() etc (GH-107929)
files:
A Misc/NEWS.d/next/C API/2023-08-14-10-59-03.gh-issue-107916.KH4Muo.rst
M Python/errors.c
diff --git a/Misc/NEWS.d/next/C API/2023-08-14-10-59-03.gh-issue-107916.KH4Muo.rst b/Misc/NEWS.d/next/C API/2023-08-14-10-59-03.gh-issue-107916.KH4Muo.rst
new file mode 100644
index 0000000000000..f1f16609b118b
--- /dev/null
+++ b/Misc/NEWS.d/next/C API/2023-08-14-10-59-03.gh-issue-107916.KH4Muo.rst
@@ -0,0 +1,4 @@
+C API functions :c:func:`PyErr_SetFromErrnoWithFilename`,
+:c:func:`PyErr_SetExcFromWindowsErrWithFilename` and
+:c:func:`PyErr_SetFromWindowsErrWithFilename` save now the error code before
+calling :c:func:`PyUnicode_DecodeFSDefault`.
diff --git a/Python/errors.c b/Python/errors.c
index d9086c507251e..fb5b3ff2c7ba5 100644
--- a/Python/errors.c
+++ b/Python/errors.c
@@ -895,10 +895,12 @@ PyErr_SetFromErrnoWithFilename(PyObject *exc, const char *filename)
{
PyObject *name = NULL;
if (filename) {
+ int i = errno;
name = PyUnicode_DecodeFSDefault(filename);
if (name == NULL) {
return NULL;
}
+ errno = i;
}
PyObject *result = PyErr_SetFromErrnoWithFilenameObjects(exc, name, NULL);
Py_XDECREF(name);
@@ -998,6 +1000,9 @@ PyObject *PyErr_SetExcFromWindowsErrWithFilename(
{
PyObject *name = NULL;
if (filename) {
+ if ((DWORD)ierr == 0) {
+ ierr = (int)GetLastError();
+ }
name = PyUnicode_DecodeFSDefault(filename);
if (name == NULL) {
return NULL;
@@ -1028,6 +1033,9 @@ PyObject *PyErr_SetFromWindowsErrWithFilename(
{
PyObject *name = NULL;
if (filename) {
+ if ((DWORD)ierr == 0) {
+ ierr = (int)GetLastError();
+ }
name = PyUnicode_DecodeFSDefault(filename);
if (name == NULL) {
return NULL;
More information about the Python-checkins
mailing list