Fix accidentally reversed NULL test in load_mark(). Suggested by

Tamito Kajiyama.  (This caused a bug only on platforms where malloc(0)
returns NULL.)

diff --git a/Modules/cPickle.c b/Modules/cPickle.c
index 614ff06c25952d1fffbaafde9f5a017411a397c5..a661c660b048c02d9b07e2184487f1adcf86f4a7 100644 (file)
--- a/Modules/cPickle.c
+++ b/Modules/cPickle.c
@@ -3295,7 +3295,7 @@ load_mark(Unpicklerobject *self) {
     if ((self->num_marks + 1) >= self->marks_size) {
         s=self->marks_size+20;
         if (s <= self->num_marks) s=self->num_marks + 1;
-        if (self->marks)
+        if (self->marks == NULL)
             self->marks=(int *)malloc(s * sizeof(int));
         else
             self->marks=(int *)realloc(self->marks, s * sizeof(int));