8266187: Memory leak in appendBootClassPath()

Wang Huang · Sun Jianye · kevinjwalls · commit aa90df6f5194 · 2021-05-04T10:02:22.000Z
Co-authored-by: Wang Huang &lt;whuang@openjdk.org&gt;
Co-authored-by: Sun Jianye &lt;sunjianye@huawei.com&gt;
Reviewed-by: kevinw, mli, alanb
diff --git a/src/java.instrument/share/native/libinstrument/InvocationAdapter.c b/src/java.instrument/share/native/libinstrument/InvocationAdapter.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2003, 2019, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2021, 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
@@ -949,6 +949,7 @@ appendBootClassPath( JPLISAgent* agent,
 
             resolved = resolve(parent, path);
             jvmtierr = (*jvmtienv)->AddToBootstrapClassLoaderSearch(jvmtienv, resolved);
+            free(resolved);
         }
 
         /* print warning if boot class path not updated */

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright (c) 2003, 2019, Oracle and/or its affiliates. All rights reserved.`
	`2`	`+ * Copyright (c) 2003, 2021, Oracle and/or its affiliates. All rights reserved.`
`3`	`3`	`* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.`
`4`	`4`	`*`
`5`	`5`	`* This code is free software; you can redistribute it and/or modify it`
`@@ -949,6 +949,7 @@ appendBootClassPath( JPLISAgent* agent,`
`949`	`949`
`950`	`950`	`resolved = resolve(parent, path);`
`951`	`951`	`jvmtierr = (*jvmtienv)->AddToBootstrapClassLoaderSearch(jvmtienv, resolved);`
	`952`	`+ free(resolved);`
`952`	`953`	`}`
`953`	`954`
`954`	`955`	`/* print warning if boot class path not updated */`