Integrated: 8304292: Memory leak related to ClassLoader::update_class_path_entry_list

Calvin Cheung ccheung at openjdk.org
Thu Aug 10 20:23:31 UTC 2023


On Thu, 3 Aug 2023 06:14:37 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

> Please review this small fix for `ClassPathEntry` leak when there's a duplicate entry in the app class path (`-cp`) and the `check_for_duplicate` argument is set to `true`, the `ClassLoader::add_to_app_classpath_entries` function should delete the entry before returning. This fix also corrects the call to `update_class_path_entry_list` from `setup_app_search_path`; the `check_for_duplicate` argument should be set to `true`. Two test cases are included to exercise the code path.
> 
> Passed tiers 1 - 4 testing.

This pull request has now been integrated.

Changeset: 88b4e3b8
Author:    Calvin Cheung <ccheung at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/88b4e3b8539c2beb29ad92bd74b300002c2ef84b
Stats:     75 lines in 5 files changed: 63 ins; 5 del; 7 mod

8304292: Memory leak related to ClassLoader::update_class_path_entry_list

Reviewed-by: dholmes, iklam

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

PR: https://git.openjdk.org/jdk/pull/15132


More information about the hotspot-runtime-dev mailing list