--- old/src/java.instrument/share/native/libinstrument/JPLISAgent.c 2016-09-20 19:24:04.042759478 -0700 +++ new/src/java.instrument/share/native/libinstrument/JPLISAgent.c 2016-09-20 19:24:03.938759172 -0700 @@ -790,9 +790,10 @@ pkg_name_buf[len] = '\0'; err = (*jvmti)->GetNamedModule(jvmti, loaderObject, pkg_name_buf, &moduleObject); + free((void*)pkg_name_buf); + check_phase_ret_blob(err, NULL); jplis_assert_msg(err == JVMTI_ERROR_NONE, "error in the JVMTI GetNamedModule"); - free((void*)pkg_name_buf); return moduleObject; } --- old/test/ProblemList.txt 2016-09-20 19:24:04.342760362 -0700 +++ new/test/ProblemList.txt 2016-09-20 19:24:04.237760053 -0700 @@ -134,8 +134,6 @@ java/lang/instrument/BootClassPath/BootClassPathTest.sh 8072130 macosx-all -java/lang/instrument/DaemonThread/TestDaemonThread.java 8161225 generic-all - java/lang/management/MemoryMXBean/Pending.java 8158837 generic-all java/lang/management/MemoryMXBean/PendingAllGC.sh 8158760 generic-all --- old/test/java/lang/instrument/DaemonThread/TestDaemonThreadLauncher.java 2016-09-20 19:24:04.635761225 -0700 +++ new/test/java/lang/instrument/DaemonThread/TestDaemonThreadLauncher.java 2016-09-20 19:24:04.528760910 -0700 @@ -1,6 +1,6 @@ /* * Copyright 2014 Goldman Sachs. - * Copyright (c) 2014, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2014, 2016, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -25,6 +25,7 @@ import jdk.testlibrary.OutputAnalyzer; import jdk.testlibrary.ProcessTools; +import static jdk.testlibrary.Asserts.*; public class TestDaemonThreadLauncher { public static void main(String args[]) throws Exception { @@ -32,6 +33,7 @@ ProcessBuilder pb = ProcessTools.createJavaProcessBuilder("-javaagent:DummyAgent.jar", "TestDaemonThread", "."); OutputAnalyzer analyzer = ProcessTools.executeProcess(pb); analyzer.shouldNotContain("ASSERTION FAILED"); + assertTrue(analyzer.getExitValue() == 0); } } }