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

Unified Diff: net/third_party/nss/ssl/ssl3con.c

Issue 7583036: Revert "net: Precede each CBC encrypted application data record with an empty one." (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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: net/third_party/nss/ssl/ssl3con.c
diff --git a/net/third_party/nss/ssl/ssl3con.c b/net/third_party/nss/ssl/ssl3con.c
index fd34b9f7cf5cda381d9e3571df3e282751520065..3f3e7fd7aeb9d78055de5075cbd5f002f09847be 100644
--- a/net/third_party/nss/ssl/ssl3con.c
+++ b/net/third_party/nss/ssl/ssl3con.c
@@ -2229,7 +2229,7 @@ ssl3_SendRecord( sslSocket * ss,
return SECFailure;
}
- do {
+ while (nIn > 0) {
PRUint32 contentLen = PR_MIN(nIn, MAX_FRAGMENT_LENGTH);
if (wrBuf->space < contentLen + SSL3_BUFFER_FUDGE) {
@@ -2306,7 +2306,7 @@ ssl3_SendRecord( sslSocket * ss,
}
}
totalSent += contentLen;
- } while (nIn > 0);
+ }
return totalSent;
}
@@ -2321,7 +2321,6 @@ ssl3_SendApplicationData(sslSocket *ss, const unsigned char *in,
{
PRInt32 totalSent = 0;
PRInt32 discarded = 0;
- PRBool isBlockCipher;
PORT_Assert( ss->opt.noLocks || ssl_HaveXmitBufLock(ss) );
if (len < 0 || !in) {
@@ -2346,28 +2345,6 @@ ssl3_SendApplicationData(sslSocket *ss, const unsigned char *in,
len--;
discarded = 1;
}
-
- ssl_GetSpecReadLock(ss);
- isBlockCipher = ss->ssl3.cwSpec->cipher_def->type == type_block;
- ssl_ReleaseSpecReadLock(ss);
-
- if (isBlockCipher && len > 0) {
- // We assume that block ciphers are used in CBC mode and prepend an
- // empty record. This effectively randomizes the IV in a backwards
- // compatible way.
- PRInt32 sent = ssl3_SendRecord(ss, content_application_data,
- in, 0 /* no payload */, flags);
- if (sent < 0) {
- return SECFailure; /* error code set by ssl3_SendRecord */
- }
- if (ss->pendingBuf.len) {
- /* must be a non-blocking socket */
- PORT_Assert(!ssl_SocketIsBlocking(ss));
- PORT_Assert(ss->lastWriteBlocked);
- return SECFailure;
- }
- }
-
while (len > totalSent) {
PRInt32 sent, toSend;
@@ -2400,7 +2377,6 @@ ssl3_SendApplicationData(sslSocket *ss, const unsigned char *in,
break;
}
}
-
if (ss->pendingBuf.len) {
/* Must be non-blocking. */
PORT_Assert(!ssl_SocketIsBlocking(ss));
« net/socket/ssl_client_socket_unittest.cc ('K') | « net/third_party/nss/patches/cbcrandomiv.patch ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698