RFR : 8133535: Better exception messaging in Ucrypto code
Seán Coffey
sean.coffey at oracle.com
Tue Sep 8 09:48:19 UTC 2015
On 08/09/2015 10:42, Ivan Gerasimov wrote:
> Thanks, Seán!
>
> On 08.09.2015 11:00, Seán Coffey wrote:
>> With recent changes from (JDK-8132082) affecting the same ucrypto
>> code, I've re-based my patch. Here's the new webrev :
>>
>> http://cr.openjdk.java.net/~coffeys/webrev.8133535.jdk9.v2/webrev/
>>
> This looks very useful.
>
> NativeCipher.java:
> 283 ("Wrong IV length: must be " +
> blockSize +
> 284 " bytes long");
> Would it make sense to include the actual length of the vector?
Yes - good catch Ivan. Will include that value.
>
> In a couple of places (e.g. NativeKey.java, line #146) the indentation
> isn't consistent.
Good point, I'll run through indentation again before final push.
Thanks,
Sean.
>
> Otherwise looks good to me!
>
> Sincerely yours,
> Ivan
>
>> Regards,
>> Sean.
>>
>> On 24/08/2015 13:54, Seán Coffey wrote:
>>> Hoping to improve some of the exception messaging that is thrown
>>> from Ucrypto code. I'm hoping to tackle other components in the
>>> security libraries on a case by case basis. Aim is to improve
>>> exception messages and capture detail for end user where possible.
>>>
>>> bug report : https://bugs.openjdk.java.net/browse/JDK-8133535
>>> webrev :
>>> http://cr.openjdk.java.net/~coffeys/webrev.8133535.jdk9/webrev/
>>>
>>
>>
>
More information about the security-dev
mailing list