From: Nikita Sobolev <mail@sobolevn.me>
Date: Mon, 12 Feb 2024 16:27:27 +0000 (+0300)
Subject: gh-87804: Fix the refleak in error handling of `_pystatvfs_fromstructstatfs` (#115335)
X-Git-Tag: v3.13.0a4~61
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=91bf01d4b15a40be4510fd9ee5e6dc8e9c019fce;p=thirdparty%2FPython%2Fcpython.git

gh-87804: Fix the refleak in error handling of `_pystatvfs_fromstructstatfs` (#115335)

It was the macro expansion! Sorry!
---

diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index 17032d9d490c..ef6d65623bf0 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -12916,14 +12916,15 @@ _pystatvfs_fromstructstatfs(PyObject *module, struct statfs st) {
 
     _Static_assert(sizeof(st.f_blocks) == sizeof(long long), "assuming large file");
 
-#define SET_ITEM(v, index, item)                   \
-    do {                                           \
-        if (item == NULL) {                        \
-            Py_DECREF(v);                          \
-            return NULL;                           \
-        }                                          \
-        PyStructSequence_SET_ITEM(v, index, item); \
-    } while (0)                                    \
+#define SET_ITEM(SEQ, INDEX, EXPR)                       \
+    do {                                                 \
+        PyObject *obj = (EXPR);                          \
+        if (obj == NULL) {                               \
+            Py_DECREF((SEQ));                            \
+            return NULL;                                 \
+        }                                                \
+        PyStructSequence_SET_ITEM((SEQ), (INDEX), obj);  \
+    } while (0)
 
     SET_ITEM(v, 0, PyLong_FromLong((long) st.f_iosize));
     SET_ITEM(v, 1, PyLong_FromLong((long) st.f_bsize));