fixed bugs in generic_setattr (returned NULL instead of

  -1 for errors)

diff --git a/Modules/flmodule.c b/Modules/flmodule.c
index 2d26533f1f09a9927750ca6ef0074bc98f0ba108..d8c423c820f2204548e51b6301d427dafc576734 100644 (file)
--- a/Modules/flmodule.c
+++ b/Modules/flmodule.c
@@ -325,7 +325,7 @@ generic_setattr(g, name, v)
 
        if (v == NULL) {
                err_setstr(TypeError, "can't delete forms object attributes");
-               return NULL;
+               return -1;
        }
 
        /* "label" is an exception: setmember doesn't set strings;
@@ -333,7 +333,7 @@ generic_setattr(g, name, v)
        if (strcmp(name, "label") == 0) {
                if (!is_stringobject(v)) {
                        err_setstr(TypeError, "label attr must be string");
-                       return NULL;
+                       return -1;
                }
                fl_set_object_label(g->ob_generic, getstringvalue(v));
                return 0;