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

Unified Diff: net/third_party/nss/patches/secretexporterlocks.patch

Issue 1844813002: Uprev NSS to 3.23 on iOS (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: One more GN fix Created 4 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
« no previous file with comments | « net/third_party/nss/patches/secitemarray.patch ('k') | net/third_party/nss/patches/sessioncache.patch » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/third_party/nss/patches/secretexporterlocks.patch
diff --git a/net/third_party/nss/patches/secretexporterlocks.patch b/net/third_party/nss/patches/secretexporterlocks.patch
index 232b93237fb3fd22a566fbeb6d88603a0832c35d..a95ef278d0ef428bfc2637d521c802cc5dcdf651 100644
--- a/net/third_party/nss/patches/secretexporterlocks.patch
+++ b/net/third_party/nss/patches/secretexporterlocks.patch
@@ -1,41 +1,32 @@
diff --git a/lib/ssl/sslinfo.c b/lib/ssl/sslinfo.c
-index 216ab0f..7048eb8 100644
+index 527b1a4..c59879c 100644
--- a/lib/ssl/sslinfo.c
+++ b/lib/ssl/sslinfo.c
-@@ -387,8 +387,13 @@ SSL_ExportKeyingMaterial(PRFileDesc *fd,
- return SECFailure;
+@@ -406,8 +406,13 @@ SSL_ExportKeyingMaterial(PRFileDesc *fd,
+ return SECFailure;
}
+ ssl_GetRecvBufLock(ss);
+ ssl_GetSSL3HandshakeLock(ss);
+
if (ss->version < SSL_LIBRARY_VERSION_3_1_TLS) {
- PORT_SetError(SSL_ERROR_FEATURE_NOT_SUPPORTED_FOR_VERSION);
-+ ssl_ReleaseSSL3HandshakeLock(ss);
-+ ssl_ReleaseRecvBufLock(ss);
- return SECFailure;
+ PORT_SetError(SSL_ERROR_FEATURE_NOT_SUPPORTED_FOR_VERSION);
++ ssl_ReleaseSSL3HandshakeLock(ss);
++ ssl_ReleaseRecvBufLock(ss);
+ return SECFailure;
}
-@@ -399,13 +404,17 @@ SSL_ExportKeyingMaterial(PRFileDesc *fd,
+@@ -418,6 +423,8 @@ SSL_ExportKeyingMaterial(PRFileDesc *fd,
}
val = PORT_Alloc(valLen);
if (!val) {
-+ ssl_ReleaseSSL3HandshakeLock(ss);
-+ ssl_ReleaseRecvBufLock(ss);
- return SECFailure;
++ ssl_ReleaseSSL3HandshakeLock(ss);
++ ssl_ReleaseRecvBufLock(ss);
+ return SECFailure;
}
i = 0;
-+
- PORT_Memcpy(val + i, &ss->ssl3.hs.client_random.rand, SSL3_RANDOM_LENGTH);
- i += SSL3_RANDOM_LENGTH;
- PORT_Memcpy(val + i, &ss->ssl3.hs.server_random.rand, SSL3_RANDOM_LENGTH);
- i += SSL3_RANDOM_LENGTH;
-+
- if (hasContext) {
- val[i++] = contextLen >> 8;
- val[i++] = contextLen;
-@@ -426,6 +435,8 @@ SSL_ExportKeyingMaterial(PRFileDesc *fd,
- valLen, out, outLen);
+@@ -445,6 +452,8 @@ SSL_ExportKeyingMaterial(PRFileDesc *fd,
+ valLen, out, outLen);
}
ssl_ReleaseSpecReadLock(ss);
+ ssl_ReleaseSSL3HandshakeLock(ss);
« no previous file with comments | « net/third_party/nss/patches/secitemarray.patch ('k') | net/third_party/nss/patches/sessioncache.patch » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698