Integrated: 8348567: [ASAN] Memory access partially overflows by NativeCallStack

[SendaoYan]

On Fri, 24 Jan 2025 09:53:43 GMT, SendaoYan <syan at openjdk.org> wrote:

> Hi all,
> This PR fix a undefined behaviour in 'CollisionsReceiveDifferentIndexes' testcase 
> locate in 'test/hotspot/gtest/nmt/test_nmt_nativecallstackstorage.cpp' file when call `NativeCallStack::NativeCallStack` function. Before this PR, 'CollisionsReceiveDifferentIndexes' test allocate 4 type variable `i` and then pass the address `&i`, but in 'NativeCallStack' function read the pointer as 8 types, so the AddressSanitizer report "Memory access partially overflows variable i". This PR change the variable `i` to `size_t` to avoid this undefined behaviour.
> Change has been verified locally, test-fix only, no risk.
> 
> Below code snippet can reproduce the same issue:
> 
> 
> #include <stdio.h>
> typedef unsigned char u_char;
> typedef u_char*       address;
> int main()
> {
>   int i = 0;
>   address* pc = (address*)(&i);
>   address _stack = pc[0];
>   printf("_stack = %p\n", _stack);
>   return 0;
> }
> 
> 
> Reproduce command:
> 
> clang -g3 -Wall -Wextra -fsanitize=address -fsanitize=leak -fsanitize=undefined partially-overflows.c && ./a.out

This pull request has now been integrated.

Changeset: bb528d5a
Author:    SendaoYan <syan at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/bb528d5ad6f28e9c1bc4e283d40c70489465bd00
Stats:     2 lines in 1 file changed: 0 ins; 0 del; 2 mod

8348567: [ASAN] Memory access partially overflows by NativeCallStack

Reviewed-by: jsjolen, dholmes

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

PR: https://git.openjdk.org/jdk/pull/23294