Thanks for doing this Max. The syntax looks fine.
Just one question. Do you think it is better to specify each socket option literally in the tool
as you have done (ie. the only supported NetworkPermission
is SO_FLOW_SLA with this change) or allow users to type in the option name as free-form text I guess you can always edit policy files manually, but if more socket options get added
in future, then we will need to update the tool again.

Michael

On 26/08/14 01:58, Wang Weijun wrote:
Hi Michael

Please review the code change at

    http://cr.openjdk.java.net/~weijun/8055901/webrev.00/

With this change, the policy tool can generate a file like

grant {
   permission jdk.net.NetworkPermission "setOption.SO_FLOW_SLA";
};

Please confirm the grammar is correct.

BTW, I also add some missing targets in a nearby permission type.

Noreg-trivial.

Thanks
Max


Reply via email to