From: Guido van Rossum <guido@python.org>
Date: Thu, 7 Sep 2000 00:11:40 +0000 (+0000)
Subject: Simple fix from Jin Fulton to avoid returning a half-initialized
X-Git-Tag: v2.0b2~443
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c84d8bd7be6ea7ce21973da06e821e01604dda4f;p=python

Simple fix from Jin Fulton to avoid returning a half-initialized
module when e.g. copy_reg.py doesn't exist.  This caused a core dump.

This closes SF bug 112944.
---

diff --git a/Modules/cPickle.c b/Modules/cPickle.c
index b64b1f1377..7161c2e413 100644
--- a/Modules/cPickle.c
+++ b/Modules/cPickle.c
@@ -4521,6 +4521,8 @@ initcPickle(void) {
     PyObject *format_version;
     PyObject *compatible_formats;
 
+    if (init_stuff(m, d) < 0) return;
+
     Picklertype.ob_type = &PyType_Type;
     Unpicklertype.ob_type = &PyType_Type;
     PdataType.ob_type = &PyType_Type;
@@ -4542,6 +4544,4 @@ initcPickle(void) {
     PyDict_SetItemString(d, "compatible_formats", compatible_formats);
     Py_XDECREF(format_version);
     Py_XDECREF(compatible_formats);
-
-    init_stuff(m, d);
 }