On Fri, 30 Oct 2020 20:30:49 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:

> The debug agent needs to deallocate memory that is allocated by calls to 
> GetAllThreads. There were two places were this was not being done, resulting 
> in a memory leak.

This looks good.
Nit: semicolon sign can be removed:
     err: ;
+        jvmtiDeallocate(threads);


Thanks,
Serguei

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

Marked as reviewed by sspitsyn (Reviewer).

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

Reply via email to