RFR: 7903336: java.lang.NullPointerException: at com.sun.javatest.TestSuite$GeneralPurposeLogger.<init>(TestSuite.java:1494)

[Leonid Kuskov]

On Fri, 30 Sep 2022 00:01:15 GMT, Dmitry Bessonov <dbessono at openjdk.org> wrote:

> 7903336: java.lang.NullPointerException: at com.sun.javatest.TestSuite$GeneralPurposeLogger.<init>(TestSuite.java:1494)
> 
> Noticed when running a test suite:
> 
> java.lang.NullPointerException
> at java.base/java.util.Objects.requireNonNull(Objects.java:233)
> at java.logging/java.util.logging.Logger.addHandler(Logger.java:2064)
> at com.sun.javatest.TestSuite$GeneralPurposeLogger.<init>(TestSuite.java:1494)
> at com.sun.javatest.TestSuite$GeneralPurposeLogger.<init>(TestSuite.java:1481)
> at com.sun.javatest.TestSuite.createLog(TestSuite.java:1303)
> at com.sun.javatest.services.Service.createLog(Service.java:325)
> at com.sun.javatest.services.ServiceManager.setParameters(ServiceManager.java:133)
> at com.sun.javatest.services.ServiceManager.setHarness(ServiceManager.java:126)
> at com.sun.javatest.TestSuite.starting(TestSuite.java:546)
> at com.sun.jck.lib.JCKTestSuite.starting(JCKTestSuite.java:268)
> at com.sun.javatest.Harness.runTests(Harness.java:751)
> at com.sun.javatest.Harness.batch(Harness.java:447)
> at com.sun.javatest.batch.RunTestsCommand.run(RunTestsCommand.java:116)
> at com.sun.javatest.tool.CommandContext.runCommands(CommandContext.java:198)
> at com.sun.javatest.tool.Main.run(Main.java:427)
> at com.sun.javatest.tool.Main.main0(Main.java:139)
> at com.sun.javatest.tool.Main.main(Main.java:117)

src/com/sun/javatest/TestSuite.java line 1487:

> 1485:         private GeneralPurposeLogger(String name, WorkDirectory wd, String resourceBundleName, TestSuite ts) {
> 1486:             super(name, resourceBundleName);
> 1487:             this.logFileName = wd.getLogFileName();

hmm, 

this.logFileName = wd.getLogFileName();
``` 
and the next line if wd != null Does this mean than wd can be null. if yes line 1486 provokes NPE

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

PR: https://git.openjdk.org/jtharness/pull/40