diff --git a/libjava/ChangeLog b/libjava/ChangeLog index aeb049138ae65cd81f5062ef1ed67b9515e92ebf..ff3c72fa08c59f4e97d386336f6822ff36fc1693 100644 --- a/libjava/ChangeLog +++ b/libjava/ChangeLog @@ -1,3 +1,22 @@ +2002-07-24 Mark Wielaard <mark@klomp.org> + + * java/lang/reflect/natField.cc (setAddr): Check isAccessible(). + * java/io/ObjectInputStream.java (setBooleanField): Before setting + field call setAccessible(true). + (setByteField): Likewise. + (setCharField): Likewise. + (setDoubleField): Likewise. + (setFloatField): Likewise. + (setIntField): Likewise. + (setLongField): Likewise. + (setShortField): Likewise. + (setObjectField): Likewise. + +2002-07-24 Tom Tromey <tromey@redhat.com> + + * java/io/ObjectInputStream.java (readObject) [TC_ARRAY]: Don't + use toString() to format array element. + 2002-07-23 Mark Wielaard <mark@klomp.org> * gnu/java/security/provider/MD5.java: Extends MessageDigest, not diff --git a/libjava/java/io/ObjectInputStream.java b/libjava/java/io/ObjectInputStream.java index 140725a8fb676ef2f0fd9bcd62f8b1004ff02e3a..e1e16f28498d49fe866d2d8ac0d8d0d1c1d8de31 100644 --- a/libjava/java/io/ObjectInputStream.java +++ b/libjava/java/io/ObjectInputStream.java @@ -244,7 +244,7 @@ public class ObjectInputStream extends InputStream int handle = assignNewHandle (array); readArrayElements (array, componentType); for (int i=0, len=Array.getLength(array); i < len; i++) - dumpElementln (" ELEMENT[" + i + "]=" + Array.get(array, i).toString()); + dumpElementln (" ELEMENT[" + i + "]=" + Array.get(array, i)); ret_val = processResolution (array, handle); break; } @@ -1401,6 +1401,7 @@ public class ObjectInputStream extends InputStream { Class klass = obj.getClass (); Field f = getField (klass, field_name); + f.setAccessible(true); f.setBoolean (obj, val); } catch (Exception _) @@ -1415,6 +1416,7 @@ public class ObjectInputStream extends InputStream { Class klass = obj.getClass (); Field f = getField (klass, field_name); + f.setAccessible(true); f.setByte (obj, val); } catch (Exception _) @@ -1429,6 +1431,7 @@ public class ObjectInputStream extends InputStream { Class klass = obj.getClass (); Field f = getField (klass, field_name); + f.setAccessible(true); f.setChar (obj, val); } catch (Exception _) @@ -1443,6 +1446,7 @@ public class ObjectInputStream extends InputStream { Class klass = obj.getClass (); Field f = getField (klass, field_name); + f.setAccessible(true); f.setDouble (obj, val); } catch (Exception _) @@ -1457,6 +1461,7 @@ public class ObjectInputStream extends InputStream { Class klass = obj.getClass (); Field f = getField (klass, field_name); + f.setAccessible(true); f.setFloat (obj, val); } catch (Exception _) @@ -1471,6 +1476,7 @@ public class ObjectInputStream extends InputStream { Class klass = obj.getClass (); Field f = getField (klass, field_name); + f.setAccessible(true); f.setInt (obj, val); } catch (Exception _) @@ -1486,6 +1492,7 @@ public class ObjectInputStream extends InputStream { Class klass = obj.getClass (); Field f = getField (klass, field_name); + f.setAccessible(true); f.setLong (obj, val); } catch (Exception _) @@ -1501,6 +1508,7 @@ public class ObjectInputStream extends InputStream { Class klass = obj.getClass (); Field f = getField (klass, field_name); + f.setAccessible(true); f.setShort (obj, val); } catch (Exception _) @@ -1516,6 +1524,7 @@ public class ObjectInputStream extends InputStream { Class klass = obj.getClass (); Field f = getField (klass, field_name); + f.setAccessible(true); // FIXME: We should check the type_code here f.set (obj, val); } diff --git a/libjava/java/lang/reflect/natField.cc b/libjava/java/lang/reflect/natField.cc index b0ae48d3383c0786e21110598893273582b79d22..b10b5f5a65aca7242a5e277b20e0e1a00c891384 100644 --- a/libjava/java/lang/reflect/natField.cc +++ b/libjava/java/lang/reflect/natField.cc @@ -257,7 +257,8 @@ static void* setAddr (java::lang::reflect::Field* field, jclass caller, jobject obj) { void *addr = getAddr(field, caller, obj); - if (field->getModifiers() & java::lang::reflect::Modifier::FINAL) + if (!field->isAccessible() + && field->getModifiers() & java::lang::reflect::Modifier::FINAL) throw new java::lang::IllegalAccessException(); return addr; }