RFR: JDK-8303181: Memory leak in ClassLoaderExt::setup_app_search_path [v2]

[David Holmes]

On Wed, 1 Mar 2023 16:04:50 GMT, Justin King <jcking at openjdk.org> wrote:

>> Fix memory leak by removing unnecessary `os::strdup` call. The string is passed as a const pointer to all functions.
>
> Justin King has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Switch to strdup_check_oom with free
>    
>    Signed-off-by: Justin King <jcking at google.com>
>  - Merge remote-tracking branch 'upstream/master' into classloaderext-leak
>  - Remove unnecessary strdup
>    
>    Signed-off-by: Justin King <jcking at google.com>

Looks good. Thanks.

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

Marked as reviewed by dholmes (Reviewer).

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