diff --git a/libjava/ChangeLog b/libjava/ChangeLog index ef14272f47935cb3d0d15c3d2254cb5cdcad9e92..4823eccaa2be5153ad64ba0c54159989bc997a42 100644 --- a/libjava/ChangeLog +++ b/libjava/ChangeLog @@ -1,3 +1,9 @@ +2002-02-06 Tom Tromey <tromey@redhat.com> + + * verify.cc (type::isnull): New method. + (require_array_type): Handle case where array is null. + (verify_instructions_0) [op_arraylength]: Likewise. + 2002-02-05 Bryce McKinlay <bryce@waitaki.otago.ac.nz> * configure.in: Set up PLATFORMOBJS not PLATFORM_SPECIFIC_SOURCES. diff --git a/libjava/verify.cc b/libjava/verify.cc index c9cfb885fdf84f595ad219bd15c30102aab840be..46cee1533ce690a40655f65f5801a70324985f14 100644 --- a/libjava/verify.cc +++ b/libjava/verify.cc @@ -522,6 +522,11 @@ private: return false; } + bool isnull () const + { + return key == null_type; + } + bool isinterface (_Jv_BytecodeVerifier *verifier) { resolve (verifier); @@ -1161,6 +1166,10 @@ private: // compatible with type ELEMENT. Returns the actual element type. type require_array_type (type array, type element) { + // An odd case. Here we just pretend that everything went ok. + if (array.isnull ()) + return element; + if (! array.isarray ()) verify_fail ("array required"); @@ -2796,7 +2805,7 @@ private: case op_arraylength: { type t = pop_type (reference_type); - if (! t.isarray ()) + if (! t.isarray () && ! t.isnull ()) verify_fail ("array type expected"); push_type (int_type); }