* parse.y (assignable_gen): when "self = 1" was evalueted, unnecessary

  error message was output, which might cause null pointer access.


git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@16484 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 3 insertions, 1 deletions

diff --git a/parse.y b/parse.y
index f8b96fbdc..31694a371 100644
--- a/parse.y
+++ b/parse.y
@@ -7752,7 +7752,9 @@ assignable_gen(struct parser_params *parser, ID id, NODE *val)
     else if (is_class_id(id)) {
 	return NEW_CVASGN(id, val);
     }
-    compile_error(PARSER_ARG "identifier %s is not valid to set", rb_id2name(id));
+    else {
+	compile_error(PARSER_ARG "identifier %s is not valid to set", rb_id2name(id));
+    }
     return 0;
 }