Possible SecurityException creating processes on Linux

Martin Buchholz martinrb at google.com
Sat Aug 28 19:02:49 UTC 2010


Hi Alan,

Thanks for the review.

On Sat, Aug 28, 2010 at 01:36, Alan Bateman <Alan.Bateman at oracle.com> wrote:

> Martin Buchholz wrote:
>
>>
>> The fix looks fine.  It might be good to indent the second and third lines
> of the test summary


Done.


> to make it easier to distinguish the test tags from the text. On Windows I
> assume the "env" command will run the Cygwin or MKS "env" program rather
> than the built-in "env" command.


I changed this to use the most portable process invocation, namely
<java.home>/bin/java -version

IOException is no longer acceptable; removed the catch.

It probably matter as the test doesn't actually need the child process to do
> anything.
>
>
Right.  But let's keep our standards as high as possible.

In the same spirit, I tightened the FilePermission from
read,write,execute => execute

Committing.

Martin

-Alan.
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/core-libs-dev/attachments/20100828/9fbfcf14/attachment.html>


More information about the core-libs-dev mailing list