From df71fc45a77f8d30e52366b6670d2d0a407a290e Mon Sep 17 00:00:00 2001 From: Tao Liu Date: Mon, 28 Oct 2024 14:07:14 -0400 Subject: [PATCH] Error message update when profile name without version is not exist --- .../internal/security/RestrictedSecurity.java | 26 ++++++++++++------- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/closed/src/java.base/share/classes/openj9/internal/security/RestrictedSecurity.java b/closed/src/java.base/share/classes/openj9/internal/security/RestrictedSecurity.java index be0d6a5790d..59d971d9f3b 100644 --- a/closed/src/java.base/share/classes/openj9/internal/security/RestrictedSecurity.java +++ b/closed/src/java.base/share/classes/openj9/internal/security/RestrictedSecurity.java @@ -344,23 +344,29 @@ private static void getProfileID(Properties props) { + selectedProfile); } String defaultMatch = null; + Boolean profileExist = false; for (Object keyObject : props.keySet()) { if (keyObject instanceof String key) { - if (key.startsWith(potentialProfileID) && key.endsWith(".desc.default")) { - // Check if property is set to true. - if (Boolean.parseBoolean(props.getProperty(key))) { - // Check if multiple defaults exist and act accordingly. - if (defaultMatch == null) { - defaultMatch = key.split("\\.desc")[0]; - } else { - printStackTraceAndExit("Multiple default RestrictedSecurity" - + " profiles for " + selectedProfile); + if (key.startsWith(potentialProfileID)) { + profileExist = true; + if (key.endsWith(".desc.default")) { + // Check if property is set to true. + if (Boolean.parseBoolean(props.getProperty(key))) { + // Check if multiple defaults exist and act accordingly. + if (defaultMatch == null) { + defaultMatch = key.split("\\.desc")[0]; + } else { + printStackTraceAndExit("Multiple default RestrictedSecurity" + + " profiles for " + selectedProfile); + } } } } } } - if (defaultMatch == null) { + if (!profileExist) { + printStackTraceAndExit(selectedProfile + " is not present in the java.security file."); + } else if (defaultMatch == null) { printStackTraceAndExit("No default RestrictedSecurity profile was found for " + selectedProfile); } else {