On Wed, 25 Mar 2026 09:51:23 GMT, Matthias Baesken <[email protected]> wrote:

>> Currently we have multiple dladdr - implementations on AIX in the codebase.
>> One of those is in hotspot, one in libawt.
>> We should unify those if possible.
>
> Matthias Baesken has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Small suggestions by David

src/hotspot/os/aix/porting_aix.cpp line 330:

> 328:   info->dli_fname = ZEROSTRING;
> 329:   info->dli_sname = ZEROSTRING;
> 330:   info->dli_saddr = nullptr;

There seems to be a difference in how this struct is initialized here and in 
awt: "ZEROSTRING" vs 0:
>memset((void *)info, 0, sizeof(Dl_info));

I assume this is not an unexpected change for the code that uses Dl_info, but 
it would be good to confirm.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30295#discussion_r2998615539

Reply via email to