Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(74)

Unified Diff: nss/lib/freebl/ecl/ecl.c

Issue 13898013: Update NSS to NSS_3_15_BETA2. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/deps/third_party/nss/
Patch Set: Update NSS versions and tag in README.chromium Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: nss/lib/freebl/ecl/ecl.c
===================================================================
--- nss/lib/freebl/ecl/ecl.c (revision 195639)
+++ nss/lib/freebl/ecl/ecl.c (working copy)
@@ -256,23 +256,21 @@
MP_CHECKOK(ec_group_set_gfp224(group, name));
#endif
break;
- case ECCurve_SECG_PRIME_521R1:
+#endif /* NSS_ECC_MORE_THAN_SUITE_B */
+ case ECCurve_SECG_PRIME_256R1:
group =
ECGroup_consGFp(&irr, &curvea, &curveb, &genx, &geny,
&order, params->cofactor);
if (group == NULL) { res = MP_UNDEF; goto CLEANUP; }
- MP_CHECKOK(ec_group_set_gfp521(group, name));
+ MP_CHECKOK(ec_group_set_gfp256(group, name));
+ MP_CHECKOK(ec_group_set_gfp256_32(group, name));
break;
agl 2013/04/24 22:27:00 I'm slightly surprised not to see something about
wtc 2013/04/24 22:51:14 Indeed. I also noticed this problem (that the P-38
-#endif /* NSS_ECC_MORE_THAN_SUITE_B */
- case ECCurve_SECG_PRIME_256R1:
+ case ECCurve_SECG_PRIME_521R1:
group =
ECGroup_consGFp(&irr, &curvea, &curveb, &genx, &geny,
&order, params->cofactor);
if (group == NULL) { res = MP_UNDEF; goto CLEANUP; }
-#ifdef NSS_ECC_MORE_THAN_SUITE_B
- MP_CHECKOK(ec_group_set_gfp256(group, name));
-#endif
- MP_CHECKOK(ec_group_set_gfp256_32(group, name));
+ MP_CHECKOK(ec_group_set_gfp521(group, name));
break;
default:
/* use generic arithmetic */

Powered by Google App Engine
This is Rietveld 408576698