[rfc][icedtea-web] PR1592 reproducers update

Andrew Azores aazores at redhat.com
Thu Jan 9 11:06:56 PST 2014


On 01/09/2014 11:17 AM, Andrew Azores wrote:
> On 01/03/2014 02:43 PM, Andrew Azores wrote:
>> Updated PR1592 tests, using a custom reproducer rather than split 
>> simple/signed. This allows method calls to be made in the normal way 
>> as well as via reflection. JNLP includes both applications and 
>> applets now, and they close properly as well.
>>
>> (snip)
>>
>> Thanks,
>> Andrew A
>
> Went back over this and realized one of the tests was written wrong. 
> The assertAccessControlException helper method in the testcase file is 
> now a little stricter about the type of AccessControlException (so 
> that the exceptions due to applets not being allowed to call 
> System.exit don't falsely fulfill this assertion), and 
> MixedSigningAppletHelper.attackDoPrivileged now properly calls 
> MixedSigningAppletSigned#testSignedReadPropertiesDoPrivileged, as it 
> should have been doing. In this case, the Unsigned JAR actually *is* 
> meant to be able to retrieve data from the Signed JAR (as is the point 
> of the AccessController.doPrivileged call), so the testcases now 
> expect this test to successfully read from System.getProperty, rather 
> than receive an AccessControlException. However, the tests still 
> verify that in situations where the Signed JAR has a method call that 
> involves a privileged action *without* being placed inside a 
> doPrivileged call, an AccessControlException will be thrown if the 
> Unsigned code attempts to access it, as expected.
>
> Thanks,
>

Sorry, please ignore the previous patch. The extra changes were not made 
based on the most recent other changes. Attached are the properly 
rebased patches, also split into three as they were originally.

Thanks,

-- 
Andrew A

-------------- next part --------------
A non-text attachment was scrubbed...
Name: custom-resources2.patch
Type: text/x-patch
Size: 120943 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140109/77a6abf1/custom-resources2-0001.patch 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: custom-srcs2.patch
Type: text/x-patch
Size: 29260 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140109/77a6abf1/custom-srcs2-0001.patch 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: custom-testcases2.patch
Type: text/x-patch
Size: 30442 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140109/77a6abf1/custom-testcases2-0001.patch 


More information about the distro-pkg-dev mailing list