RFR: 8282219: jdk/java/lang/ProcessBuilder/Basic.java fails on AIX [v4]

[Roger Riggs]

On Wed, 23 Feb 2022 18:49:22 GMT, Ichiroh Takiguchi <itakiguchi at openjdk.org> wrote:

>> Run jtreg:jdk/java/lang/ProcessBuilder/Basic.java on AIX.
>> The test was failed by:
>>   Incorrect handling of envstrings containing NULs
>> 
>> According to my investigation, this issue was happened after following change was applied.
>> JDK-8272600: (test) Use native "sleep" in Basic.java
>> 
>> test.nativepath value was added into AIX's LIBPATH during running this testcase.
>> On AIX, test.nativepath value should be removed from LIBPATH value before comparing the values.
>
> Ichiroh Takiguchi has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add null check

Jtreg constructs the environment before launching the Agent to run the test.
https://github.com/openjdk/jtreg/blob/163ae223409f789cb733d32ed8d33686e14a81d9/src/share/classes/com/sun/javatest/regtest/exec/MainAction.java#L415

For AIX, it appends the `test.nativepath` to LIBPATH.
https://github.com/openjdk/jtreg/blob/163ae223409f789cb733d32ed8d33686e14a81d9/src/share/classes/com/sun/javatest/regtest/exec/Action.java#L146

As far as I can tell, the test does not modify the environment and originally passes LIBPATH without modification.

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

PR: https://git.openjdk.java.net/jdk/pull/7574