Bugs item #1433886, was opened at 2006-02-17 16:56
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=1433886&group_id=5470
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Python Interpreter Core
Group: Python 2.3
Status: Open
Resolution: None
Priority: 5
Submitted By: Quentin Barnes (qbarnes)
Assigned to: Nobody/Anonymous (nobody)
Summary: pointer aliasing causes core dump, with workaround
Initial Comment:
When building 2.3.3 on Solaris using Studio 11 and
"-xalias_level=std" to enable ISO C99 pointer aliasing
rules, the interpreter would core dump when running
test_descr2.py. From examining the source, I believe
this problem exists in 2.4.2 as well, but did not
verify it.
I was able to simplify the code to invoke the problem
down to:
====
class C1(object):
def __new__(cls):
return super(C1, cls).__new__(cls)
a = C1()
====
I tracked the problem to super_init() in
Objects/typeobject.c.
The local variable "obj" is of type "PyObject *" and
"type" is of type "PyTypeObject *". In this function,
both variables can be pointing to the same object in
memory. Since the pointers are not of compatible
types, the compiler ends up optimizing out a memory
load between Py_INCREF(obj) and Py_INCREF(type) caching
a previously read value. This causes the object
reference counter to only be incremented once instead
of twice. When the object is deallocated, the
interpreter blows up.
A workaround is to cast one of the pointers to the
others type so that the compiler can see the pointer
aliasing potential. This is what I did in the patch.
What I suspect needs to be done as a more global fix is
to discontinue use of the PyObject_VAR_HEAD macro hack.
Casting between structures containing this macro
creates ISO C99 pointer aliasing violations. The
contents of the macro needs to be in its own structure
which is referenced so a compliant compiler can know of
possible aliasing.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=1433886&group_id=5470
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
