Fix some grammar and converge formatting of the two versions of TcpInput.c

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@17186 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Input.c b/MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Input.c
index 2e87b3fc7aff7b0c94dff00631a717db56e62a36..209127d18f7ea49c895b15aaff770fb2ad8ebccd 100644 (file)
--- a/MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Input.c
+++ b/MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Input.c
@@ -1,7 +1,7 @@
 /** @file\r
   TCP input process routines.\r
 \r
-Copyright (c) 2005 - 2010, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2005 - 2015, Intel Corporation. All rights reserved.<BR>\r
 This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
@@ -61,7 +61,7 @@ TcpFastRecover (
     //\r
     // Step 1A: Invoking fast retransmission.\r
     //\r
-    FlightSize = TCP_SUB_SEQ (Tcb->SndNxt, Tcb->SndUna);\r
+    FlightSize        = TCP_SUB_SEQ (Tcb->SndNxt, Tcb->SndUna);\r
 \r
     Tcb->Ssthresh     = MAX (FlightSize >> 1, (UINT32) (2 * Tcb->SndMss));\r
     Tcb->Recover      = Tcb->SndNxt;\r
@@ -109,7 +109,7 @@ TcpFastRecover (
       // Step 5 - Full ACK:\r
       // deflate the congestion window, and exit fast recovery\r
       //\r
-      FlightSize = TCP_SUB_SEQ (Tcb->SndNxt, Tcb->SndUna);\r
+      FlightSize        = TCP_SUB_SEQ (Tcb->SndNxt, Tcb->SndUna);\r
 \r
       Tcb->CWnd         = MIN (Tcb->Ssthresh, FlightSize + Tcb->SndMss);\r
 \r
@@ -556,15 +556,15 @@ TcpQueueData (
   if (IsListEmpty (Head)) {\r
 \r
     InsertTailList (Head, &Nbuf->List);\r
-    return ;\r
+    return;\r
   }\r
 \r
   //\r
   // Find the point to insert the buffer\r
   //\r
   for (Prev = Head, Cur = Head->ForwardLink;\r
-      Cur != Head;\r
-      Prev = Cur, Cur = Cur->ForwardLink) {\r
+       Cur != Head;\r
+       Prev = Cur, Cur = Cur->ForwardLink) {\r
 \r
     Node = NET_LIST_USER_STRUCT (Cur, NET_BUF, List);\r
 \r
@@ -585,7 +585,7 @@ TcpQueueData (
       if (TCP_SEQ_LEQ (Seg->End, TCPSEG_NETBUF (Node)->End)) {\r
 \r
         NetbufFree (Nbuf);\r
-        return ;\r
+        return;\r
       }\r
 \r
       TcpTrimSegment (Nbuf, TCPSEG_NETBUF (Node)->End, Seg->End);\r
@@ -737,7 +737,7 @@ TcpInput (
           );\r
 \r
   if ((Tcb == NULL) || (Tcb->State == TCP_CLOSED)) {\r
-    DEBUG ((EFI_D_INFO, "TcpInput: send reset because no TCB find\n"));\r
+    DEBUG ((EFI_D_INFO, "TcpInput: send reset because no TCB found\n"));\r
 \r
     Tcb = NULL;\r
     goto SEND_RESET;\r
@@ -751,7 +751,7 @@ TcpInput (
   //\r
   if (TcpParseOption (Nbuf->Tcp, &Option) == -1) {\r
     DEBUG ((EFI_D_ERROR, "TcpInput: reset the peer because"\r
-      " of mal-format option for Tcb %p\n", Tcb));\r
+      " of malformed option for TCB %p\n", Tcb));\r
 \r
     goto SEND_RESET;\r
   }\r
@@ -799,7 +799,7 @@ TcpInput (
       Tcb     = TcpCloneTcb (Parent);\r
       if (Tcb == NULL) {\r
         DEBUG ((EFI_D_ERROR, "TcpInput: discard a segment because"\r
-          " failed to clone a child for TCB%x\n", Tcb));\r
+          " failed to clone a child for TCB %p\n", Tcb));\r
 \r
         goto DISCARD;\r
       }\r
@@ -865,7 +865,7 @@ TcpInput (
     //\r
 \r
     //\r
-    // Fourth step: Check SYN. Pay attention to sitimulatous open\r
+    // Fourth step: Check SYN. Pay attention to simultaneous open\r
     //\r
     if (TCP_FLG_ON (Seg->Flag, TCP_FLG_SYN)) {\r
 \r
@@ -902,7 +902,7 @@ TcpInput (
         goto StepSix;\r
       } else {\r
         //\r
-        // Received a SYN segment without ACK, simultanous open.\r
+        // Received a SYN segment without ACK, simultaneous open.\r
         //\r
         TcpSetState (Tcb, TCP_SYN_RCVD);\r
 \r
@@ -911,7 +911,7 @@ TcpInput (
 \r
         TcpTrimInWnd (Tcb, Nbuf);\r
 \r
-        DEBUG ((EFI_D_WARN, "TcpInput: simultanous open "\r
+        DEBUG ((EFI_D_WARN, "TcpInput: simultaneous open "\r
           "for TCB %p in SYN_SENT\n", Tcb));\r
 \r
         goto StepSix;\r
@@ -1290,7 +1290,8 @@ StepSix:
   TcpSetKeepaliveTimer (Tcb);\r
 \r
   if (TCP_FLG_ON (Seg->Flag, TCP_FLG_URG) &&\r
-      !TCP_FIN_RCVD (Tcb->State)) {\r
+      !TCP_FIN_RCVD (Tcb->State)) \r
+  {\r
 \r
     DEBUG ((EFI_D_INFO, "TcpInput: received urgent data "\r
       "from peer for connected TCB %p\n", Tcb));\r

diff --git a/NetworkPkg/TcpDxe/TcpInput.c b/NetworkPkg/TcpDxe/TcpInput.c
index e63469adb9e3f9c1d5b6fcd1d3ee736652b141d6..d0118f1d886d8bd128669047ae066d360ad50f60 100644 (file)
--- a/NetworkPkg/TcpDxe/TcpInput.c
+++ b/NetworkPkg/TcpDxe/TcpInput.c
@@ -1,7 +1,7 @@
 /** @file\r
   TCP input process routines.\r
 \r
-  Copyright (c) 2009 - 2010, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2009 - 2015, Intel Corporation. All rights reserved.<BR>\r
 \r
   This program and the accompanying materials\r
   are licensed and made available under the terms and conditions of the BSD License\r
@@ -531,8 +531,8 @@ TcpDeliverData (
       Urgent = 0;\r
 \r
       if (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_RCVD_URG) &&\r
-          TCP_SEQ_LEQ (Seg->Seq, Tcb->RcvUp)\r
-          ) {\r
+          TCP_SEQ_LEQ (Seg->Seq, Tcb->RcvUp))\r
+      {\r
 \r
         if (TCP_SEQ_LEQ (Seg->End, Tcb->RcvUp)) {\r
           Urgent = Nbuf->TotalSize;\r
@@ -596,8 +596,7 @@ TcpQueueData (
   //\r
   for (Prev = Head, Cur = Head->ForwardLink;\r
        Cur != Head;\r
-       Prev = Cur, Cur = Cur->ForwardLink\r
-       ) {\r
+       Prev = Cur, Cur = Cur->ForwardLink) {\r
 \r
     Node = NET_LIST_USER_STRUCT (Cur, NET_BUF, List);\r
 \r
@@ -753,7 +752,7 @@ TcpInput (
 \r
   if ((Head->HeadLen < 5) || (Len < 0)) {\r
 \r
-    DEBUG ((EFI_D_INFO, "TcpInput: received an mal-formated packet\n"));\r
+    DEBUG ((EFI_D_INFO, "TcpInput: received a malformed packet\n"));\r
     goto DISCARD;\r
   }\r
 \r
@@ -788,7 +787,7 @@ TcpInput (
           );\r
 \r
   if ((Tcb == NULL) || (Tcb->State == TCP_CLOSED)) {\r
-    DEBUG ((EFI_D_INFO, "TcpInput: send reset because no TCB find\n"));\r
+    DEBUG ((EFI_D_INFO, "TcpInput: send reset because no TCB found\n"));\r
 \r
     Tcb = NULL;\r
     goto SEND_RESET;\r
@@ -803,7 +802,7 @@ TcpInput (
   if (TcpParseOption (Nbuf->Tcp, &Option) == -1) {\r
     DEBUG (\r
       (EFI_D_ERROR,\r
-      "TcpInput: reset the peer because of mal-format option for Tcb %p\n",\r
+      "TcpInput: reset the peer because of malformed option for TCB %p\n",\r
       Tcb)\r
       );\r
 \r
@@ -860,7 +859,7 @@ TcpInput (
       if (Tcb == NULL) {\r
         DEBUG (\r
           (EFI_D_ERROR,\r
-          "TcpInput: discard a segment because failed to clone a child for TCB%p\n",\r
+          "TcpInput: discard a segment because failed to clone a child for TCB %p\n",\r
           Tcb)\r
           );\r
 \r
@@ -940,7 +939,7 @@ TcpInput (
     //\r
 \r
     //\r
-    // Fourth step: Check SYN. Pay attention to sitimulatous open\r
+    // Fourth step: Check SYN. Pay attention to simultaneous open\r
     //\r
     if (TCP_FLG_ON (Seg->Flag, TCP_FLG_SYN)) {\r
 \r
@@ -961,8 +960,8 @@ TcpInput (
         TcpDeliverData (Tcb);\r
 \r
         if ((Tcb->CongestState == TCP_CONGEST_OPEN) &&\r
-            TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_RTT_ON)\r
-            ) {\r
+            TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_RTT_ON))\r
+        {\r
 \r
           TcpComputeRtt (Tcb, Tcb->RttMeasure);\r
           TCP_CLEAR_FLG (Tcb->CtrlFlag, TCP_CTRL_RTT_ON);\r
@@ -992,7 +991,7 @@ TcpInput (
 \r
         DEBUG (\r
           (EFI_D_WARN,\r
-          "TcpInput: simultanous open for TCB %p in SYN_SENT\n",\r
+          "TcpInput: simultaneous open for TCB %p in SYN_SENT\n",\r
           Tcb)\r
           );\r
 \r
@@ -1034,8 +1033,8 @@ TcpInput (
 \r
   if ((TCP_SEQ_LT (Seg->Seq, Tcb->RcvWl2)) &&\r
       (Tcb->RcvWl2 == Seg->End) &&\r
-      !TCP_FLG_ON (Seg->Flag, TCP_FLG_SYN | TCP_FLG_FIN)\r
-        ) {\r
+      !TCP_FLG_ON (Seg->Flag, TCP_FLG_SYN | TCP_FLG_FIN))\r
+  {\r
 \r
     TCP_SET_FLG (Tcb->CtrlFlag, TCP_CTRL_ACK_NOW);\r
   }\r
@@ -1058,10 +1057,10 @@ TcpInput (
       // if it comes from a LISTEN TCB.\r
       //\r
     } else if ((Tcb->State == TCP_ESTABLISHED) ||\r
-             (Tcb->State == TCP_FIN_WAIT_1) ||\r
-             (Tcb->State == TCP_FIN_WAIT_2) ||\r
-             (Tcb->State == TCP_CLOSE_WAIT)\r
-            ) {\r
+               (Tcb->State == TCP_FIN_WAIT_1) ||\r
+               (Tcb->State == TCP_FIN_WAIT_2) ||\r
+               (Tcb->State == TCP_CLOSE_WAIT))\r
+    {\r
 \r
       SOCK_ERROR (Tcb->Sk, EFI_CONNECTION_RESET);\r
 \r
@@ -1114,7 +1113,9 @@ TcpInput (
 \r
   if (Tcb->State == TCP_SYN_RCVD) {\r
 \r
-    if (TCP_SEQ_LT (Tcb->SndUna, Seg->Ack) && TCP_SEQ_LEQ (Seg->Ack, Tcb->SndNxt)) {\r
+    if (TCP_SEQ_LT (Tcb->SndUna, Seg->Ack) &&\r
+        TCP_SEQ_LEQ (Seg->Ack, Tcb->SndNxt))\r
+    {\r
 \r
       Tcb->SndWnd     = Seg->Wnd;\r
       Tcb->SndWndMax  = MAX (Tcb->SndWnd, Tcb->SndWndMax);\r
@@ -1127,7 +1128,7 @@ TcpInput (
 \r
       DEBUG (\r
         (EFI_D_INFO,\r
-        "TcpInput: connection established  for TCB %p in SYN_RCVD\n",\r
+        "TcpInput: connection established for TCB %p in SYN_RCVD\n",\r
         Tcb)\r
         );\r
 \r
@@ -1176,7 +1177,9 @@ TcpInput (
     // RcvWl2 equals to the variable "LastAckSent"\r
     // defined there.\r
     //\r
-    if (TCP_SEQ_LEQ (Seg->Seq, Tcb->RcvWl2) && TCP_SEQ_LT (Tcb->RcvWl2, Seg->End)) {\r
+    if (TCP_SEQ_LEQ (Seg->Seq, Tcb->RcvWl2) &&\r
+        TCP_SEQ_LT (Tcb->RcvWl2, Seg->End))\r
+    {\r
 \r
       Tcb->TsRecent     = Option.TSVal;\r
       Tcb->TsRecentAge  = mTcpTick;\r
@@ -1206,8 +1209,8 @@ TcpInput (
   if ((Seg->Ack == Tcb->SndUna) &&\r
       (Tcb->SndUna != Tcb->SndNxt) &&\r
       (Seg->Wnd == Tcb->SndWnd) &&\r
-      (0 == Len)\r
-      ) {\r
+      (0 == Len))\r
+  {\r
 \r
     Tcb->DupAck++;\r
   } else {\r
@@ -1219,8 +1222,8 @@ TcpInput (
   // Congestion avoidance, fast recovery and fast retransmission.\r
   //\r
   if (((Tcb->CongestState == TCP_CONGEST_OPEN) && (Tcb->DupAck < 3)) ||\r
-      (Tcb->CongestState == TCP_CONGEST_LOSS)\r
-      ) {\r
+      (Tcb->CongestState == TCP_CONGEST_LOSS))\r
+  {\r
 \r
     if (TCP_SEQ_GT (Seg->Ack, Tcb->SndUna)) {\r
 \r
@@ -1249,8 +1252,8 @@ TcpInput (
     Tcb->SndUna = Seg->Ack;\r
 \r
     if (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_SND_URG) &&\r
-        TCP_SEQ_LT (Tcb->SndUp, Seg->Ack)\r
-        ) {\r
+        TCP_SEQ_LT (Tcb->SndUp, Seg->Ack))\r
+    {\r
 \r
       TCP_CLEAR_FLG (Tcb->CtrlFlag, TCP_CTRL_SND_URG);\r
     }\r
@@ -1260,8 +1263,8 @@ TcpInput (
   // Update window info\r
   //\r
   if (TCP_SEQ_LT (Tcb->SndWl1, Seg->Seq) ||\r
-      ((Tcb->SndWl1 == Seg->Seq) && TCP_SEQ_LEQ (Tcb->SndWl2, Seg->Ack))\r
-      ) {\r
+      ((Tcb->SndWl1 == Seg->Seq) && TCP_SEQ_LEQ (Tcb->SndWl2, Seg->Ack)))\r
+  {\r
 \r
     Right = Seg->Ack + Seg->Wnd;\r
 \r
@@ -1269,8 +1272,8 @@ TcpInput (
 \r
       if ((Tcb->SndWl1 == Seg->Seq) &&\r
           (Tcb->SndWl2 == Seg->Ack) &&\r
-          (Len == 0)\r
-          ) {\r
+          (Len == 0))\r
+      {\r
 \r
         goto NO_UPDATE;\r
       }\r
@@ -1281,12 +1284,16 @@ TcpInput (
         Tcb)\r
         );\r
 \r
-      if ((Tcb->CongestState == TCP_CONGEST_RECOVER) && (TCP_SEQ_LT (Right, Tcb->Recover))) {\r
+      if ((Tcb->CongestState == TCP_CONGEST_RECOVER) &&\r
+          (TCP_SEQ_LT (Right, Tcb->Recover)))\r
+      {\r
 \r
         Tcb->Recover = Right;\r
       }\r
 \r
-      if ((Tcb->CongestState == TCP_CONGEST_LOSS) && (TCP_SEQ_LT (Right, Tcb->LossRecover))) {\r
+      if ((Tcb->CongestState == TCP_CONGEST_LOSS) &&\r
+          (TCP_SEQ_LT (Right, Tcb->LossRecover)))\r
+      {\r
 \r
         Tcb->LossRecover = Right;\r
       }\r
@@ -1311,7 +1318,9 @@ TcpInput (
 \r
 NO_UPDATE:\r
 \r
-  if (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_FIN_SENT) && (Tcb->SndUna == Tcb->SndNxt)) {\r
+  if (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_FIN_SENT) &&\r
+      (Tcb->SndUna == Tcb->SndNxt))\r
+  {\r
 \r
     DEBUG (\r
       (EFI_D_INFO,\r
@@ -1417,7 +1426,9 @@ StepSix:
 \r
     Urg = Seg->Seq + Seg->Urg;\r
 \r
-    if (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_RCVD_URG) && TCP_SEQ_GT (Urg, Tcb->RcvUp)) {\r
+    if (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_RCVD_URG) &&\r
+        TCP_SEQ_GT (Urg, Tcb->RcvUp))\r
+    {\r
 \r
       Tcb->RcvUp = Urg;\r
     } else {\r
@@ -1480,8 +1491,8 @@ StepSix:
 \r
   if ((Tcb->State != TCP_CLOSED) &&\r
       (TcpToSendData (Tcb, 0) == 0) &&\r
-      (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_ACK_NOW) || (Nbuf->TotalSize != 0))\r
-      ) {\r
+      (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_ACK_NOW) || (Nbuf->TotalSize != 0)))\r
+  {\r
 \r
     TcpToSendAck (Tcb);\r
   }\r
@@ -1574,7 +1585,12 @@ TcpIcmpInput (
     goto CLEAN_EXIT;\r
   }\r
 \r
-  IcmpErrStatus = IpIoGetIcmpErrStatus (IcmpErr, Tcb->Sk->IpVersion, &IcmpErrIsHard, &IcmpErrNotify);\r
+  IcmpErrStatus = IpIoGetIcmpErrStatus (\r
+                    IcmpErr,\r
+                    Tcb->Sk->IpVersion,\r
+                    &IcmpErrIsHard,\r
+                    &IcmpErrNotify\r
+                    );\r
 \r
   if (IcmpErrNotify) {\r
 \r