Fix instability in SSL client/server sockets

net/socket/ssl_client_socket_nss.cc

Index: net/socket/ssl_client_socket_nss.cc
diff --git a/net/socket/ssl_client_socket_nss.cc b/net/socket/ssl_client_socket_nss.cc
index e827f398db65eb56a946e5c1ed286f455ccccb12..71afdf09f5928abec1f8d0ad5f2e720e7ea7b345 100644
--- a/net/socket/ssl_client_socket_nss.cc
+++ b/net/socket/ssl_client_socket_nss.cc
@@ -1740,9 +1740,17 @@ bool SSLClientSocketNSS::DoTransportIO() {
   EnterFunction("");
   bool network_moved = false;
   if (nss_bufs_ != NULL) {

[Wez, 2011/07/19 20:33:07]

nit: Actually, a comment here to explain the need for the two loops would help.

[Sergey Ulanov, 2011/07/19 21:10:10]

Done.

-    int nsent = BufferSend();
-    int nreceived = BufferRecv();
-    network_moved = (nsent > 0 || nreceived >= 0);
+    int rv;
+    do {
+      rv = BufferSend();
+      if (rv > 0)
+        network_moved = true;
+    } while (rv > 0);
+    do {
+      rv = BufferRecv();
+      if (rv >= 0)
+        network_moved = true;
+    } while (rv > 0);
   }
   LeaveFunction(network_moved);
   return network_moved;