[kaffe] kaffe/kaffevm/readClass.c: Should class version 1.5 be supported?

Ito Kazumitsu kaz at maczuka.gcd.org
Mon Apr 2 07:21:07 PDT 2007


Hi,

I happened to find that, luckily, a class file of version 1.5
compiled by Sun's javac from source like this:

import java.util.ArrayList;
public class Test1 {
    @Deprecated
    public static void main(String[] args) {
        ArrayList<String> a = new ArrayList<String>();
        a.add("aaa");
        for (String s : a) System.out.println(s);
    }
}

can be run with Kaffe normally.

On the other hand, kaffe/kaffevm/readClass.c seems to be trying
to prevent class file version 1.5 from running.

But UnsupportedClassVersionError does not occur because of the
bug in kaffe/kaffevm/readClass.c of not returning NULL.

The question is whether UnsupportedClassVersionError should be
thrown in this case or the class file should be run normally.

I prefer the latter, and this is my proposed patch.

Index: kaffe/kaffe/kaffevm/readClass.c
===================================================================
RCS file: /cvs/kaffe/kaffe/kaffe/kaffevm/readClass.c,v
retrieving revision 1.28
diff -u -r1.28 readClass.c
--- kaffe/kaffe/kaffevm/readClass.c	17 Apr 2006 17:57:07 -0000	1.28
+++ kaffe/kaffe/kaffevm/readClass.c	2 Apr 2007 13:44:23 -0000
@@ -62,12 +62,14 @@
 	if (! ((major_version == MAJOR_VERSION_V1_1 && minor_version == MINOR_VERSION_V1_1) ||
 	       (major_version == MAJOR_VERSION_V1_2 && minor_version == MINOR_VERSION_V1_2) ||
 	       (major_version == MAJOR_VERSION_V1_3 && minor_version == MINOR_VERSION_V1_3) ||
-	       (major_version == MAJOR_VERSION_V1_4 && minor_version == MINOR_VERSION_V1_4))) {
+	       (major_version == MAJOR_VERSION_V1_4 && minor_version == MINOR_VERSION_V1_4) ||
+	       (major_version == MAJOR_VERSION_V1_5 && minor_version == MINOR_VERSION_V1_5))) {
 		postExceptionMessage(einfo,
 				     JAVA_LANG(UnsupportedClassVersionError),
 				     "%d.%d",
 				     major_version,
 				     minor_version);
+		return NULL;
 	}
 
 	if (readConstantPool(classThis, fp, einfo) == false) {




More information about the kaffe mailing list