Issue 17296: Cannot unpickle classes derived from 'Exception'

When pickling/unpickling a class that derives from the builtin class Exception, unpickling results in a

TypeError: ('__init__() takes at least 2 arguments (1 given)', <class '__main__.TestException'>, ())

A standard exception like ValueError can be pickled/unpickled without any problem.

This was observed for versions 2.7.3 and 3.2.3. for both pickle and cPickle.

A script (cpickle) that shows that behavior is included.

This is related (but I do not quite understand how) to the closed Issue1692335.

That issue was only fixed in 3.3.  Does your code work in 3.3?

I have not tried in 3.3. I have no running installation of 3.3.

I need a solution for 2.7 for a Zope project that was just ported to 2.7. My test for Python3 was halfheartedly on my standard Python3 installation (3.2) after reading Issue1692335.

The reason I ask is because that issue ends with "reopen if you think this should be backported", so a backport is at least within the real of possibility :)

I modified your script to run under both python2 and python3.  I get the error for 2.5, 2.6, and 2.7.  The script produces the same output (without the EXCEPTION ## EXCEPTION) under 2.4 and 3.3.

That is correct.

Under 2.4 and 3.3 it should show neither the line "EXCEPTION ## EXCEPTION" nor the following line "TypeError: ('__init__() takes at least 2 arguments....."

That means, that in version 2.4 and 3.3 that unpickling problem doesn't exist. In version 2.4 I tested it myself; exactly, there is no problem. In version 3.3 I simply take your word, that the bug is fixed.

A backport to 2.7 would be in the interest of the Zope community (I dare say ;)), at least in ours.

In our project, after having migrated to Zope 2.13/Python2.7 we found this bug and now we are quite worried what else might happen with our huge pickled database.

I don't have the expertise required to do the 2.7 backport.  My naive attempt is attached, but the message attribute is not preserved (test failure).  If someone can fix the patch, I'll commit it.

New changeset 2c9f7ed28384 by R David Murray in branch '3.2':
#17296: backport fix for issue 1692335, naive exception pickling.
http://hg.python.org/cpython/rev/2c9f7ed28384

Fixed the patch for 2.7

The latest 2.7 patch does not cause the following tests (from
issue 1692335) to pass:

 http://bugs.python.org/file17509/issue1692335-tests.patch

I think it works with the other test also now. Have also included the failed test mentioned by Tres in the patch since it wasn't present in the python2.7 test suite.

As this was fixed in 3.3 that means all actively maintained versions have the fix, and so closing as fixed.

stage: resolved

messages: + msg183982

messages: + msg182964