Re: RFR: 8208157: requires.VMProps throws NPE for missing properties in "release" file

[Erik Joelsson]

Looks good.

/Erik


On 2018-07-24 15:48, Alexandre (Shura) Iline wrote:

Hi,

Could you please tale a quick look on this simple fix?

diff --git a/test/jtreg-ext/requires/VMProps.java 
b/test/jtreg-ext/requires/VMProps.java
--- a/test/jtreg-ext/requires/VMProps.java
+++ b/test/jtreg-ext/requires/VMProps.java
@@ -432,7 +432,8 @@
                  System.getProperty("java.home") + "/release"))) {
              Properties properties = new Properties();
              properties.load(in);
-            return properties.getProperty("IMPLEMENTOR").replace("\"", "");
+            String implementorProperty = properties.getProperty("IMPLEMENTOR");
+            return (implementorProperty == null) ? "null" : 
implementorProperty.replace("\"", "");
          } catch (IOException e) {
              e.printStackTrace();
          }


Thank you.

Shura