RFR: 8304292: Memory leak related to ClassLoader::update_class_path_entry_list [v3]

Calvin Cheung ccheung at openjdk.org
Thu Aug 10 00:07:28 UTC 2023


> 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.

Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:

  @iklam comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15132/files
  - new: https://git.openjdk.org/jdk/pull/15132/files/aa44f73a..86de8828

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15132&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15132&range=01-02

  Stats: 9 lines in 2 files changed: 4 ins; 1 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/15132.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15132/head:pull/15132

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


More information about the hotspot-runtime-dev mailing list