RFR 8130302: jarsigner and keytool -providerClass needs be re-examined for modules

[Wang Weijun]

Ping again to security-dev. Anyone can approve it?

The latest webrev is at

   http://cr.openjdk.java.net/~weijun/8130302/webrev.06

Change from webrev.05 [1] is tiny.

Thanks
Max

[1] http://cr.openjdk.java.net/~weijun/8130302/webrev.06/interdiff.patch.html

> On Jun 16, 2016, at 9:33 AM, Wang Weijun <weijun.wang at oracle.com> wrote:
> 
> 
>> On Jun 16, 2016, at 7:50 AM, Valerie Peng <valerie.peng at oracle.com> wrote:
>> 
>> No big difference to me.
> 
> Good, I'll remove the cast.
> 
> @security-dev, can someone approve the whole webrev.05?
> 
>   http://cr.openjdk.java.net/~weijun/8130302/webrev.05
> 
> Thanks
> Max
> 
>> Valerie
>> 
>> On 6/15/2016 8:40 AM, Wang Weijun wrote:
>>>> On Jun 15, 2016, at 10:57 PM, Mandy Chung<mandy.chung at oracle.com>  wrote:
>>>> 
>>>>>> 241             throw (InvalidParameterException)
>>>>>> 
>>>>>> This cast should not be needed?
>>>>>> 
>>>>> } catch (UcryptoException ue) {
>>>>>  throw (InvalidParameterException)
>>>>>      new InvalidParameterException("Error using " + configArg).
>>>>>          initCause(ue.getCause());
>>>>> }
>>>>> 
>>>>> initCause() returns Throwable but the method's signature throws InvalidParameterException.
>>>>> 
>>>> Perhaps have a local variable for InvalidParameterException exception.
>>> Valerie, are you OK with this?
>>> 
>>> --Max
>>> 
>>>> Mandy
>