[issue4236] Crash when importing builtin module during interpreter shutdown

2010-12-14 Thread R. David Murray 

R. David Murray rdmur...@bitdance.com added the comment:

Brett applied his doc patch in r69092.  Attached is a patch that combines 
Simon's patch with Martin's test program turned into a unit test.  I confirm 
that the test suite passes with the patch applied (and fails with just the test 
applied).

From the text of the original error message, I wonder if there is some way to 
trigger this error at interpreter startup, and if so whether or not that is 
tested by the test suite.

Otherwise, is there any reason not to apply this patch?

--
nosy: +r.david.murray
stage: needs patch - patch review
Added file: 
http://bugs.python.org/file20041/check-import-machinery-only-with-test.diff

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2010-12-14 Thread Brett Cannon 

Brett Cannon br...@python.org added the comment:

Nothing jumps to my mind.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2010-12-14 Thread R. David Murray 

R. David Murray rdmur...@bitdance.com added the comment:

Committed to py3k in r87251, 3.1 in r87252, and 2.7 in r87255.

--
resolution:  - fixed
stage: patch review - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2010-11-20 Thread Simon Cross 

Simon Cross hodges...@gmail.com added the comment:

I'm attaching a patch to relax the check in PyModule_Create2 as suggested by 
the Amaury (http://bugs.python.org/issue4236#msg75409).

The patch uses PyThreadState_Get()-interp-modules == NULL to determine 
whether the import machinery has been cleaned up yet.

The test suite still appears to pass.

--
nosy: +hodgestar
Added file: http://bugs.python.org/file19655/check-import-machinery-only.diff

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2010-08-08 Thread Terry J. Reedy 

Changes by Terry J. Reedy tjre...@udel.edu:


--
versions:  -Python 2.6, Python 3.0

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2010-07-13 Thread Alexander Belopolsky 

Alexander Belopolsky belopol...@users.sourceforge.net added the comment:

The bug is still present in 2.7, and 3.2.

--
nosy: +belopolsky

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2009-05-29 Thread R. David Murray 

Changes by R. David Murray rdmur...@bitdance.com:


--
components: +Interpreter Core
stage:  - needs patch
type:  - crash
versions: +Python 2.6, Python 2.7, Python 3.1, Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-24 Thread Brett Cannon 

Changes by Brett Cannon [EMAIL PROTECTED]:


--
assignee: brett.cannon - 

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-20 Thread Martin v. Löwis 

Martin v. Löwis [EMAIL PROTECTED] added the comment:

Here is a test case (a.py) which produces a FatalError back to Python
2.4 at least)

Added file: http://bugs.python.org/file12073/a.py

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-20 Thread Barry A. Warsaw 

Barry A. Warsaw [EMAIL PROTECTED] added the comment:

After consultation with MvL and Crys_ on irc, we've agreed that this
should be fixed someday but it's a pathological case that shouldn't hold
up the release.  I'm lowering to critical because I don't think it
should even hold up the final release.

--
assignee:  - brett.cannon
priority: release blocker - critical

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-19 Thread Brett Cannon 

Brett Cannon [EMAIL PROTECTED] added the comment:

So if you look at Python/pythonrun.c, there is a comment from Tim Peters
(from r34776) where he explicitly points out that this is possibility
but that it has never been reported before. Oops. =)

--
nosy: +brett.cannon

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-19 Thread Brett Cannon 

Brett Cannon [EMAIL PROTECTED] added the comment:

I don't think there is anything to fix here beyond the docs for __del__.
You should never expect anything to be working in __del__, and that
includes the import machinery. It should be bare-bones, not trying to
pull in new code!

I have attached a patch which tries to clarify this fact in the language
docs for __del__.

Added file: http://bugs.python.org/file12066/clarify___del__.diff

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-07 Thread Amaury Forgeot d'Arc 

Amaury Forgeot d'Arc [EMAIL PROTECTED] added the comment:

Christian,
The initial example only works in interactive mode.
To make your script crash, you have to set
__builtins__._ = Crasher()

There are several other places where Crasher() will crash on shutdown:
 sys.argv.insert(0, Crasher())
 sys.ps1 = Crasher()
 sys.last_type = Crasher()
 sys.path_hooks.append(Crasher())
 sys.path_importer_cache[''] = Crasher()
But
 sys.meta_path.append(Crasher())
does not crash.

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-07 Thread Barry A. Warsaw 

Changes by Barry A. Warsaw [EMAIL PROTECTED]:


--
priority: deferred blocker - release blocker

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-06 Thread Martin v. Löwis 

Martin v. Löwis [EMAIL PROTECTED] added the comment:

PyModule_Create2 might be new, but the warning certainly is not - for
2.x, it lives in Py_InitModule4. Indeed, I can reproduce the problem
with 2.7a0, replacing the import of warnings with an import of imp.

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-06 Thread Christian Heimes 

Christian Heimes [EMAIL PROTECTED] added the comment:

Strange, I can't reproduce the problem with any Python version. Even
py3k doesn't crash with either import imp and import warnings.

$ cat ../issue4236.py
class Crasher(object):
def __del__(self):
print(__del__ called)
import imp

crasher = Crasher()

$ python2.4 issue4236.py
__del__ called
Exception exceptions.ImportError: 'No module named imp' in bound method
Crasher.__del__ of __main__.Crasher object at 0x7f0e6c4e44d0 ignored

$ python2.5 issue4236.py
__del__ called
Exception exceptions.ImportError: 'No module named imp' in bound method
Crasher.__del__ of __main__.Crasher object at 0x7fbbde5ef1d0 ignored

$ python2.6 issue4236.py
__del__ called
Exception ImportError: 'No module named imp' in bound method
Crasher.__del__ of __main__.Crasher object at 0x7f3d810dbf90 ignored

$ py3k/python issue4236.py
__del__ called
Exception ImportError: 'No module named imp' in bound method
Crasher.__del__ of __main__.Crasher object at 0x9ab3e0 ignored

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-05 Thread Barry A. Warsaw 

Barry A. Warsaw [EMAIL PROTECTED] added the comment:

This seems like a corner case to me.  We should fix it before the final
release, but I don't think it's worth holding up 3.0rc2 for it.  Deferring.

--
priority: release blocker - deferred blocker

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-11-03 Thread Barry A. Warsaw 

Barry A. Warsaw [EMAIL PROTECTED] added the comment:

Can you add some tests for this problem?

--
nosy: +barry

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-10-31 Thread Amaury Forgeot d'Arc 

Amaury Forgeot d'Arc [EMAIL PROTECTED] added the comment:

Setting builtins._=None is not enough.

My example is very simple, but the crash I had initially was during the
deletion of sys.last_tb: the C() object may be stored anywhere.

I agree that the test in PyModule_Create2 is too strong: the import
machinery is still functional at this point. It could be replaced by
something that becomes true on _PyImport_Init(), and false on
_PyImport_Fini().

--
nosy: +loewis

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-10-30 Thread STINNER Victor 

STINNER Victor [EMAIL PROTECTED] added the comment:

The problem occurs in PyImport_Cleanup() on this line:
  PyDict_SetItemString(dict, _, Py_None);

So the problem is that _ (special variable in the Python 
interpreter) is removed too late, or that a destructor loads a module 
when Python is exiting.

Call trace:

Py_Finalize() - PyImport_Cleanup() - 
builtin___import__(warnings) - builtin___import__(_warnings) - 
_PyWarnings_Init() - PyModule_Create2()

Full backtrace:
(gdb) where
#0  0xe410 in __kernel_vsyscall ()
#1  0xb7d68875 in raise () from /lib/tls/i686/cmov/libc.so.6
#2  0xb7d6a201 in abort () from /lib/tls/i686/cmov/libc.so.6
#3  0x080ba54b in Py_FatalError (msg=0x8163ad0 Interpreter not 
initialized (version mismatch?)) at Python/pythonrun.c:1922
#4  0x081248e2 in PyModule_Create2 (module=0x8174e20, 
module_api_version=1013) at Objects/moduleobject.c:71
#5  0x080882d7 in _PyWarnings_Init () at Python/_warnings.c:879
#6  0x080abcb0 in init_builtin (name=0xbf8627a3 _warnings) at 
Python/import.c:1867
#7  0x080aba9f in load_module (name=0xbf8627a3 _warnings, fp=0x0, 
buf=0xbf8627a3 _warnings, type=6, loader=0x0) at 
Python/import.c:1795
#8  0x080ad54f in import_submodule (mod=0x81695a4, 
subname=0xbf863833 _warnings, fullname=0xbf863833 _warnings) at 
Python/import.c:2567
#9  0x080acd21 in load_next (mod=0x81695a4, altmod=0x81695a4, 
p_name=0xbf863814, buf=0xbf863833 _warnings, p_buflen=0xbf86381c)
at Python/import.c:2372
#10 0x080ac26e in import_module_level (name=0x0, globals=0xb7b2802c, 
locals=0xb7b2802c, fromlist=0xb7b208fc, level=0) at 
Python/import.c:2089
#11 0x080ac5b2 in PyImport_ImportModuleLevel 
(name=0xb7b2c650 _warnings, globals=0xb7b2802c, locals=0xb7b2802c, 
fromlist=0xb7b208fc,
level=0) at Python/import.c:2140
#12 0x08088b0d in builtin___import__ (self=0xb7be8eac, 
args=0xb7b2098c, kwds=0x0) at Python/bltinmodule.c:173
#13 0x081241c6 in PyCFunction_Call (func=0xb7be8f6c, arg=0xb7b2098c, 
kw=0x0) at Objects/methodobject.c:84
#14 0x080ec5b8 in PyObject_Call (func=0xb7be8f6c, arg=0xb7b2098c, 
kw=0x0) at Objects/abstract.c:2184
#15 0x08094bae in PyEval_CallObjectWithKeywords (func=0xb7be8f6c, 
arg=0xb7b2098c, kw=0x0) at Python/ceval.c:3278
#16 0x080916dd in PyEval_EvalFrameEx (f=0x826661c, throwflag=0) at 
Python/ceval.c:1985
#17 0x08093d64 in PyEval_EvalCodeEx (co=0xb7b25608, 
globals=0xb7b2802c, locals=0xb7b2802c, args=0x0, argcount=0, kws=0x0, 
kwcount=0,
defs=0x0, defcount=0, kwdefs=0x0, closure=0x0) at 
Python/ceval.c:2835
#18 0x0808d2b4 in PyEval_EvalCode (co=0xb7b25608, globals=0xb7b2802c, 
locals=0xb7b2802c) at Python/ceval.c:519
#19 0x080a9be8 in PyImport_ExecCodeModuleEx 
(name=0xbf866dd3 warnings, co=0xb7b25608,
pathname=0xbf864ca3 /home/haypo/prog/py3k/Lib/warnings.pyc) at 
Python/import.c:729
#20 0x080aa41e in load_source_module (name=0xbf866dd3 warnings, 
pathname=0xbf864ca3 /home/haypo/prog/py3k/Lib/warnings.pyc, 
fp=0x82650f8)
at Python/import.c:1020
#21 0x080aba14 in load_module (name=0xbf866dd3 warnings, 
fp=0x82650f8, buf=0xbf865d43 /home/haypo/prog/py3k/Lib/warnings.py, 
type=1,
loader=0x0) at Python/import.c:1773
#22 0x080ad54f in import_submodule (mod=0x81695a4, 
subname=0xbf866dd3 warnings, fullname=0xbf866dd3 warnings) at 
Python/import.c:2567
#23 0x080acd21 in load_next (mod=0x81695a4, altmod=0x81695a4, 
p_name=0xbf866db4, buf=0xbf866dd3 warnings, p_buflen=0xbf866dbc)
at Python/import.c:2372
#24 0x080ac26e in import_module_level (name=0x0, globals=0xb7c0402c, 
locals=0x81695a4, fromlist=0x81695a4, level=0) at Python/import.c:2089
#25 0x080ac5b2 in PyImport_ImportModuleLevel 
(name=0xb7b1f850 warnings, globals=0xb7c0402c, locals=0x81695a4, 
fromlist=0x81695a4, level=0)
at Python/import.c:2140
#26 0x08088b0d in builtin___import__ (self=0xb7be8eac, 
args=0xb7b87e6c, kwds=0x0) at Python/bltinmodule.c:173
#27 0x081241c6 in PyCFunction_Call (func=0xb7be8f6c, arg=0xb7b87e6c, 
kw=0x0) at Objects/methodobject.c:84
#28 0x080ec5b8 in PyObject_Call (func=0xb7be8f6c, arg=0xb7b87e6c, 
kw=0x0) at Objects/abstract.c:2184
#29 0x08094bae in PyEval_CallObjectWithKeywords (func=0xb7be8f6c, 
arg=0xb7b87e6c, kw=0x0) at Python/ceval.c:3278
#30 0x080916dd in PyEval_EvalFrameEx (f=0x8264fa4, throwflag=0) at 
Python/ceval.c:1985
#31 0x08093d64 in PyEval_EvalCodeEx (co=0xb7b62c80, 
globals=0xb7c0402c, locals=0x0, args=0xb7b1f4d8, argcount=1, kws=0x0, 
kwcount=0,
defs=0x0, defcount=0, kwdefs=0x0, closure=0x0) at 
Python/ceval.c:2835
---Type return to continue, or q return to quit---
#32 0x08111b1e in function_call (func=0xb7b2626c, arg=0xb7b1f4cc, 
kw=0x0) at Objects/funcobject.c:628
#33 0x080ec5b8 in PyObject_Call (func=0xb7b2626c, arg=0xb7b1f4cc, 
kw=0x0) at Objects/abstract.c:2184
#34 0x080fe956 in method_call (func=0xb7b2626c, arg=0xb7b1f4cc, 
kw=0x0) at Objects/classobject.c:323
#35 0x080ec5b8 in PyObject_Call (func=0xb7b1f6cc, arg=0xb7bde02c, 
kw=0x0) at Objects/abstract.c:2184
#36 0x08094bae in PyEval_CallObjectWithKeywords

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-10-30 Thread STINNER Victor 

STINNER Victor [EMAIL PROTECTED] added the comment:

Here is a patch fixing this issue: set builtins._=None at exit.

The new introduced function, PyRun_InteractiveCleanup(), might also be 
called at the end of other functions like PyRun_InteractiveOne() or 
PyRun_InteractiveOneFlags(). If not, PyRun_InteractiveCleanup() should 
be exported.

--
keywords: +patch
Added file: http://bugs.python.org/file11914/interactive_cleanup.patch

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-10-30 Thread STINNER Victor 

STINNER Victor [EMAIL PROTECTED] added the comment:

Another solution: just remove the initialization test from 
PyModule_Create2(). I don't know if it's a safety test or something 
like that.

Added file: http://bugs.python.org/file11915/pymodule_create2.patch

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-10-30 Thread STINNER Victor 

STINNER Victor [EMAIL PROTECTED] added the comment:

Python2 is not affected because the bug comes from PyModule_Create2() 
which was introduced in Python3.

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-10-29 Thread Amaury Forgeot d'Arc 

New submission from Amaury Forgeot d'Arc [EMAIL PROTECTED]:

The following interactive session crashes the interpreter on shutdown:

 class C:
...   def __del__(self):
...  import warnings
...
 C()
__main__.C object at 0x00C29D58
 ^Z
Fatal Python error: Interpreter not initialized (version mismatch?)


This problem actually happens with the file object, which can call PyErr_WarnEx 
when it 
is deallocated and closed, see example in issue4226.
This Fatal message is new in 3.0. If I ignore it all seems correct, and the 
module is 
correctly imported.

--
messages: 75344
nosy: amaury.forgeotdarc
priority: release blocker
severity: normal
status: open
title: Crash when importing builtin module during interpreter shutdown
versions: Python 3.0

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4236] Crash when importing builtin module during interpreter shutdown

2008-10-29 Thread Christian Heimes 

Christian Heimes [EMAIL PROTECTED] added the comment:

I can reproduce the same output with CTRL+D on Linux.

$ python3.0
Python 3.0rc1+ (py3k:67029M, Oct 26 2008, 23:48:21)
[GCC 4.2.4 (Ubuntu 4.2.4-1ubuntu3)] on linux2
Type help, copyright, credits or license for more information.
 class C:
... def __del__(self):
... print(del)
... import warnings
...
[33876 refs]
 C()
__main__.C object at 0x7f4740a98d80
[33893 refs]
 exit()
[33900 refs]
del
Fatal Python error: Interpreter not initialized (version mismatch?)
Aborted

--
nosy: +christian.heimes

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4236
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com