RFR (S) 8250844: Make sure {type, obj}ArrayOopDesc accessors check the bounds

Aleksey Shipilev shade at redhat.com
Fri Jul 31 09:26:39 UTC 2020


On 7/31/20 11:11 AM, Reingruber, Richard wrote:
> it does make sense to add the range checks and your patch looks good to me.
> // Not Reviewer though.

Thanks!

> Maybe bounds should be checked even at a lower level, namely in HeapAccess?

I briefly considered it yesterday. But AFAIU, HeapAccess is too low-level: it does not know about
arrays, only about oop + offset. So {type,obj}ArrayOopDesc seems as low as we can get.

-- 
-Aleksey



More information about the hotspot-runtime-dev mailing list