On Wed, 13 Jan 2021 08:19:59 GMT, Aleksey Shipilev <sh...@openjdk.org> wrote:

>> This change eliminates memory leaks in the JVMTI implementation reported by 
>> SonarCloud.
>> 
>> The leaks occur when the Java heap is exhausted.
>
> This looks good to me, thanks!

@shipilev thanks for doing the analysis and reporting the issues. I found 2 
leaks. Do you see more that could be related to JDK-8227745 
(https://github.com/openjdk/jdk/commit/40f847e2)?

I wanted to do a SonarCloud scan myself but was uncertain about the requested 
permissions, me using the service for work, etc. Would it be possible to 
publish the SC report?

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

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

Reply via email to