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

Issue 3135002: Update to NSS 3.12.7 and NSPR 4.8.6.... (Closed)

Created:
10 years, 4 months ago by wtc
Modified:
9 years, 7 months ago
Reviewers:
agl
CC:
chromium-reviews
Visibility:
Public.

Description

Update to NSS 3.12.7 and NSPR 4.8.6. Remove two NSPR patches because they were checked in to the NSPR upstream. Remove the mozilla/security/nss/lib/pki1 directory. It's been removed in the upstream. R=agl BUG=none TEST=no build errors or test failures. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=55513

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+703 lines, -7110 lines) Patch
M nss/README.chromium View 3 chunks +3 lines, -9 lines 0 comments Download
M nss/mozilla/nsprpub/lib/ds/plarena.h View 1 chunk +4 lines, -3 lines 0 comments Download
M nss/mozilla/nsprpub/lib/ds/plarena.c View 2 chunks +16 lines, -6 lines 0 comments Download
M nss/mozilla/nsprpub/lib/ds/plarenas.h View 1 chunk +5 lines, -0 lines 0 comments Download
M nss/mozilla/nsprpub/lib/libc/include/plgetopt.h View 1 chunk +3 lines, -2 lines 0 comments Download
M nss/mozilla/nsprpub/lib/libc/src/plgetopt.c View 4 chunks +36 lines, -10 lines 0 comments Download
M nss/mozilla/nsprpub/pr/include/md/_pth.h View 1 chunk +6 lines, -0 lines 0 comments Download
M nss/mozilla/nsprpub/pr/include/pratom.h View 1 chunk +10 lines, -6 lines 0 comments Download
M nss/mozilla/nsprpub/pr/include/prbit.h View 5 chunks +43 lines, -0 lines 0 comments Download
M nss/mozilla/nsprpub/pr/include/prinit.h View 1 chunk +2 lines, -2 lines 0 comments Download
M nss/mozilla/nsprpub/pr/include/prlink.h View 2 chunks +2 lines, -2 lines 0 comments Download
M nss/mozilla/nsprpub/pr/include/prlog.h View 1 chunk +0 lines, -8 lines 0 comments Download
M nss/mozilla/nsprpub/pr/include/prtypes.h View 3 chunks +11 lines, -3 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/linking/prlink.c View 3 chunks +4 lines, -4 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/malloc/prmem.c View 2 chunks +2 lines, -2 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/md/unix/uxrng.c View 1 chunk +4 lines, -0 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/md/windows/w95io.c View 2 chunks +8 lines, -8 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/md/windows/w95thred.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/nsprpub/pr/src/misc/prdtoa.c View 5 chunks +15 lines, -0 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/misc/prinit.c View 2 chunks +2 lines, -2 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/misc/prlog2.c View 2 chunks +4 lines, -26 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/misc/prnetdb.c View 2 chunks +12 lines, -1 line 0 comments Download
M nss/mozilla/nsprpub/pr/src/misc/prsystem.c View 1 chunk +14 lines, -0 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/misc/prtime.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/nsprpub/pr/src/pthreads/ptsynch.c View 18 chunks +47 lines, -34 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/pthreads/ptthread.c View 1 chunk +2 lines, -2 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/threads/combined/pruthr.c View 1 chunk +2 lines, -2 lines 0 comments Download
M nss/mozilla/nsprpub/pr/src/threads/prtpd.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/base/arena.c View 6 chunks +15 lines, -13 lines 0 comments Download
M nss/mozilla/security/nss/lib/certdb/alg1485.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/certdb/cert.h View 2 chunks +5 lines, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/certdb/certdb.c View 4 chunks +4 lines, -14 lines 0 comments Download
M nss/mozilla/security/nss/lib/certdb/certi.h View 5 chunks +13 lines, -14 lines 0 comments Download
M nss/mozilla/security/nss/lib/certdb/certt.h View 3 chunks +4 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/certdb/crl.c View 7 chunks +9 lines, -105 lines 0 comments Download
M nss/mozilla/security/nss/lib/certdb/genname.c View 3 chunks +41 lines, -17 lines 0 comments Download
M nss/mozilla/security/nss/lib/certhigh/certvfy.c View 2 chunks +6 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/certhigh/ocsp.c View 2 chunks +2 lines, -2 lines 0 comments Download
M nss/mozilla/security/nss/lib/cryptohi/keythi.h View 3 chunks +19 lines, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/cryptohi/sechash.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/cryptohi/seckey.c View 8 chunks +43 lines, -22 lines 1 comment Download
M nss/mozilla/security/nss/lib/cryptohi/secvfy.c View 5 chunks +9 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/dev/devslot.c View 3 chunks +3 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/dev/devtoken.c View 3 chunks +3 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/dev/devutil.c View 4 chunks +6 lines, -7 lines 0 comments Download
M nss/mozilla/security/nss/lib/freebl/camellia.c View 2 chunks +5 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/freebl/mpi/mpi.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/freebl/mpi/mpi-priv.h View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/freebl/mpi/mpmontg.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/freebl/shvfy.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/freebl/unix_rand.c View 1 chunk +5 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/nss/nss.h View 3 chunks +5 lines, -5 lines 0 comments Download
M nss/mozilla/security/nss/lib/nss/nssinit.c View 4 chunks +4 lines, -4 lines 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/debug_module.c View 3 chunks +4 lines, -4 lines 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11akey.c View 1 chunk +5 lines, -0 lines 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11auth.c View 1 chunk +5 lines, -9 lines 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11cert.c View 2 chunks +10 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11cxt.c View 1 chunk +2 lines, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11err.c View 1 chunk +3 lines, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11load.c View 3 chunks +5 lines, -5 lines 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11merge.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11nobj.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11pub.h View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11skey.c View 3 chunks +8 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11slot.c View 2 chunks +2 lines, -2 lines 0 comments Download
M nss/mozilla/security/nss/lib/pk11wrap/pk11util.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/pkcs7/p7common.c View 2 chunks +2 lines, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/pkcs7/p7local.c View 5 chunks +6 lines, -4 lines 0 comments Download
M nss/mozilla/security/nss/lib/pki/certificate.c View 2 chunks +2 lines, -2 lines 0 comments Download
M nss/mozilla/security/nss/lib/pki/cryptocontext.c View 2 chunks +1 line, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/pki/nsspki.h View 2 chunks +1 line, -5 lines 0 comments Download
M nss/mozilla/security/nss/lib/pki/nsspkit.h View 3 chunks +11 lines, -2 lines 0 comments Download
M nss/mozilla/security/nss/lib/pki/pki3hack.c View 3 chunks +3 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/pki/pkibase.c View 6 chunks +12 lines, -5 lines 0 comments Download
M nss/mozilla/security/nss/lib/pki/trustdomain.c View 2 chunks +1 line, -5 lines 0 comments Download
D nss/mozilla/security/nss/lib/pki1/nsspki1.h View 1 chunk +0 lines, -2872 lines 0 comments Download
D nss/mozilla/security/nss/lib/pki1/nsspki1t.h View 1 chunk +0 lines, -205 lines 0 comments Download
D nss/mozilla/security/nss/lib/pki1/oiddata.h View 1 chunk +0 lines, -214 lines 0 comments Download
D nss/mozilla/security/nss/lib/pki1/pki1.h View 1 chunk +0 lines, -3037 lines 0 comments Download
D nss/mozilla/security/nss/lib/pki1/pki1t.h View 1 chunk +0 lines, -107 lines 0 comments Download
M nss/mozilla/security/nss/lib/smime/cms.h View 2 chunks +2 lines, -2 lines 0 comments Download
M nss/mozilla/security/nss/lib/softoken/sftkdb.c View 6 chunks +56 lines, -7 lines 0 comments Download
M nss/mozilla/security/nss/lib/softoken/sftkpars.c View 5 chunks +8 lines, -6 lines 0 comments Download
M nss/mozilla/security/nss/lib/softoken/softkver.h View 1 chunk +6 lines, -5 lines 0 comments Download
M nss/mozilla/security/nss/lib/ssl/sslerr.h View 2 chunks +2 lines, -2 lines 0 comments Download
M nss/mozilla/security/nss/lib/util/nssilock.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/util/nssutil.h View 1 chunk +3 lines, -3 lines 0 comments Download
M nss/mozilla/security/nss/lib/util/portreg.c View 1 chunk +1 line, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/util/secdig.h View 2 chunks +2 lines, -2 lines 0 comments Download
M nss/mozilla/security/nss/lib/util/secerr.h View 1 chunk +3 lines, -0 lines 0 comments Download
M nss/mozilla/security/nss/lib/util/secoid.c View 2 chunks +26 lines, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/util/secoidt.h View 2 chunks +7 lines, -1 line 0 comments Download
M nss/mozilla/security/nss/lib/util/secport.c View 2 chunks +2 lines, -6 lines 0 comments Download
M nss/nss.gyp View 3 chunks +0 lines, -7 lines 0 comments Download
D nss/patches/nspr-private-extern.patch View 1 chunk +0 lines, -147 lines 0 comments Download
D nss/patches/nspr-remove-flat-carbon-headers.patch View 1 chunk +0 lines, -22 lines 0 comments Download
M nss/patches/nss-static.patch View 13 chunks +28 lines, -28 lines 0 comments Download
M nss/scripts/nspr-checkout.sh View 1 chunk +1 line, -1 line 0 comments Download
M nss/scripts/nss-checkout.sh View 2 chunks +1 line, -4 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
wtc
It's impractical to review all the changes. You just need to review the changes to: ...
10 years, 4 months ago (2010-08-09 22:57:59 UTC) #1
agl
LGTM. (Does this include the EDH minimum? I suspect not since it should probably be ...
10 years, 4 months ago (2010-08-09 23:22:10 UTC) #2
wtc
10 years, 4 months ago (2010-08-10 00:24:30 UTC) #3
http://codereview.chromium.org/3135002/diff/1/42
File nss/mozilla/security/nss/lib/cryptohi/seckey.c (right):

http://codereview.chromium.org/3135002/diff/1/42#newcode232
nss/mozilla/security/nss/lib/cryptohi/seckey.c:232: if (!param ||
!param->base.data || !param->prime.data ||
This is a DH key size check.  Without the DH key size check
in ssl3_HandleServerKeyExchange in libssl, we will get here
and still reject the 256-bit DH domain parameters that the
server suggested.

In my testing with Firefox, the test here alone is enough
to cause SSL handshake to fail.  It's just that the error
code SEC_ERROR_INVALID_ARGS is less informative than
SSL_ERROR_RX_MALFORMED_SERVER_KEY_EXCH.

To find the check in ssl3_HandleServerKeyExchange, search
for "< 512" in ssl3con.c.

Powered by Google App Engine
This is Rietveld 408576698