RFR: 8311301: MethodExitTest may fail with stack buffer overrun [v2]

[Daniel Jeliński]

On Wed, 5 Jul 2023 06:40:11 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

>> Please review this test-only fix that fixes the size of variables that are used in methods that expect a pointer.
>> 
>> On Windows, type `long` is 32 bits, pointers are 64 bits large. The method `GetThreadLocalStorage` writes a pointer (64 bits) to the address given by its parameter, which overflows a `long`. The code generated by VS compiler ignores this, but the code generated by clang crashes the test.
>> 
>> No new tests. MethodExitTest continues to pass on supported platforms, and passes on clang+win with this fix.
>
> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use void* instead

Thanks for the reviews!

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

PR Comment: https://git.openjdk.org/jdk/pull/14770#issuecomment-1623247781