Re: [jdk19] RFR: 8289148: j.l.foreign.VaList::nextVarg call could throw IndexOutOfBoundsException or even crash the VM [v2]

2022-07-05 Thread Jorn Vernee
On Tue, 5 Jul 2022 15:14:20 GMT, Jorn Vernee wrote: >> This seems a bit too much. >> >> The class javadoc further up already describes a va list as "a stateful >> cursor used to iterate over a set of arguments". If that description is >> insufficient, I think it should be amended at that point

Re: [jdk19] RFR: 8289148: j.l.foreign.VaList::nextVarg call could throw IndexOutOfBoundsException or even crash the VM [v2]

2022-07-05 Thread Jorn Vernee
> This patch changes all VaList implementations to throw > `NoSuchElementException` when out of bounds reads occur on a VaList that is > created using the Java builder API. The docs are updated accordingly. > > For VaLists that are created from native addresses, we don't know their > bounds, so