Fix JRUBY-6865

headius · dekellum · commit 254d8e5e2482 · 2012-09-04T11:21:46.000-07:00
JRuby allows invalid attr_accessor (MRI raises exception) Our test for local variable names (which applies in both JRuby and MRI to attr names) did not check for trailing ?. I added that one case. Note that in MRI all of this passes through ID parsing, which has logic for checking the type of ID I did not attempt to sort out. There are other cases we probably let through that MRI does not. Test submitted to MRI via ruby/ruby#171
diff --git a/src/org/jruby/RubyModule.java b/src/org/jruby/RubyModule.java
@@ -1307,6 +1307,11 @@ private void addAccessor(ThreadContext context, String internedName, Visibility
             visibility = PRIVATE;
             // FIXME warning
         }
+
+        if (!(IdUtil.isLocal(internedName) || IdUtil.isConstant(internedName))) {
+            throw runtime.newNameError("invalid attribute name", internedName);
+        }
+
         final String variableName = ("@" + internedName).intern();
         if (readable) {
             addMethod(internedName, new JavaMethodZero(this, visibility, CallConfiguration.FrameNoneScopeNone) {
diff --git a/src/org/jruby/util/IdUtil.java b/src/org/jruby/util/IdUtil.java
@@ -57,12 +57,16 @@ public static boolean isInstanceVariable(String id) {
     public static boolean isGlobal(String id) {
         return id.length()>0 && id.charAt(0) == '$';
     }
+
+    public static boolean isPredicate(String id) {
+        return id.endsWith("?");
+    }
     
     /**
      * rb_is_local_id and is_local_id
      */    
 	public static boolean isLocal(String id) {
-	    return !isGlobal(id) && !isClassVariable(id) && !isInstanceVariable(id) && !isConstant(id);
+	    return !isGlobal(id) && !isClassVariable(id) && !isInstanceVariable(id) && !isConstant(id) && !isPredicate(id);
     }
 
 	public static boolean isAttrSet(String id) {
