Steve Dower added the comment:

The fix for this now uses module without initializing it, which could lead to a 
crash if get_dotted_path() tries to raise an exception (it puts module into the 
error string with %R). See Modules/_pickle.c#l1656 and Modules/_pickle.c#l1538.

I think the fix is to initialize module with Py_None and currently there's no 
need to bump the refcount (though I'm always wary of doing that in case things 
change in the future). If that sounds right I'm happy to put the fix in, else 
Antoine can do it :)

----------
nosy: +steve.dower

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue22676>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to