RFR [8024521] (process) Async close issues with Process InputStream

[Alan Bateman]

On 17/10/2013 20:13, Ivan Gerasimov wrote:
> Thank you Alan!
>
> Yes, I missed that fact that reading from the recycled file descriptor 
> is actually a  problem by itself.
>
> Would you please take a look at another updated webrev?
>
> It contains another implementation suggested by Paul Sandoz with some 
> minor changes.
>
> http://cr.openjdk.java.net/~igerasim/8024521/2/webrev/ 
> <http://cr.openjdk.java.net/%7Eigerasim/8024521/2/webrev/>
>
> Here we synchronize close() with calls to available() and read() and 
> check for asynchronous close() that could have happened in between.
Thanks for the update, this looks much better. Are you planning to 
update the comments in processExited, they are a bit of out of date now.

I agree with Martin on the test. Minimally we should ensure that we have 
an automated test that exercises this scenario, even if it doesn't 
manage to reliably duplicate the issue with an un-patched -JDK.

-Alan