Re: [Sablevm-developer] Crash on System.arraycopy()

[David <db...@cs...>]

On Sun, Mar 14, 2004 at 09:51:04PM -0500, Etienne Gagnon wrote:
> Hi David,
>=20
> I don't know if we have yet implemented the simple unsigned comparison
> for array bounds check.
>=20

Hi Etienne,

The interpreter uses an unsigned comparisons, at least at some places.
Never checked if everywhere.

> I think that, again, unsigned comparison would save the day, here.
>=20
> (_svmt_u32) src_position + (_svmt_u32) length > (_svmt_u32)=20
> src_instance->size
>=20

Yes, this works.  I checked in this solution to the bug fix in staging.


David

---

David B=E9langer
Graduate Student
School of Computer Science
McGill University
Office: MC226

Web page:   http://www.cs.mcgill.ca/~dbelan2/
Public key: http://www.cs.mcgill.ca/~dbelan2/public_key.txt