[15] RFR: 8242335: Additional Tests for RSASSA-PSS

sibabrata.sahoo at oracle.com sibabrata.sahoo at oracle.com
Mon May 4 06:27:11 UTC 2020


Hi Valerie,

Here is the updated web rev addressing the following comments. There are 7 lines of additional change added [Line: 126-133] to re-verify the signature with another Signature instance.

Webrev: http://cr.openjdk.java.net/~ssahoo/8242335/webrev.01/ <http://cr.openjdk.java.net/~ssahoo/8242335/webrev.01/>

Thanks,
Siba



> On 01-May-2020, at 4:58 AM, Valerie Peng <valerie.peng at oracle.com> wrote:
> 
> Hi, Siba,
> 
> Here are my comments:
> 
> <SerializedPSSKey.java>
> 
> - Typos: Initialisation -> Initialization.
> 
> - the 2nd algorithm specific initialization using reflection doesn't seem too useful. The code path should be exactly the same as the non-reflection one. Maybe just remove it.
> 
> Rest looks fine.
> 
> Thanks,
> Valerie
> On 4/28/2020 12:38 AM, sibabrata.sahoo at oracle.com <mailto:sibabrata.sahoo at oracle.com> wrote:
>> Hi Valerie,
>> 
>> Please review the patch for,
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8242335 <https://bugs.openjdk.java.net/browse/JDK-8242335>
>> Webrev: http://cr.openjdk.java.net/~ssahoo/8242335/webrev.00/ <http://cr.openjdk.java.net/~ssahoo/8242335/webrev.00/>
>> 
>> These are additional Tests developed to support RSASSA-PSS algorithm. One is key compatibility with OpenSSL generated while the other one is the API Tests for serialized keys.
>> 
>> Thanks,
>> Siba
>> 
>> 
>> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/security-dev/attachments/20200504/8532cb1a/attachment.htm>


More information about the security-dev mailing list