[rfc][icedtea-web][policyeditor] CustomPermissions toString correction for basic permissions

[Andrew Azores]

Hi,

There are two major types of permissions that go into policy files - 
standard permissions, and basic permissions. Standard permissions have 
the following format:

permission $permission_type "$permission_target", 
"$permission_action1,$permission_action2";

Basic permissions have the following format:

permission $permission_type "$permission_target";

Without this patch, the CustomPermission class erroneously formats its 
basic permissions to include an empty actions string. So, in the case of:

permission java.lang.RuntimePermission "createClassLoader";

Instead, it would be presented as:

permission java.lang.RuntimePermission "createClassLoader", "";

This patch very simply fixes this bug (which was due to comparing a 
string to an enum member containing a string field, rather than 
comparing a string to the string field *contained* by the enum member) 
and adds a unit test for this case.

Thanks,

-- 
Andrew A

-------------- next part --------------
A non-text attachment was scrubbed...
Name: custompermission-tostring-actions.patch
Type: text/x-patch
Size: 2544 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140606/55b6a81e/custompermission-tostring-actions.patch>