8303181: Memory leak in ClassLoaderExt::setup_app_search_path

Reviewed-by: ccheung, dholmes

Author: jcking

src/hotspot/share/classfile/classLoaderExt.cpp

@@ -67,7 +67,7 @@ void ClassLoaderExt::append_boot_classpath(ClassPathEntry* new_entry) {
 void ClassLoaderExt::setup_app_search_path(JavaThread* current) {
   Arguments::assert_is_dumping_archive();
   _app_class_paths_start_index = ClassLoader::num_boot_classpath_entries();
-  char* app_class_path = os::strdup(Arguments::get_appclasspath());
+  char* app_class_path = os::strdup_check_oom(Arguments::get_appclasspath(), mtClass);
 
   if (strcmp(app_class_path, ".") == 0) {
     // This doesn't make any sense, even for AppCDS, so let's skip it. We
@@ -78,6 +78,8 @@ void ClassLoaderExt::setup_app_search_path(JavaThread* current) {
     trace_class_path("app loader class path=", app_class_path);
     ClassLoader::setup_app_search_path(current, app_class_path);
   }
+
+  os::free(app_class_path);
 }
 
 void ClassLoaderExt::process_module_table(JavaThread* current, ModuleEntryTable* met) {