Your fix looks good. On 26 Jul 2013, at 01:33, Sean Mullan wrote:
> On 07/25/2013 05:08 PM, Sean Mullan wrote: >> Hello, >> >> Could you please review my fix for 8001319: >> >> webrev: http://cr.openjdk.java.net/~mullan/webrevs/8012288/webrev.00/ > > Correction above > > webrev: http://cr.openjdk.java.net/~mullan/webrevs/8001319/webrev.00/ > >> bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8001319 >> >> In this fix, we are adding a new SecurityPermission "insertProvider" >> target name for adding new security providers, and discouraging use of >> the existing "insertProvider.{provider name}" target name. Please read >> the bug report for more information. >> >> Thanks, >> Sean >
