On Wed, 13 Jan 2021 08:19:59 GMT, Aleksey Shipilev <[email protected]> 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 > ([40f847e](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? I think those are only two instances. I am only aware of [this one](https://sonarcloud.io/project/issues?fileUuids=AXaE0amt8L9hkQskFrwX&id=jdk&resolved=false&types=BUG). I meant to set up my own some time later. ------------- PR: https://git.openjdk.java.net/jdk/pull/2055
