From: Victor Stinner <victor.stinner@gmail.com>
Date: Wed, 6 Nov 2013 22:50:10 +0000 (+0100)
Subject: Issue #19512: fileio_init() reuses PyId_name identifier instead of "name"
X-Git-Tag: v3.4.0b1~362
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=d9d0419a9fc85d1f4c8744f68156fd6b305d5ff0;p=thirdparty%2FPython%2Fcpython.git

Issue #19512: fileio_init() reuses PyId_name identifier instead of "name"
literal string
---

diff --git a/Modules/_io/fileio.c b/Modules/_io/fileio.c
index 27995e543b03..0e1e709efd94 100644
--- a/Modules/_io/fileio.c
+++ b/Modules/_io/fileio.c
@@ -59,6 +59,8 @@ typedef struct {
 
 PyTypeObject PyFileIO_Type;
 
+_Py_IDENTIFIER(name);
+
 #define PyFileIO_Check(op) (PyObject_TypeCheck((op), &PyFileIO_Type))
 
 int
@@ -427,7 +429,7 @@ fileio_init(PyObject *oself, PyObject *args, PyObject *kwds)
     _setmode(self->fd, O_BINARY);
 #endif
 
-    if (PyObject_SetAttrString((PyObject *)self, "name", nameobj) < 0)
+    if (_PyObject_SetAttrId((PyObject *)self, &PyId_name, nameobj) < 0)
         goto error;
 
     if (self->appending) {
@@ -1036,7 +1038,6 @@ mode_string(fileio *self)
 static PyObject *
 fileio_repr(fileio *self)
 {
-    _Py_IDENTIFIER(name);
     PyObject *nameobj, *res;
 
     if (self->fd < 0)