<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Vinnie,<br>
<br>
Thanks for the comment!<br>
The reason that I use "<" instead of "<=" on line 150 is
because that (1024, 160) is supported by the DSAGenParameterSpec
class and thus I didn't view this as legacy value. Either init(...)
calls should work for 1024, so I'd like to use the newer form of
init(...) for 1024 to separate it from the legacy values.<br>
I noticed that some comments in ParameterCache class is a bit
outdated, so I updated webrev again just to update the comments to
mention the default/pre-computed parameters for (2048, 224) and
(2048, 256).<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~valeriep/7199939/webrev.01/">http://cr.openjdk.java.net/~valeriep/7199939/webrev.01/</a><br>
Thanks,<br>
Valerie<br>
<br>
On 09/24/12 02:39, Vincent Ryan wrote:
<blockquote
cite="mid:EF025D63-BBC0-40F2-B2A1-D623464DE958@oracle.com"
type="cite">Fixes look good.
<div>Just a minor comment at l.150 of ParameterCache: the test
could use '<='??
<div><br>
<div>
<div>
<div>
<div>On 22 Sep 2012, at 00:45, Valerie (Yu-Ching) Peng
wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite">
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
<div text="#000000" bgcolor="#FFFFFF">
<pre>Hi, Vinnie,
Can you please review these following 2 JDK8 fixes? They are straight forward regressions found by SQE tests introduced as byproducts of the NSA Suite B changes.
Since they are already caught by existing SQE tests, so I didn't add more regression tests.
<a moz-do-not-send="true" href="http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7199939">7199939</a>: DSA 576 ad 640 bit keys fail when initializing for No precomputed parameters
Webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Evaleriep/7199939/webrev.00/">http://cr.openjdk.java.net/~valeriep/7199939/webrev.00/</a>
Fix: Fixed the impl of DSAKeyPairGenerator.initialize(int, SecureRandom) so it will attempt to generate new parameters (instead
of error out) when no precomputed parameters are available. Also fixed ParameterCache.getNewDSAParameterSpec(...) method to handle legacy DSA key sizes.
<a moz-do-not-send="true" href="http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7199941">7199941</a>: test about AES/ECB mode fails
Webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Evaleriep/7199941/webrev.00/">http://cr.openjdk.java.net/~valeriep/7199941/webrev.00/</a>
Fix: Moved the initialization of the field 'blockMode' back to the right code block.
The failed SQE tests now pass after the code changes.
Thanks!
Valerie
</pre>
</div>
</blockquote>
</div>
<br>
</div>
</div>
</div>
</div>
</blockquote>
<br>
</body>
</html>