[jdk17u-dev] RFR: 8275320: NMT should perform buffer overrun checks

[Christoph Langer]

On Wed, 2 Nov 2022 08:35:44 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Not a clean downport:
>> 
>> - testutils.cpp/.hpp does not exist yet. I took this over (it is just a bunch of test helpers for gtests)
>> - os.cpp, in os::malloc, the size overflow check added with the downport of JDK-8286519 needed to be reformulated.
>> 
>> I also added a small, JDK17 only gtest to test for size argument overflow. Ideally that should have been done with JDK-8286519. It is release only since in debug we will get an assert which is fine but I did not feel like adding a death test for it. 
>> 
>> Note that in head, I am in the process of fixing https://bugs.openjdk.org/browse/JDK-8295865, and that will introduce much more thorough regression tests. Should we ever downport JDK-8295865 to 17, the test introduced here can be removed.
>
> Ping.

@tstuefe would you mind moving this from draft to non-draft?

-------------

PR: https://git.openjdk.org/jdk17u-dev/pull/842