Thanks Vitaly.  Best to cleanup.  I'll make that change.

-neil

On 6/19/2014 5:39 PM, Vitaly Davidovich wrote:

Hi Neil,

1054                     envName = malloc(pbuflen);

envName is never freed looks like. Probably not a big deal since this runs only if launcher is traced, but thought I'd mention it.

Sent from my phone

On Jun 19, 2014 8:30 PM, "Neil Toda" <neil.t...@oracle.com <mailto:neil.t...@oracle.com>> wrote:


    Launcher support for modified Native Memory Tracking mechanism in
    JVM in JDK9.

    Webrev  : http://cr.openjdk.java.net/~ntoda/8042469/webrev-03/
    <http://cr.openjdk.java.net/%7Entoda/8042469/webrev-03/>
    bug         : https://bugs.openjdk.java.net/browse/JDK-8042469
    CCC         : http://ccc.us.oracle.com/8042469

    Thanks.

    -neil


Reply via email to