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

Issue 11275240: Update net/third_party/nss/ssl to NSS 3.14. (Closed)

Created:
8 years, 1 month ago by wtc
Modified:
8 years, 1 month ago
Reviewers:
rsleevi, agl
CC:
chromium-reviews, Aaron Boodman, cbentzel+watch_chromium.org, darin-cc_chromium.org, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Adjust the patched code #

Patch Set 3 : Regenerate patch files, add ecpointform.patch #

Patch Set 4 : #

Patch Set 5 : Update applypatches.sh #

Patch Set 6 : Correct the NSS version in which EC_POINT_FORM_UNCOMPRESSED was moved to blapit.h #

Total comments: 2

Patch Set 7 : Add secretexporterlocks.patch. Improve ecpointform.patch. #

Patch Set 8 : Upload before commit #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1191 lines, -8427 lines) Patch
M net/third_party/nss/README.chromium View 1 2 3 4 5 6 4 chunks +15 lines, -40 lines 0 comments Download
M net/third_party/nss/patches/applypatches.sh View 1 2 3 4 5 6 1 chunk +11 lines, -25 lines 0 comments Download
M net/third_party/nss/patches/cachecerts.patch View 1 2 7 chunks +18 lines, -32 lines 0 comments Download
M net/third_party/nss/patches/channelid.patch View 1 2 19 chunks +121 lines, -142 lines 0 comments Download
D net/third_party/nss/patches/checkuncache.patch View 1 chunk +0 lines, -164 lines 0 comments Download
M net/third_party/nss/patches/clientauth.patch View 1 2 21 chunks +99 lines, -91 lines 0 comments Download
D net/third_party/nss/patches/dhvalues.patch View 1 chunk +0 lines, -53 lines 0 comments Download
M net/third_party/nss/patches/didhandshakeresume.patch View 1 2 2 chunks +10 lines, -10 lines 0 comments Download
D net/third_party/nss/patches/dtls.patch View 1 chunk +0 lines, -3322 lines 0 comments Download
D net/third_party/nss/patches/dtlssrtp.patch View 1 chunk +0 lines, -468 lines 0 comments Download
A net/third_party/nss/patches/ecpointform.patch View 1 2 3 4 5 6 1 chunk +19 lines, -0 lines 0 comments Download
M net/third_party/nss/patches/falsestartnpn.patch View 1 2 3 chunks +9 lines, -11 lines 0 comments Download
D net/third_party/nss/patches/getchannelinfo.patch View 1 chunk +0 lines, -27 lines 0 comments Download
M net/third_party/nss/patches/getrequestedclientcerttypes.patch View 1 2 4 chunks +34 lines, -34 lines 0 comments Download
D net/third_party/nss/patches/keylog.patch View 1 chunk +0 lines, -189 lines 0 comments Download
M net/third_party/nss/patches/negotiatedextension.patch View 1 2 2 chunks +5 lines, -15 lines 0 comments Download
M net/third_party/nss/patches/ocspstapling.patch View 1 2 20 chunks +95 lines, -95 lines 0 comments Download
M net/third_party/nss/patches/peercertchain.patch View 1 2 2 chunks +25 lines, -53 lines 0 comments Download
D net/third_party/nss/patches/recordlayerversion.patch View 1 chunk +0 lines, -196 lines 0 comments Download
D net/third_party/nss/patches/renegoclientversion.patch View 1 chunk +0 lines, -114 lines 0 comments Download
M net/third_party/nss/patches/renegoscsv.patch View 1 2 1 chunk +4 lines, -4 lines 0 comments Download
M net/third_party/nss/patches/restartclientauth.patch View 1 2 4 chunks +28 lines, -27 lines 0 comments Download
A net/third_party/nss/patches/secretexporterlocks.patch View 1 2 3 4 5 6 1 chunk +44 lines, -0 lines 0 comments Download
M net/third_party/nss/patches/sslkeylogerror.patch View 1 2 2 chunks +7 lines, -8 lines 0 comments Download
D net/third_party/nss/patches/sslprotocolvariant.patch View 1 chunk +0 lines, -52 lines 0 comments Download
M net/third_party/nss/patches/tlsunique.patch View 1 2 3 chunks +43 lines, -47 lines 0 comments Download
M net/third_party/nss/patches/versionskew.patch View 1 2 4 chunks +12 lines, -24 lines 0 comments Download
M net/third_party/nss/ssl.gyp View 1 chunk +1 line, -2 lines 0 comments Download
M net/third_party/nss/ssl/Makefile View 2 chunks +5 lines, -35 lines 0 comments Download
M net/third_party/nss/ssl/SSLerrs.h View 4 chunks +15 lines, -41 lines 0 comments Download
M net/third_party/nss/ssl/authcert.c View 1 chunk +4 lines, -36 lines 0 comments Download
D net/third_party/nss/ssl/bodge/ec.h View 1 2 3 4 5 6 1 chunk +0 lines, -52 lines 0 comments Download
M net/third_party/nss/ssl/cmpcert.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/derive.c View 5 chunks +19 lines, -37 lines 0 comments Download
D net/third_party/nss/ssl/dtls1con.c View 1 chunk +0 lines, -1165 lines 0 comments Download
A + net/third_party/nss/ssl/dtlscon.c View 5 chunks +11 lines, -43 lines 0 comments Download
M net/third_party/nss/ssl/manifest.mn View 3 chunks +4 lines, -37 lines 0 comments Download
M net/third_party/nss/ssl/notes.txt View 1 chunk +3 lines, -35 lines 0 comments Download
D net/third_party/nss/ssl/nsskea.c View 1 chunk +0 lines, -78 lines 0 comments Download
M net/third_party/nss/ssl/os2_err.h View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/os2_err.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/preenc.h View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/prelib.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/ssl.h View 1 2 chunks +5 lines, -38 lines 0 comments Download
M net/third_party/nss/ssl/ssl.rc View 1 chunk +3 lines, -35 lines 0 comments Download
M net/third_party/nss/ssl/ssl3con.c View 58 chunks +247 lines, -187 lines 0 comments Download
M net/third_party/nss/ssl/ssl3ecc.c View 1 2 3 4 5 6 2 chunks +13 lines, -40 lines 0 comments Download
M net/third_party/nss/ssl/ssl3ext.c View 21 chunks +60 lines, -61 lines 0 comments Download
M net/third_party/nss/ssl/ssl3gthr.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/ssl3prot.h View 1 chunk +4 lines, -37 lines 0 comments Download
M net/third_party/nss/ssl/sslauth.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslcon.c View 4 chunks +7 lines, -40 lines 0 comments Download
M net/third_party/nss/ssl/ssldef.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslenum.c View 1 chunk +4 lines, -38 lines 0 comments Download
M net/third_party/nss/ssl/sslerr.h View 3 chunks +13 lines, -41 lines 0 comments Download
M net/third_party/nss/ssl/sslerr.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslerrstrs.c View 1 chunk +3 lines, -35 lines 0 comments Download
M net/third_party/nss/ssl/sslgathr.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslimpl.h View 1 9 chunks +8 lines, -55 lines 0 comments Download
M net/third_party/nss/ssl/sslinfo.c View 1 2 3 4 5 6 3 chunks +6 lines, -40 lines 0 comments Download
M net/third_party/nss/ssl/sslinit.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslmutex.h View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslmutex.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslnonce.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslproto.h View 1 chunk +4 lines, -37 lines 0 comments Download
M net/third_party/nss/ssl/sslreveal.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslsecur.c View 5 chunks +9 lines, -41 lines 0 comments Download
M net/third_party/nss/ssl/sslsnce.c View 6 chunks +18 lines, -41 lines 0 comments Download
M net/third_party/nss/ssl/sslsock.c View 11 chunks +39 lines, -65 lines 0 comments Download
M net/third_party/nss/ssl/sslt.h View 1 chunk +4 lines, -37 lines 0 comments Download
M net/third_party/nss/ssl/ssltrace.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/sslver.c View 1 chunk +3 lines, -35 lines 0 comments Download
M net/third_party/nss/ssl/unix_err.h View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/unix_err.c View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/win32err.h View 1 chunk +4 lines, -36 lines 0 comments Download
M net/third_party/nss/ssl/win32err.c View 1 chunk +4 lines, -36 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
wtc
Please review patch set 5. Thanks.
8 years, 1 month ago (2012-11-10 01:56:22 UTC) #1
wtc
On 2012/11/10 01:56:22, wtc wrote: > Please review patch set 5. Thanks. Argh, I meant ...
8 years, 1 month ago (2012-11-10 01:57:12 UTC) #2
agl
lgtm
8 years, 1 month ago (2012-11-12 16:10:15 UTC) #3
wtc
https://codereview.chromium.org/11275240/diff/91/net/third_party/nss/ssl/sslinfo.c File net/third_party/nss/ssl/sslinfo.c (left): https://codereview.chromium.org/11275240/diff/91/net/third_party/nss/ssl/sslinfo.c#oldcode380 net/third_party/nss/ssl/sslinfo.c:380: ssl_GetSSL3HandshakeLock(ss); I need to look into why the NSS ...
8 years, 1 month ago (2012-11-12 18:31:00 UTC) #4
wtc
I improved the CL at bit. You can just review the diffs between patch sets ...
8 years, 1 month ago (2012-11-13 01:18:57 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/wtc@chromium.org/11275240/23025
8 years, 1 month ago (2012-11-13 05:03:32 UTC) #6
commit-bot: I haz the power
Failed to apply patch for net/third_party/nss/ssl/dtlscon.c: While running patch -p0 --forward --force --no-backup-if-mismatch; patching file ...
8 years, 1 month ago (2012-11-13 05:04:08 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/wtc@chromium.org/11275240/23025
8 years, 1 month ago (2012-11-13 14:44:13 UTC) #8
commit-bot: I haz the power
8 years, 1 month ago (2012-11-13 14:44:41 UTC) #9
Failed to apply patch for net/third_party/nss/ssl/dtlscon.c:
While running patch -p0 --forward --force --no-backup-if-mismatch;
  patching file net/third_party/nss/ssl/dtlscon.c
  Hunk #1 FAILED at 1.
  Hunk #2 FAILED at 286.
  Hunk #3 FAILED at 453.
  Hunk #4 FAILED at 497.
  Hunk #5 FAILED at 623.
  5 out of 5 hunks FAILED -- saving rejects to file
net/third_party/nss/ssl/dtlscon.c.rej

Patch:   N   net/third_party/nss/ssl/dtlscon.c
Index: net/third_party/nss/ssl/dtlscon.c
===================================================================
--- net/third_party/nss/ssl/dtlscon.c	(revision 166942)
+++ net/third_party/nss/ssl/dtlscon.c	(working copy)
@@ -1,43 +1,11 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+
 /*
  * DTLS Protocol
- *
- * ***** BEGIN LICENSE BLOCK *****
- * Version: MPL 1.1/GPL 2.0/LGPL 2.1
- *
- * The contents of this file are subject to the Mozilla Public License Version
- * 1.1 (the "License"); you may not use this file except in compliance with
- * the License. You may obtain a copy of the License at
- * http://www.mozilla.org/MPL/
- *
- * Software distributed under the License is distributed on an "AS IS" basis,
- * WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License
- * for the specific language governing rights and limitations under the
- * License.
- *
- * The Original Code is the Netscape security libraries.
- *
- * The Initial Developer of the Original Code is
- * Netscape Communications Corporation.
- * Portions created by the Initial Developer are Copyright (C) 1994-2000
- * the Initial Developer. All Rights Reserved.
- *
- * Contributor(s):
- *   Eric Rescorla <ekr@rtfm.com>
- *
- * Alternatively, the contents of this file may be used under the terms of
- * either the GNU General Public License Version 2 or later (the "GPL"), or
- * the GNU Lesser General Public License Version 2.1 or later (the "LGPL"),
- * in which case the provisions of the GPL or the LGPL are applicable instead
- * of those above. If you wish to allow use of your version of this file only
- * under the terms of either the GPL or the LGPL, and not to allow others to
- * use your version of this file under the terms of the MPL, indicate your
- * decision by deleting the provisions above and replace them with the notice
- * and other provisions required by the GPL or the LGPL. If you do not delete
- * the provisions above, a recipient may use your version of this file under
- * the terms of any one of the MPL, the GPL or the LGPL.
- *
- * ***** END LICENSE BLOCK ***** */
-/* $Id: $ */
+ */
+/* $Id: dtlscon.c,v 1.5 2012/09/28 01:46:45 wtc%google.com Exp $ */
 
 #include "ssl.h"
 #include "sslimpl.h"
@@ -286,7 +254,7 @@
 
             /* At this point we are advancing our state machine, so
              * we can free our last flight of messages */
-            dtls_FreeHandshakeMessages(ss->ssl3.hs.lastMessageFlight);
+            dtls_FreeHandshakeMessages(&ss->ssl3.hs.lastMessageFlight);
 	    ss->ssl3.hs.recvdHighWater = -1;
 	    dtls_CancelTimer(ss);
 
@@ -453,7 +421,7 @@
 
 		    /* At this point we are advancing our state machine, so
 		     * we can free our last flight of messages */
-		    dtls_FreeHandshakeMessages(ss->ssl3.hs.lastMessageFlight);
+		    dtls_FreeHandshakeMessages(&ss->ssl3.hs.lastMessageFlight);
 		    dtls_CancelTimer(ss);
 
 		    /* If there have been no retries this time, reset the
@@ -497,7 +465,7 @@
 	PORT_SetError(SEC_ERROR_NO_MEMORY);
 	rv = SECFailure;
     } else {
-	PR_APPEND_LINK(&msg->link, ss->ssl3.hs.lastMessageFlight);
+	PR_APPEND_LINK(&msg->link, &ss->ssl3.hs.lastMessageFlight);
     }
 
     return rv;
@@ -623,8 +591,8 @@
      * stuffed something in ss->pendingBuf
      */
     PORT_Assert(!ss->pendingBuf.len);
-    for (msg_p = PR_LIST_HEAD(ss->ssl3.hs.lastMessageFlight);
-	 msg_p != ss->ssl3.hs.lastMessageFlight;
+    for (msg_p = PR_LIST_HEAD(&ss->ssl3.hs.lastMessageFlight);
+	 msg_p != &ss->ssl3.hs.lastMessageFlight;
 	 msg_p = PR_NEXT_LINK(msg_p)) {
         DTLSQueuedMessage *msg = (DTLSQueuedMessage *)msg_p;

Powered by Google App Engine
This is Rietveld 408576698