RFR: 8252997: Null-proofing for linker_md.c
Daniel D. Daugherty
daniel.daugherty at oracle.com
Fri Sep 18 21:17:03 UTC 2020
On 9/18/20 5:13 PM, Chris Plummer wrote:
> On 9/18/20 1:59 PM, Serguei Spitsyn wrote:
>> On Fri, 18 Sep 2020 20:23:15 GMT, Chris Plummer
>> <cjplummer at openjdk.org> wrote:
>>
>>>> In the platform-specific implementations of linker_md.c, we see the
>>>> dll_build_name methods begin with a call to
>>>> jvmtiAllocate.
>>>> We then appear to rush ahead and try to use that variable without
>>>> checking for a null.
>>>>
>>>> I propose moving the null check to the point *before* we try to use
>>>> that variable, to avoid any possible SEGV.
>>>>
>>>> Bug link: https://bugs.openjdk.java.net/browse/JDK-8252997
>>> Both files need copyright updates before pushing. Otherwise the
>>> changes look good.
>> The fix looks okay to me.
>> But I have a dejavu that I've already reviewed it.
>>
>> -------------
>>
>> PR: https://git.openjdk.java.net/jdk/pull/246
> Serguei, it was initially was sent out as a non-GIT review even though
> we had transitioned to GIT. Dan asked that it be moved to GIT. So like
> me, you probably looked at the changes but didn't send out an official
> "reviewed" email since it still needed to be moved to GIT.
>
> Chris
Severin asked that it be moved to GitHub.
I noted that the Serviceability team needed to on the review.
Dan
More information about the serviceability-dev
mailing list