Return -1 for errors.

diff --git a/Objects/moduleobject.c b/Objects/moduleobject.c
index cf96a70915027a28c8b8a5f5a0e9d088fe701cd8..3d2cab2d21019b0f3d022cfa89269e9203bd10cb 100644 (file)
--- a/Objects/moduleobject.c
+++ b/Objects/moduleobject.c
@@ -107,7 +107,7 @@ module_setattr(m, name, v)
 {
        if (strcmp(name, "__dict__") == 0 || strcmp(name, "__name__") == 0) {
                err_setstr(NameError, "can't assign to reserved member name");
-               return NULL;
+               return -1;
        }
        if (v == NULL)
                return dictremove(m->md_dict, name);

diff --git a/Objects/object.c b/Objects/object.c
index f2b4d1f22e846e98ce7bbea65f6fa1ed74553962..4170267d7cb8db19d82944f6469ba03a0b160709 100644 (file)
--- a/Objects/object.c
+++ b/Objects/object.c
@@ -157,7 +157,7 @@ setattr(v, name, w)
                        err_setstr(TypeError, "attribute-less object");
                else
                        err_setstr(TypeError, "object has read-only attributes");
-               return NULL;
+               return -1;
        }
        else {
                return (*v->ob_type->tp_setattr)(v, name, w);

diff --git a/Python/structmember.c b/Python/structmember.c
index 42c563e9294a50ac2ef163e6de040b8ad85859c0..b5f9bc3efb12636eda417e97061af7f7aaeb2796 100644 (file)
--- a/Python/structmember.c
+++ b/Python/structmember.c
@@ -130,5 +130,5 @@ setmember(addr, mlist, name, v)
        }
        
        err_setstr(NameError, name);
-       return NULL;
+       return -1;
 }