RFR: 8248876: LoadObject with bad base address created for exec file on linux [v2]

[Yasumasa Suenaga]

On Tue, 9 Feb 2021 09:32:34 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>>> Oops, I didn't mean to suggest the comment should go entirely. It's actually very useful to say something here, as the way we use the dynamic field, setting it then fixing it up.
>>> Is it possible to re-add the comment in a new change?
>> 
>> I thought your thumbs-up emoji means to agree to remove all comments.
>> 
>> As you know, I've integrated this PR, so we need to file as a new ticket.
>> I will do it and send a PR with following patch if you are ok.
>> 
>> diff --git a/src/jdk.hotspot.agent/linux/native/libsaproc/ps_core.c b/src/jdk.hotspot.agent/linux/native/libsaproc/ps_core.c
>> index dff4cb3eeca..9f7c4050ef2 100644
>> --- a/src/jdk.hotspot.agent/linux/native/libsaproc/ps_core.c
>> +++ b/src/jdk.hotspot.agent/linux/native/libsaproc/ps_core.c
>> @@ -507,7 +507,11 @@ static uintptr_t read_exec_segments(struct ps_prochandle* ph, ELF_EHDR* exec_ehd
>>          result = exec_php->p_vaddr;
>>          ph->core->dynamic_addr = exec_php->p_vaddr;
>>        } else { // ET_DYN
>> +        // Base address of executable is based on entry point (AT_ENTRY).
>>          result = ph->core->dynamic_addr - exec_ehdr->e_entry;
>> +
>> +        // dynamic_addr has entry point of executable.
>> +        // Thus we should subtract it.
>>          ph->core->dynamic_addr += exec_php->p_vaddr - exec_ehdr->e_entry;
>>        }
>>        print_debug("address of _DYNAMIC is 0x%lx\n", ph->core->dynamic_addr);
>
> Thanks!

@kevinjwalls I've sent PR about adding comments in #2478. I hope you review it!

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

PR: https://git.openjdk.java.net/jdk/pull/2366