RFR: 8325567: jspawnhelper without args fails with segfault

Aleksey Shipilev shade at openjdk.org
Wed Mar 6 09:29:48 UTC 2024


On Tue, 5 Mar 2024 20:34:47 GMT, Vladimir Petko <vpetko at openjdk.org> wrote:

> > The change in jspawnhelper looks good.
> > I think it would be simpler to have a separate `jdk/java/lang/ProcessBuilder/JspawnhelperMisuse.java` test that simply invokes the `jspawnhelper` and verifies the proper message is printed. It would be more straightforward than trying to fit the _protocol_ test? Plus, we can test 0, 1, 3, 4 args, not only 0 args in that test, and we can also test 2 args with bad format.
> 
> Yes, this makes the test cleaner[1]. 

Note that we have `OutputAnalyzer` for these test cases: https://github.com/openjdk/jdk/blob/master/test/lib/jdk/test/lib/process/OutputAnalyzer.java -- lots of test use it, and it would be something like just:


  Process p = ProcessTools.startProcess(...);
  OutputAnalyzer oa = new OutputAnalyzer(p);
  oa.shouldNotHaveExitValue(0);
  oa.shouldContain("This command is not for general use");

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

PR Comment: https://git.openjdk.org/jdk/pull/18112#issuecomment-1980434240


More information about the core-libs-dev mailing list