]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Network/IScsiDxe/IScsiProto.c
Add transit bit check for detecting partial response in iSCSI stack.
[mirror_edk2.git] / MdeModulePkg / Universal / Network / IScsiDxe / IScsiProto.c
index 860755839791b833b3b5a0188fbb9fea3e27d025..aff036bb7afe67173d931c4302b35e2aeaaff442 100644 (file)
@@ -1,7 +1,8 @@
-/*++\r
+/** @file\r
+  The implementation of iSCSI protocol based on RFC3720.\r
 \r
-Copyright (c) 2004 - 2007, Intel Corporation\r
-All rights reserved. This program and the accompanying materials\r
+Copyright (c) 2004 - 2013, 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
 http://opensource.org/licenses/bsd-license.php\r
@@ -9,91 +10,59 @@ http://opensource.org/licenses/bsd-license.php
 THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
 WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
-Module Name:\r
-\r
-  IScsiProto.c\r
-\r
-Abstract:\r
-\r
---*/\r
+**/\r
 \r
 #include "IScsiImpl.h"\r
 \r
-static UINT32 mDataSegPad = 0;\r
+UINT32 mDataSegPad = 0;\r
+\r
+/**\r
+  Attach the iSCSI connection to the iSCSI session. \r
 \r
+  @param[in, out]  Session The iSCSI session.\r
+  @param[in, out]  Conn    The iSCSI connection.\r
+**/\r
 VOID\r
 IScsiAttatchConnection (\r
-  IN ISCSI_SESSION     *Session,\r
-  IN ISCSI_CONNECTION  *Conn\r
+  IN OUT ISCSI_SESSION     *Session,\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Attach the iSCSI connection to the iSCSI session. \r
-\r
-Arguments:\r
-\r
-  Session - The iSCSI session.\r
-  Conn    - The iSCSI connection.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
   InsertTailList (&Session->Conns, &Conn->Link);\r
   Conn->Session = Session;\r
   Session->NumConns++;\r
 }\r
 \r
+/**\r
+  Detach the iSCSI connection from the session it belongs to. \r
+\r
+  @param[in, out]  Conn The iSCSI connection.\r
+**/\r
 VOID\r
 IScsiDetatchConnection (\r
-  IN ISCSI_CONNECTION  *Conn\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Detach the iSCSI connection from the session it belongs to. \r
-\r
-Arguments:\r
-\r
-  Conn - The iSCSI connection.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
   RemoveEntryList (&Conn->Link);\r
   Conn->Session->NumConns--;\r
   Conn->Session = NULL;\r
 }\r
 \r
-EFI_STATUS\r
-IScsiCheckSN (\r
-  IN UINT32  *ExpSN,\r
-  IN UINT32  NewSN\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Check the sequence number according to RFC3720. \r
 \r
-Arguments:\r
-\r
-  ExpSN - The currently expected sequence number.\r
-  NewSN - The sequence number to check.\r
-\r
-Returns:\r
+  @param[in, out]  ExpSN   The currently expected sequence number.\r
+  @param[in]       NewSN   The sequence number to check.\r
 \r
-  EFI_SUCCESS - The check passed and the ExpSN is increased.\r
-\r
---*/\r
+  @retval EFI_SUCCESS         The check passed and the ExpSN is increased.\r
+  @retval EFI_NOT_READY       Response was sent due to a retransmission request.\r
+  @retval EFI_PROTOCOL_ERROR  Some kind of iSCSI protocol error happened.\r
+**/\r
+EFI_STATUS\r
+IScsiCheckSN (\r
+  IN OUT UINT32  *ExpSN,\r
+  IN UINT32      NewSN\r
+  )\r
 {\r
   if (!ISCSI_SEQ_EQ (NewSN, *ExpSN)) {\r
     if (ISCSI_SEQ_LT (NewSN, *ExpSN)) {\r
@@ -113,29 +82,19 @@ Returns:
   }\r
 }\r
 \r
+/**\r
+  Update the sequence numbers for the iSCSI command.\r
+\r
+  @param[in, out]  Session  The iSCSI session.\r
+  @param[in]       MaxCmdSN Maximum CmdSN from the target.\r
+  @param[in]       ExpCmdSN Next expected CmdSN from the target.\r
+**/\r
 VOID\r
 IScsiUpdateCmdSN (\r
-  IN ISCSI_SESSION  *Session,\r
-  IN UINT32         MaxCmdSN,\r
-  IN UINT32         ExpCmdSN\r
+  IN OUT ISCSI_SESSION  *Session,\r
+  IN UINT32             MaxCmdSN,\r
+  IN UINT32             ExpCmdSN\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Update the sequence numbers for the iSCSI command.\r
-\r
-Arguments:\r
-\r
-  Session  - The iSCSI session.\r
-  MaxCmdSN - Maximum CmdSN from the target.\r
-  ExpCmdSN - Next expected CmdSN from the target.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
   if (ISCSI_SEQ_LT (MaxCmdSN, ExpCmdSN - 1)) {\r
     return ;\r
@@ -150,27 +109,19 @@ Returns:
   }\r
 }\r
 \r
-EFI_STATUS\r
-IScsiConnLogin (\r
-  IN ISCSI_CONNECTION  *Conn\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   This function does the iSCSI connection login.\r
 \r
-Arguments:\r
+  @param[in, out]  Conn      The iSCSI connection to login.\r
 \r
-  Conn - The iSCSI connection to login.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCESS        - The iSCSI connection is logged into the iSCSI target.\r
-  EFI_TIMEOUT        - Timeout happened during the login procedure.\r
-  EFI_PROTOCOL_ERROR - Some kind of iSCSI protocol error happened.\r
-\r
---*/\r
+  @retval EFI_SUCCESS        The iSCSI connection is logged into the iSCSI target.\r
+  @retval EFI_TIMEOUT        Timeout happened during the login procedure.\r
+  @retval Others             Other errors as indicated.  \r
+**/\r
+EFI_STATUS\r
+IScsiConnLogin (\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
+  )\r
 {\r
   EFI_STATUS  Status;\r
 \r
@@ -210,50 +161,32 @@ Returns:
   return Status;\r
 }\r
 \r
+/**\r
+  Reset the iSCSI connection.\r
+\r
+  @param[in, out]  Conn The iSCSI connection to reset.\r
+**/\r
 VOID\r
 IScsiConnReset (\r
-  IN ISCSI_CONNECTION  *Conn\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Reset the iSCSI connection.\r
-\r
-Arguments:\r
-\r
-  Conn - The iSCSI connection to reset.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
   Tcp4IoReset (&Conn->Tcp4Io);\r
 }\r
 \r
+/**\r
+  Create a TCP connection for the iSCSI session.\r
+\r
+  @param[in]  Private The iSCSI driver data.\r
+  @param[in]  Session Maximum CmdSN from the target.\r
+\r
+  @return The newly created iSCSI connection.\r
+**/\r
 ISCSI_CONNECTION *\r
 IScsiCreateConnection (\r
   IN ISCSI_DRIVER_DATA  *Private,\r
   IN ISCSI_SESSION      *Session\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Create a TCP connection for the iSCSI session.\r
-\r
-Arguments:\r
-\r
-  Private - The iSCSI driver data.\r
-  Session - Maximum CmdSN from the target.\r
-\r
-Returns:\r
-\r
-  The newly created iSCSI connection.\r
-\r
---*/\r
 {\r
   ISCSI_CONNECTION    *Conn;\r
   TCP4_IO_CONFIG_DATA Tcp4IoConfig;\r
@@ -272,17 +205,17 @@ Returns:
   Conn->ExpStatSN       = 0;\r
   Conn->PartialReqSent  = FALSE;\r
   Conn->PartialRspRcvd  = FALSE;\r
-  Conn->CID             = Session->NextCID++;\r
+  Conn->Cid             = Session->NextCid++;\r
 \r
   Status = gBS->CreateEvent (\r
-                  EFI_EVENT_TIMER,\r
+                  EVT_TIMER,\r
                   TPL_CALLBACK,\r
                   NULL,\r
                   NULL,\r
                   &Conn->TimeoutEvent\r
                   );\r
   if (EFI_ERROR (Status)) {\r
-    gBS->FreePool (Conn);\r
+    FreePool (Conn);\r
     return NULL;\r
   }\r
 \r
@@ -291,9 +224,9 @@ Returns:
   //\r
   // set the default connection-only parameters\r
   //\r
-  Conn->MaxRecvDataSegmentLength  = MAX_RECV_DATA_SEG_LEN_IN_FFP;\r
-  Conn->HeaderDigest              = ISCSI_DIGEST_NONE;\r
-  Conn->DataDigest                = ISCSI_DIGEST_NONE;\r
+  Conn->MaxRecvDataSegmentLength  = DEFAULT_MAX_RECV_DATA_SEG_LEN;\r
+  Conn->HeaderDigest              = IScsiDigestNone;\r
+  Conn->DataDigest                = IScsiDigestNone;\r
 \r
   CopyMem (&Tcp4IoConfig.LocalIp, &Session->ConfigData.NvData.LocalIp, sizeof (EFI_IPv4_ADDRESS));\r
   CopyMem (&Tcp4IoConfig.SubnetMask, &Session->ConfigData.NvData.SubnetMask, sizeof (EFI_IPv4_ADDRESS));\r
@@ -313,68 +246,67 @@ Returns:
             );\r
   if (EFI_ERROR (Status)) {\r
     gBS->CloseEvent (Conn->TimeoutEvent);\r
-    gBS->FreePool (Conn);\r
+    FreePool (Conn);\r
     Conn = NULL;\r
   }\r
 \r
   return Conn;\r
 }\r
 \r
+/**\r
+  Destroy an iSCSI connection.\r
+\r
+  @param[in]  Conn The connection to destroy.\r
+**/\r
 VOID\r
 IScsiDestroyConnection (\r
   IN ISCSI_CONNECTION  *Conn\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Destroy an iSCSI connection.\r
-\r
-Arguments:\r
-\r
-  Conn - The connection to destroy.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
   Tcp4IoDestroySocket (&Conn->Tcp4Io);\r
   NetbufQueFlush (&Conn->RspQue);\r
   gBS->CloseEvent (Conn->TimeoutEvent);\r
-  gBS->FreePool (Conn);\r
+  FreePool (Conn);\r
 }\r
 \r
-EFI_STATUS\r
-IScsiSessionLogin (\r
-  IN ISCSI_DRIVER_DATA  *Private\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Login the iSCSI session.\r
 \r
-Arguments:\r
+  @param[in]  Private          The iSCSI driver data.\r
 \r
-  Private - The iSCSI driver data.\r
+  @retval EFI_SUCCESS          The iSCSI session login procedure finished.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval EFI_NO_MEDIA         There was a media error.\r
+  @retval Others               Other errors as indicated.\r
 \r
-Returns:\r
-\r
-  EFI_SUCCESS          - The iSCSI session login procedure finished.\r
-  EFI_OUT_OF_RESOURCES - Failed to allocate memory.\r
-  EFI_PROTOCOL_ERROR   - Some kind of iSCSI protocol error happened.\r
-\r
---*/\r
+**/\r
+EFI_STATUS\r
+IScsiSessionLogin (\r
+  IN ISCSI_DRIVER_DATA  *Private\r
+  )\r
 {\r
   EFI_STATUS        Status;\r
   ISCSI_SESSION     *Session;\r
   ISCSI_CONNECTION  *Conn;\r
   EFI_TCP4_PROTOCOL *Tcp4;\r
+  BOOLEAN           MediaPresent;\r
 \r
   Session = &Private->Session;\r
 \r
+  //\r
+  // Check media status before session login\r
+  //\r
+  MediaPresent = TRUE;\r
+  NetLibDetectMedia (Private->Controller, &MediaPresent);\r
+  if (!MediaPresent) {\r
+    return EFI_NO_MEDIA;\r
+  }\r
+\r
+  //\r
+  // Set session identifier\r
+  //\r
+  CopyMem (Session->Isid, Session->ConfigData.NvData.IsId, 6);\r
+\r
   //\r
   // Create a connection for the session.\r
   //\r
@@ -409,29 +341,21 @@ Returns:
   return Status;\r
 }\r
 \r
-EFI_STATUS\r
-IScsiSendLoginReq (\r
-  IN ISCSI_CONNECTION  *Conn\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Build and send the iSCSI login request to the iSCSI target according to\r
   the current login stage.\r
 \r
-Arguments:\r
-\r
-  Conn - The connection in the iSCSI login phase.\r
+  @param[in]  Conn             The connection in the iSCSI login phase.\r
 \r
-Returns:\r
-\r
-  EFI_SUCCESS          - The iSCSI login request PDU is built and sent on this\r
-                         connection.\r
-  EFI_OUT_OF_RESOURCES - Failed to allocate memory.\r
-  EFI_PROTOCOL_ERROR   - Some kind of iSCSI protocol error happened.\r
-\r
---*/\r
+  @retval EFI_SUCCESS          The iSCSI login request PDU is built and sent on this\r
+                               connection.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval EFI_DEVICE_ERROR     Some kind of device error happened.\r
+**/\r
+EFI_STATUS\r
+IScsiSendLoginReq (\r
+  IN ISCSI_CONNECTION  *Conn\r
+  )\r
 {\r
   NET_BUF     *Pdu;\r
   EFI_STATUS  Status;\r
@@ -453,27 +377,18 @@ Returns:
   return Status;\r
 }\r
 \r
+/**\r
+  Receive and process the iSCSI login response.\r
+\r
+  @param[in]  Conn             The connection in the iSCSI login phase.\r
+  \r
+  @retval EFI_SUCCESS          The iSCSI login response PDU is received and processed.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiReceiveLoginRsp (\r
   IN ISCSI_CONNECTION  *Conn\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Receive and process the iSCSI login response.\r
-\r
-Arguments:\r
-\r
-  Conn - The connection in the iSCSI login phase.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCESS          - The iSCSI login response PDU is received and processed.\r
-  EFI_OUT_OF_RESOURCES - Failed to allocate memory.\r
-  EFI_PROTOCOL_ERROR   - Some kind of iSCSI protocol error happened.\r
-\r
---*/\r
 {\r
   EFI_STATUS  Status;\r
   NET_BUF     *Pdu;\r
@@ -488,6 +403,7 @@ Returns:
   //\r
   // A Login Response is received, process it.\r
   //\r
+  ASSERT (Pdu != NULL);\r
   Status = IScsiProcessLoginRsp (Conn, Pdu);\r
 \r
   NetbufFree (Pdu);\r
@@ -495,34 +411,27 @@ Returns:
   return Status;\r
 }\r
 \r
-EFI_STATUS\r
-IScsiAddKeyValuePair (\r
-  IN NET_BUF          *Pdu,\r
-  IN CHAR8            *Key,\r
-  IN CHAR8            *Value\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Add an iSCSI key-value pair as a string into the data segment of the Login Request PDU.\r
   The DataSegmentLength and the actual size of the net buffer containing this PDU will be\r
   updated.\r
 \r
-Arguments:\r
-\r
-  Pdu   - The iSCSI PDU whose data segment the key-value pair will be added to.\r
-  Key   - The key name string.\r
-  Value - The value string.\r
+  @param[in, out]  Pdu         The iSCSI PDU whose data segment the key-value pair will\r
+                               be added to.\r
+  @param[in]       Key         The key name string.\r
+  @param[in]       Value       The value string.\r
 \r
-Returns:\r
-\r
-  EFI_SUCCESS          - The key-valu pair is added to the PDU's datasegment and\r
-                         the correspondence length fields are updated.\r
-  EFI_OUT_OF_RESOURCES - There is not enough space in the PDU to add the key-value\r
-                         pair.\r
-\r
---*/\r
+  @retval EFI_SUCCESS          The key-valu pair is added to the PDU's datasegment and\r
+                               the correspondence length fields are updated.\r
+  @retval EFI_OUT_OF_RESOURCES There is not enough space in the PDU to add the key-value\r
+                               pair.\r
+**/\r
+EFI_STATUS\r
+IScsiAddKeyValuePair (\r
+  IN OUT NET_BUF      *Pdu,\r
+  IN CHAR8            *Key,\r
+  IN CHAR8            *Value\r
+  )\r
 {\r
   UINT32              DataSegLen;\r
   UINT32              KeyLen;\r
@@ -532,6 +441,9 @@ Returns:
   CHAR8               *Data;\r
 \r
   LoginReq    = (ISCSI_LOGIN_REQUEST *) NetbufGetByte (Pdu, 0, NULL);\r
+  if (LoginReq == NULL) {\r
+    return EFI_PROTOCOL_ERROR;\r
+  }\r
   DataSegLen  = NTOH24 (LoginReq->DataSegmentLength);\r
 \r
   KeyLen      = (UINT32) AsciiStrLen (Key);\r
@@ -575,25 +487,18 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
-NET_BUF *\r
-IScsiPrepareLoginReq (\r
-  IN ISCSI_CONNECTION  *Conn\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Prepare the iSCSI login request to be sent according to the current login status.\r
 \r
-Arguments:\r
-\r
-  Conn - The connection in the iSCSI login phase.\r
+  @param[in, out]  Conn The connection in the iSCSI login phase.\r
 \r
-Returns:\r
-\r
-  The pointer to the net buffer containing the iSCSI login request built.\r
-\r
---*/\r
+  @return The pointer to the net buffer containing the iSCSI login request built.\r
+  @retval Others    Other errors as indicated.\r
+**/\r
+NET_BUF *\r
+IScsiPrepareLoginReq (\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
+  )\r
 {\r
   ISCSI_SESSION       *Session;\r
   NET_BUF             *Nbuf;\r
@@ -608,6 +513,7 @@ Returns:
   }\r
 \r
   LoginReq = (ISCSI_LOGIN_REQUEST *) NetbufAllocSpace (Nbuf, sizeof (ISCSI_LOGIN_REQUEST), NET_BUF_TAIL);\r
+  ASSERT (LoginReq != NULL);\r
   ZeroMem (LoginReq, sizeof (ISCSI_LOGIN_REQUEST));\r
 \r
   //\r
@@ -617,9 +523,9 @@ Returns:
   ISCSI_SET_STAGES (LoginReq, Conn->CurrentStage, Conn->NextStage);\r
   LoginReq->VersionMax        = ISCSI_VERSION_MAX;\r
   LoginReq->VersionMin        = ISCSI_VERSION_MIN;\r
-  LoginReq->TSIH              = HTONS (Session->TSIH);\r
+  LoginReq->Tsih              = HTONS (Session->Tsih);\r
   LoginReq->InitiatorTaskTag  = HTONL (Session->InitiatorTaskTag);\r
-  LoginReq->CID               = HTONS (Conn->CID);\r
+  LoginReq->Cid               = HTONS (Conn->Cid);\r
   LoginReq->CmdSN             = HTONL (Session->CmdSN);\r
 \r
   //\r
@@ -630,7 +536,7 @@ Returns:
   // with their increasing StatSN values.\r
   //\r
   LoginReq->ExpStatSN = HTONL (Conn->ExpStatSN);\r
-  CopyMem (LoginReq->ISID, Session->ISID, sizeof (LoginReq->ISID));\r
+  CopyMem (LoginReq->Isid, Session->Isid, sizeof (LoginReq->Isid));\r
 \r
   if (Conn->PartialRspRcvd) {\r
     //\r
@@ -674,28 +580,22 @@ Returns:
   return Nbuf;\r
 }\r
 \r
-EFI_STATUS\r
-IScsiProcessLoginRsp (\r
-  IN ISCSI_CONNECTION  *Conn,\r
-  IN NET_BUF           *Pdu\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Process the iSCSI Login Response.\r
 \r
-Arguments:\r
-\r
-  Conn - The connection on which the iSCSI login response is received.\r
-  Pdu  - The iSCSI login response PDU.\r
+  @param[in, out]  Conn The connection on which the iSCSI login response is received.\r
+  @param[in, out]  Pdu  The iSCSI login response PDU.\r
 \r
-Returns:\r
-\r
-  EFI_SUCCESS        - The iSCSI login response PDU is processed and all check are passed.\r
-  EFI_PROTOCOL_ERROR - Some kind of iSCSI protocol error happened.\r
-\r
---*/\r
+  @retval EFI_SUCCESS        The iSCSI login response PDU is processed and all check are passed.\r
+  @retval EFI_PROTOCOL_ERROR Some kind of iSCSI protocol error happened.\r
+  @retval EFI_MEDIA_CHANGED  Target is redirected.\r
+  @retval Others             Other errors as indicated.\r
+**/\r
+EFI_STATUS\r
+IScsiProcessLoginRsp (\r
+  IN OUT ISCSI_CONNECTION  *Conn,\r
+  IN OUT NET_BUF           *Pdu\r
+  )\r
 {\r
   EFI_STATUS            Status;\r
   ISCSI_SESSION         *Session;\r
@@ -710,6 +610,9 @@ Returns:
   Session   = Conn->Session;\r
 \r
   LoginRsp  = (ISCSI_LOGIN_RESPONSE *) NetbufGetByte (Pdu, 0, NULL);\r
+  if (LoginRsp == NULL) {\r
+    return EFI_PROTOCOL_ERROR;\r
+  }\r
   if (!ISCSI_CHECK_OPCODE (LoginRsp, ISCSI_OPCODE_LOGIN_RSP)) {\r
     //\r
     // It's not a Login Response\r
@@ -777,7 +680,7 @@ Returns:
       (CurrentStage != Conn->CurrentStage) ||\r
       (!Conn->TransitInitiated && Transit) ||\r
       (Transit && (NextStage != Conn->NextStage)) ||\r
-      (CompareMem (Session->ISID, LoginRsp->ISID, sizeof (LoginRsp->ISID)) != 0) ||\r
+      (CompareMem (Session->Isid, LoginRsp->Isid, sizeof (LoginRsp->Isid)) != 0) ||\r
       (LoginRsp->InitiatorTaskTag != Session->InitiatorTaskTag)\r
       ) {\r
     //\r
@@ -796,6 +699,14 @@ Returns:
   LoginRsp->MaxCmdSN  = NTOHL (LoginRsp->MaxCmdSN);\r
 \r
   if ((Conn->CurrentStage == ISCSI_SECURITY_NEGOTIATION) && (Conn->CHAPStep == ISCSI_CHAP_INITIAL)) {\r
+    //\r
+    // If the Login Request is a leading Login Request, the target MUST use\r
+    // the value presented in CmdSN as the target value for ExpCmdSN.\r
+    //\r
+    if ((Session->State == SESSION_STATE_FREE) && (Session->CmdSN != LoginRsp->ExpCmdSN)) {\r
+      return EFI_PROTOCOL_ERROR;\r
+    }\r
+\r
     //\r
     // It's the initial Login Response, initialize the local ExpStatSN, MaxCmdSN\r
     // and ExpCmdSN.\r
@@ -831,7 +742,7 @@ Returns:
   NetbufQueAppend (&Conn->RspQue, Pdu);\r
 \r
   Conn->PartialRspRcvd = Continue;\r
-  if (Continue) {\r
+  if ((!Transit) || Continue) {\r
     //\r
     // It's a partial response, have to wait for another or more Request/Response\r
     // conversations to get the full response.\r
@@ -844,14 +755,14 @@ Returns:
     //\r
     // In security negotiation stage, let CHAP module handle it.\r
     //\r
-    Status = IScsiCHAPOnRspReceived (Conn, Transit);\r
+    Status = IScsiCHAPOnRspReceived (Conn);\r
     break;\r
 \r
   case ISCSI_LOGIN_OPERATIONAL_NEGOTIATION:\r
     //\r
     // Response received with negotiation resonse on iSCSI parameters, check them.\r
     //\r
-    Status = IScsiCheckOpParams (Conn, Transit);\r
+    Status = IScsiCheckOpParams (Conn);\r
     break;\r
 \r
   default:\r
@@ -875,7 +786,7 @@ Returns:
       // CurrentStage is iSCSI Full Feature, it's the Login-Final Response,\r
       // get the TSIH from the Login Response.\r
       //\r
-      Session->TSIH = NTOHS (LoginRsp->TSIH);\r
+      Session->Tsih = NTOHS (LoginRsp->Tsih);\r
     }\r
   }\r
   //\r
@@ -886,32 +797,26 @@ Returns:
   return Status;\r
 }\r
 \r
-EFI_STATUS\r
-IScsiUpdateTargetAddress (\r
-  IN ISCSI_SESSION  *Session,\r
-  IN CHAR8          *Data,\r
-  IN UINT32         Len\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Updated the target information according the data received in the iSCSI\r
   login response with an target redirection status.\r
 \r
-Arguments:\r
-\r
-  Session - The iSCSI session.\r
-  Data    - The data segment which should contain the TargetAddress key-value list.\r
-  Len     - Length of the data.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCESS          - The target address is updated.\r
-  EFI_OUT_OF_RESOURCES - Failed to allocate memory.\r
-  EFI_NOT_FOUND        - The TargetAddress key is not found.\r
-\r
---*/\r
+  @param[in, out] Session      The iSCSI session.\r
+  @param[in]      Data         The data segment which should contain the\r
+                               TargetAddress key-value list.\r
+  @param[in]      Len          Length of the data.\r
+  \r
+  @retval EFI_SUCCESS          The target address is updated.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval EFI_NOT_FOUND        The TargetAddress key is not found.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
+EFI_STATUS\r
+IScsiUpdateTargetAddress (\r
+  IN OUT ISCSI_SESSION  *Session,\r
+  IN CHAR8              *Data,\r
+  IN UINT32             Len\r
+  )\r
 {\r
   LIST_ENTRY      *KeyValueList;\r
   CHAR8           *TargetAddress;\r
@@ -943,7 +848,7 @@ Returns:
 \r
     IpStr = TargetAddress;\r
 \r
-    while (*TargetAddress && (*TargetAddress != ':') && (*TargetAddress != ',')) {\r
+    while ((*TargetAddress != 0) && (*TargetAddress != ':') && (*TargetAddress != ',')) {\r
       //\r
       // NULL, ':' or ',' ends the IPv4 string.\r
       //\r
@@ -989,54 +894,55 @@ Returns:
   return Status;\r
 }\r
 \r
+/**\r
+  The callback function to free the net buffer list.\r
+\r
+  @param[in]  Arg The opaque parameter.\r
+**/\r
 VOID\r
+EFIAPI\r
 IScsiFreeNbufList (\r
   VOID *Arg\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  The callback function to free the net buffer list.\r
-\r
-Arguments:\r
-\r
-  Arg - The opaque parameter.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
   ASSERT (Arg != NULL);\r
 \r
   NetbufFreeList ((LIST_ENTRY     *) Arg);\r
-  gBS->FreePool (Arg);\r
+  FreePool (Arg);\r
 }\r
 \r
+/**\r
+  The callback function called in NetBufFree, it does nothing.\r
+\r
+  @param[in]   Arg  The opaque parameter.\r
+**/\r
 VOID\r
+EFIAPI\r
 IScsiNbufExtFree (\r
   VOID *Arg\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  The callback function called in NetBufFree, it does nothing.\r
-\r
-Arguments:\r
-\r
-  Arg - The opaque parameter.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
 }\r
 \r
+/**\r
+  Receive an iSCSI response PDU. An iSCSI response PDU contains an iSCSI PDU header and\r
+  an optional data segment. The two parts will be put into two blocks of buffers in the\r
+  net buffer. The digest check will be conducted in this function if needed and the digests\r
+  will be trimmed from the PDU buffer.\r
+\r
+  @param[in]   Conn        The iSCSI connection to receive data from.\r
+  @param[out]  Pdu         The received iSCSI pdu.\r
+  @param[in]   Context     The context used to describe information on the caller provided\r
+                           buffer to receive data segment of the iSCSI pdu, it's optional.\r
+  @param[in]  HeaderDigest Whether there will be header digest received.\r
+  @param[in]  DataDigest   Whether there will be data digest.\r
+  @param[in]  TimeoutEvent The timeout event, it's optional.\r
+\r
+  @retval EFI_SUCCESS          An iSCSI pdu is received.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval EFI_PROTOCOL_ERROR   Some kind of iSCSI protocol error happened.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiReceivePdu (\r
   IN ISCSI_CONNECTION                      *Conn,\r
@@ -1046,31 +952,6 @@ IScsiReceivePdu (
   IN BOOLEAN                               DataDigest,\r
   IN EFI_EVENT                             TimeoutEvent OPTIONAL\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Receive an iSCSI response PDU. An iSCSI response PDU contains an iSCSI PDU header and\r
-  an optional data segment. The two parts will be put into two blocks of buffers in the\r
-  net buffer. The digest check will be conducted in this function if needed and the digests\r
-  will be trimmed from the PDU buffer.\r
-\r
-Arguments:\r
-\r
-  Conn         - The iSCSI connection to receive data from.\r
-  Pdu          - The received iSCSI pdu.\r
-  Context      - The context used to describe information on the caller provided\r
-                 buffer to receive data segment of the iSCSI pdu, it's optional.\r
-  HeaderDigest - Whether there will be header digest received.\r
-  DataDigest   - Whether there will be data digest.\r
-  TimeoutEvent - The timeout event, it's optional.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCESS - An iSCSI pdu is received.\r
-  EFI_TIMEOUT - Timeout happenend.\r
-\r
---*/\r
 {\r
   LIST_ENTRY      *NbufList;\r
   UINT32          Len;\r
@@ -1084,7 +965,7 @@ Returns:
   NET_BUF         *DataSeg;\r
   UINT32          PadAndCRC32[2];\r
 \r
-  NbufList = AllocatePool (sizeof (LIST_ENTRY    ));\r
+  NbufList = AllocatePool (sizeof (LIST_ENTRY));\r
   if (NbufList == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
@@ -1097,11 +978,12 @@ Returns:
   Len     = sizeof (ISCSI_BASIC_HEADER) + (HeaderDigest ? sizeof (UINT32) : 0);\r
   PduHdr  = NetbufAlloc (Len);\r
   if (PduHdr == NULL) {\r
-    Status = EFI_OUT_OF_RESOURCES;\r
-    goto ON_EXIT;\r
+    FreePool (NbufList);\r
+    return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
   Header = NetbufAllocSpace (PduHdr, Len, NET_BUF_TAIL);\r
+  ASSERT (Header != NULL);\r
   InsertTailList (NbufList, &PduHdr->List);\r
 \r
   //\r
@@ -1125,9 +1007,14 @@ Returns:
   Len = ISCSI_GET_DATASEG_LEN (Header);\r
   if (Len == 0) {\r
     //\r
-    // No data segment.\r
+    // No data segment.Form the pdu from a list of pdu segments.\r
     //\r
-    goto FORM_PDU;\r
+    *Pdu = NetbufFromBufList (NbufList, 0, 0, IScsiFreeNbufList, NbufList);\r
+    if (*Pdu == NULL) {\r
+      Status = EFI_OUT_OF_RESOURCES;\r
+      goto ON_EXIT;\r
+    }\r
+    return Status;\r
   }\r
   //\r
   // Get the length of the padding bytes of the data segment.\r
@@ -1155,8 +1042,7 @@ Returns:
       // The first to receive the useful data. The second to receive the padding.\r
       //\r
       Fragment[1].Len   = PadLen + (DataDigest ? sizeof (UINT32) : 0);\r
-      Fragment[1].Bulk  = (UINT8 *) ((UINTN) &PadAndCRC32[1] - PadLen);\r
-\r
+      Fragment[1].Bulk  = (UINT8 *)PadAndCRC32 + (4 - PadLen);\r
       FragmentCount     = 2;\r
     } else {\r
       FragmentCount = 1;\r
@@ -1221,7 +1107,6 @@ Returns:
     NetbufTrim (DataSeg, PadLen, NET_BUF_TAIL);\r
   }\r
 \r
-FORM_PDU:\r
   //\r
   // Form the pdu from a list of pdu segments.\r
   //\r
@@ -1242,28 +1127,19 @@ ON_EXIT:
   return Status;\r
 }\r
 \r
-EFI_STATUS\r
-IScsiCheckOpParams (\r
-  IN ISCSI_CONNECTION  *Conn,\r
-  IN BOOLEAN           Transit\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Check and get the result of the prameter negotiation.\r
 \r
-Arguments:\r
-\r
-  Conn - The connection in iSCSI login.\r
-  Pdu  - The iSCSI response PDU containing the parameter list.\r
-\r
-Returns:\r
+  @param[in, out]  Conn          The connection in iSCSI login.\r
 \r
-  EFI_SUCCESS        - The parmeter check is passed and negotiation is finished.\r
-  EFI_PROTOCOL_ERROR - Some kind of iSCSI protocol error happened.\r
-\r
---*/\r
+  @retval EFI_SUCCESS          The parmeter check is passed and negotiation is finished.\r
+  @retval EFI_PROTOCOL_ERROR   Some kind of iSCSI protocol error happened.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+**/\r
+EFI_STATUS\r
+IScsiCheckOpParams (\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
+  )\r
 {\r
   EFI_STATUS      Status;\r
   LIST_ENTRY      *KeyValueList;\r
@@ -1292,7 +1168,7 @@ Returns:
   //\r
   KeyValueList = IScsiBuildKeyValueList (Data, Len);\r
   if (KeyValueList == NULL) {\r
-    gBS->FreePool (Data);\r
+    FreePool (Data);\r
     return Status;\r
   }\r
   //\r
@@ -1304,11 +1180,11 @@ Returns:
   }\r
 \r
   if (AsciiStrCmp (Value, "CRC32") == 0) {\r
-    if (Conn->HeaderDigest != ISCSI_DIGEST_CRC32) {\r
+    if (Conn->HeaderDigest != IScsiDigestCRC32) {\r
       goto ON_ERROR;\r
     }\r
   } else if (AsciiStrCmp (Value, ISCSI_KEY_VALUE_NONE) == 0) {\r
-    Conn->HeaderDigest = ISCSI_DIGEST_NONE;\r
+    Conn->HeaderDigest = IScsiDigestNone;\r
   } else {\r
     goto ON_ERROR;\r
   }\r
@@ -1321,11 +1197,11 @@ Returns:
   }\r
 \r
   if (AsciiStrCmp (Value, "CRC32") == 0) {\r
-    if (Conn->DataDigest != ISCSI_DIGEST_CRC32) {\r
+    if (Conn->DataDigest != IScsiDigestCRC32) {\r
       goto ON_ERROR;\r
     }\r
   } else if (AsciiStrCmp (Value, ISCSI_KEY_VALUE_NONE) == 0) {\r
-    Conn->DataDigest = ISCSI_DIGEST_NONE;\r
+    Conn->DataDigest = IScsiDigestNone;\r
   } else {\r
     goto ON_ERROR;\r
   }\r
@@ -1365,16 +1241,11 @@ Returns:
   Session->ImmediateData = (BOOLEAN) (Session->ImmediateData && (AsciiStrCmp (Value, "Yes") == 0));\r
 \r
   //\r
-  // MaxRecvDataSegmentLength, result function is Mininum.\r
+  // MaxRecvDataSegmentLength is declarative.\r
   //\r
   Value = IScsiGetValueByKeyFromList (KeyValueList, ISCSI_KEY_MAX_RECV_DATA_SEGMENT_LENGTH);\r
   if (Value != NULL) {\r
-    //\r
-    // MaxRecvDataSegmentLength is declarative.\r
-    //\r
-    NumericValue                    = AsciiStrDecimalToUintn (Value);\r
-\r
-    Conn->MaxRecvDataSegmentLength  = (UINT32) MIN (Conn->MaxRecvDataSegmentLength, NumericValue);\r
+    Conn->MaxRecvDataSegmentLength = (UINT32) AsciiStrDecimalToUintn (Value);\r
   }\r
   //\r
   // MaxBurstLength, result funtion is Mininum.\r
@@ -1390,9 +1261,10 @@ Returns:
   //\r
   // FirstBurstLength, result function is Minimum. Irrelevant when InitialR2T=Yes and\r
   // ImmediateData=No.\r
+  // This Key/Value is negotiation type.\r
   //\r
   Value = IScsiGetValueByKeyFromList (KeyValueList, ISCSI_KEY_FIRST_BURST_LENGTH);\r
-  if ((Value == NULL) && !(Session->InitialR2T && !Session->ImmediateData)) {\r
+  if (Value == NULL) {\r
     goto ON_ERROR;\r
   }\r
 \r
@@ -1499,43 +1371,34 @@ ON_ERROR:
 \r
   IScsiFreeKeyValueList (KeyValueList);\r
 \r
-  gBS->FreePool (Data);\r
+  FreePool (Data);\r
 \r
   return Status;\r
 }\r
 \r
+/**\r
+  Fill the oprational prameters.\r
+\r
+  @param[in]       Conn        The connection in iSCSI login.\r
+  @param[in, out]  Pdu         The iSCSI login request PDU to fill the parameters.\r
+\r
+  @retval EFI_SUCCESS          The parmeters are filled into the iSCSI login request PDU.\r
+**/\r
 EFI_STATUS\r
 IScsiFillOpParams (\r
   IN ISCSI_CONNECTION  *Conn,\r
-  IN NET_BUF           *Pdu\r
+  IN OUT NET_BUF       *Pdu\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Fill the oprational prameters.\r
-\r
-Arguments:\r
-\r
-  Conn - The connection in iSCSI login.\r
-  Pdu  - The iSCSI login request PDU to fill the parameters.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCESS          - The parmeters are filled into the iSCSI login request PDU.\r
-  EFI_OUT_OF_RESOURCES - There is not enough space in the PDU to hold the parameters.\r
-\r
---*/\r
 {\r
   ISCSI_SESSION *Session;\r
   CHAR8         Value[256];\r
 \r
   Session = Conn->Session;\r
 \r
-  AsciiSPrint (Value, sizeof (Value), "%a", (Conn->HeaderDigest == ISCSI_DIGEST_CRC32) ? "None,CRC32" : "None");\r
+  AsciiSPrint (Value, sizeof (Value), "%a", (Conn->HeaderDigest == IScsiDigestCRC32) ? "None,CRC32" : "None");\r
   IScsiAddKeyValuePair (Pdu, ISCSI_KEY_HEADER_DIGEST, Value);\r
 \r
-  AsciiSPrint (Value, sizeof (Value), "%a", (Conn->DataDigest == ISCSI_DIGEST_CRC32) ? "None,CRC32" : "None");\r
+  AsciiSPrint (Value, sizeof (Value), "%a", (Conn->DataDigest == IScsiDigestCRC32) ? "None,CRC32" : "None");\r
   IScsiAddKeyValuePair (Pdu, ISCSI_KEY_DATA_DIGEST, Value);\r
 \r
   AsciiSPrint (Value, sizeof (Value), "%d", Session->ErrorRecoveryLevel);\r
@@ -1547,7 +1410,7 @@ Returns:
   AsciiSPrint (Value, sizeof (Value), "%a", Session->ImmediateData ? "Yes" : "No");\r
   IScsiAddKeyValuePair (Pdu, ISCSI_KEY_IMMEDIATE_DATA, Value);\r
 \r
-  AsciiSPrint (Value, sizeof (Value), "%d", Conn->MaxRecvDataSegmentLength);\r
+  AsciiSPrint (Value, sizeof (Value), "%d", MAX_RECV_DATA_SEG_LEN_IN_FFP);\r
   IScsiAddKeyValuePair (Pdu, ISCSI_KEY_MAX_RECV_DATA_SEGMENT_LENGTH, Value);\r
 \r
   AsciiSPrint (Value, sizeof (Value), "%d", Session->MaxBurstLength);\r
@@ -1577,29 +1440,21 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
-EFI_STATUS\r
-IScsiPadSegment (\r
-  IN NET_BUF  *Pdu,\r
-  IN UINT32   Len\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Pad the iSCSI AHS or data segment to an integer number of 4 byte words.\r
 \r
-Arguments:\r
-\r
-  Pdu - The iSCSI pdu which contains segments to pad.\r
-  Len - The length of the last semgnet in the PDU.\r
-\r
-Returns:\r
+  @param[in, out]  Pdu         The iSCSI pdu which contains segments to pad.\r
+  @param[in]       Len         The length of the last semgnet in the PDU.\r
 \r
-  EFI_SUCCESS          - The segment is padded or no need to pad it.\r
-  EFI_OUT_OF_RESOURCES - There is not enough remaining free space to add the\r
-                         padding bytes.\r
-\r
---*/\r
+  @retval EFI_SUCCESS          The segment is padded or no need to pad it.\r
+  @retval EFI_OUT_OF_RESOURCES There is not enough remaining free space to add the\r
+                               padding bytes.\r
+**/\r
+EFI_STATUS\r
+IScsiPadSegment (\r
+  IN OUT NET_BUF  *Pdu,\r
+  IN UINT32       Len\r
+  )\r
 {\r
   UINT32  PadLen;\r
   UINT8   *Data;\r
@@ -1618,32 +1473,25 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
-LIST_ENTRY     *\r
+/**\r
+  Build a key-value list from the data segment.\r
+\r
+  @param[in]  Data The data segment containing the key-value pairs.\r
+  @param[in]  Len  Length of the data segment.\r
+\r
+  @return The key-value list.\r
+  @retval NULL Other errors as indicated.\r
+**/\r
+LIST_ENTRY *\r
 IScsiBuildKeyValueList (\r
   IN CHAR8  *Data,\r
   IN UINT32 Len\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Build a key-value list from the data segment.\r
-\r
-Arguments:\r
-\r
-  Data - The data segment containing the key-value pairs.\r
-  Len  - Length of the data segment.\r
-\r
-Returns:\r
-\r
-  The key-value list.\r
-\r
---*/\r
 {\r
   LIST_ENTRY            *ListHead;\r
   ISCSI_KEY_VALUE_PAIR  *KeyValuePair;\r
 \r
-  ListHead = AllocatePool (sizeof (LIST_ENTRY    ));\r
+  ListHead = AllocatePool (sizeof (LIST_ENTRY));\r
   if (ListHead == NULL) {\r
     return NULL;\r
   }\r
@@ -1671,7 +1519,7 @@ Returns:
       Data++;\r
       Len--;\r
     } else {\r
-      gBS->FreePool (KeyValuePair);\r
+      FreePool (KeyValuePair);\r
       goto ON_ERROR;\r
     }\r
 \r
@@ -1692,28 +1540,20 @@ ON_ERROR:
   return NULL;\r
 }\r
 \r
-CHAR8 *\r
-IScsiGetValueByKeyFromList (\r
-  IN LIST_ENTRY      *KeyValueList,\r
-  IN CHAR8           *Key\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Get the value string by the key name from the key-value list. If found,\r
   the key-value entry will be removed from the list.\r
 \r
-Arguments:\r
+  @param[in, out]  KeyValueList  The key-value list.\r
+  @param[in]       Key           The key name to find.\r
 \r
-  KeyValueList - The key-value list.\r
-  Key          - The key name to find.\r
-\r
-Returns:\r
-\r
-  The value string.\r
-\r
---*/\r
+  @return The value string.\r
+**/\r
+CHAR8 *\r
+IScsiGetValueByKeyFromList (\r
+  IN OUT LIST_ENTRY  *KeyValueList,\r
+  IN CHAR8           *Key\r
+  )\r
 {\r
   LIST_ENTRY            *Entry;\r
   ISCSI_KEY_VALUE_PAIR  *KeyValuePair;\r
@@ -1728,7 +1568,7 @@ Returns:
       Value = KeyValuePair->Value;\r
 \r
       RemoveEntryList (&KeyValuePair->List);\r
-      gBS->FreePool (KeyValuePair);\r
+      FreePool (KeyValuePair);\r
       break;\r
     }\r
   }\r
@@ -1736,25 +1576,15 @@ Returns:
   return Value;\r
 }\r
 \r
+/**\r
+  Free the key-value list.\r
+\r
+  @param[in]  KeyValueList The key-value list.\r
+**/\r
 VOID\r
 IScsiFreeKeyValueList (\r
   IN LIST_ENTRY      *KeyValueList\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Free the key-value list.\r
-\r
-Arguments:\r
-\r
-  KeyValueList - The key-value list.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
   LIST_ENTRY            *Entry;\r
   ISCSI_KEY_VALUE_PAIR  *KeyValuePair;\r
@@ -1763,34 +1593,26 @@ Returns:
     Entry         = NetListRemoveHead (KeyValueList);\r
     KeyValuePair  = NET_LIST_USER_STRUCT (Entry, ISCSI_KEY_VALUE_PAIR, List);\r
 \r
-    gBS->FreePool (KeyValuePair);\r
+    FreePool (KeyValuePair);\r
   }\r
 \r
-  gBS->FreePool (KeyValueList);\r
+  FreePool (KeyValueList);\r
 }\r
 \r
-EFI_STATUS\r
-IScsiNormalizeName (\r
-  IN CHAR8  *Name,\r
-  IN UINTN  Len\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Normalize the iSCSI name according to RFC.\r
 \r
-Arguments:\r
-\r
-  Name - The iSCSI name.\r
-  Len  - length of the iSCSI name.\r
+  @param[in, out]  Name       The iSCSI name.\r
+  @param[in]       Len        length of the iSCSI name.\r
 \r
-Returns:\r
-\r
-  EFI_SUCCESS        - The iSCSI name is valid and normalized.\r
-  EFI_PROTOCOL_ERROR - The iSCSI name is mal-formatted or not in the IQN format.\r
-\r
---*/\r
+  @retval EFI_SUCCESS        The iSCSI name is valid and normalized.\r
+  @retval EFI_PROTOCOL_ERROR The iSCSI name is mal-formatted or not in the IQN format.\r
+**/\r
+EFI_STATUS\r
+IScsiNormalizeName (\r
+  IN OUT CHAR8  *Name,\r
+  IN UINTN      Len\r
+  )\r
 {\r
   UINTN Index;\r
 \r
@@ -1826,28 +1648,21 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Create an iSCSI task control block.\r
+\r
+  @param[in]   Conn           The connection on which the task control block will be created.\r
+  @param[out]  Tcb            The newly created task control block.\r
+\r
+  @retval EFI_SUCCESS          The task control block is created.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval EFI_NOT_READY        The target can not accept new commands.\r
+**/\r
 EFI_STATUS\r
 IScsiNewTcb (\r
   IN  ISCSI_CONNECTION  *Conn,\r
   OUT ISCSI_TCB         **Tcb\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Create an iSCSI task control block.\r
-\r
-Arguments:\r
-\r
-  Conn - The connection on which the task control block will be created.\r
-  Tcb  - The newly created task control block.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCESS          - The task control block is created.\r
-  EFI_OUT_OF_RESOURCES - Failed to allocate memory.\r
-\r
---*/\r
 {\r
   ISCSI_SESSION *Session;\r
   ISCSI_TCB     *NewTcb;\r
@@ -1885,52 +1700,34 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Delete the tcb from the connection and destroy it.\r
+\r
+  @param[in]  Tcb The tcb to delete.\r
+**/\r
 VOID\r
 IScsiDelTcb (\r
   IN ISCSI_TCB  *Tcb\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Delete the tcb from the connection and destroy it.\r
-\r
-Arguments:\r
-\r
-  Tcb - The tcb to delete.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
   RemoveEntryList (&Tcb->Link);\r
 \r
-  gBS->FreePool (Tcb);\r
+  FreePool (Tcb);\r
 }\r
 \r
+/**\r
+  Find the task control block by the initator task tag.\r
+\r
+  @param[in]  TcbList         The tcb list.\r
+  @param[in]  InitiatorTaskTag The initiator task tag.\r
+\r
+  @return The task control block found.\r
+**/\r
 ISCSI_TCB *\r
 IScsiFindTcbByITT (\r
   IN LIST_ENTRY      *TcbList,\r
   IN UINT32          InitiatorTaskTag\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Find the task control block by the initator task tag.\r
-\r
-Arguments:\r
-\r
-  TcbList          - The tcb list.\r
-  InitiatorTaskTag - The initiator task tag.\r
-\r
-Returns:\r
-\r
-  The task control block found.\r
-\r
---*/\r
 {\r
   ISCSI_TCB       *Tcb;\r
   LIST_ENTRY      *Entry;\r
@@ -1950,29 +1747,21 @@ Returns:
   return Tcb;\r
 }\r
 \r
+/**\r
+  Create a data segment, pad it and calculate the CRC if needed.\r
+\r
+  @param[in]  Data       The data to fill into the data segment.\r
+  @param[in]  Len        Length of the data.\r
+  @param[in]  DataDigest Whether to calculate CRC for this data segment.\r
+\r
+  @return The net buffer wrapping the data segment.\r
+**/\r
 NET_BUF *\r
 IScsiNewDataSegment (\r
   IN UINT8    *Data,\r
   IN UINT32   Len,\r
   IN BOOLEAN  DataDigest\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Create a data segment, pad it and calculate the CRC if needed.\r
-\r
-Arguments:\r
-\r
-  Data       - The data to fill into the data segment.\r
-  Len        - Length of the data.\r
-  DataDigest - Whether to calculate CRC for this data segment.\r
-\r
-Returns:\r
-\r
-  The net buffer wrapping the data segment.\r
-\r
---*/\r
 {\r
   NET_FRAGMENT  Fragment[2];\r
   UINT32        FragmentCount;\r
@@ -1997,30 +1786,23 @@ Returns:
   return DataSeg;\r
 }\r
 \r
+/**\r
+  Create a iSCSI SCSI command PDU to encapsulate the command issued\r
+  by SCSI through the EXT SCSI PASS THRU Protocol.\r
+\r
+  @param[in]  Packet The EXT SCSI PASS THRU request packet containing the SCSI command.\r
+  @param[in]  Lun    The LUN.\r
+  @param[in]  Tcb    The tcb assocated with this SCSI command.\r
+\r
+  @return The  created iSCSI SCSI command PDU.\r
+  @retval NULL Other errors as indicated.\r
+**/\r
 NET_BUF *\r
 IScsiNewScsiCmdPdu (\r
   IN EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET *Packet,\r
   IN UINT64                                     Lun,\r
   IN ISCSI_TCB                                  *Tcb\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Create a iSCSI SCSI command PDU to encapsulate the command issued\r
-  by SCSI through the EXT SCSI PASS THRU Protocol.\r
-\r
-Arguments:\r
-\r
-  Packet - The EXT SCSI PASS THRU request packet containing the SCSI command.\r
-  Lun    - The LUN.\r
-  Tcb    - The tcb assocated with this SCSI command.\r
-\r
-Returns:\r
-\r
-  The created iSCSI SCSI command PDU.\r
-\r
---*/\r
 {\r
   LIST_ENTRY                      *NbufList;\r
   NET_BUF                         *Pdu;\r
@@ -2058,6 +1840,10 @@ Returns:
   }\r
 \r
   ScsiCmd = (SCSI_COMMAND *) NetbufAllocSpace (PduHeader, Length, NET_BUF_TAIL);\r
+  if (ScsiCmd == NULL) {\r
+    NetbufFree (PduHeader);\r
+    return NULL;\r
+  }    \r
   Header  = (ISCSI_ADDITIONAL_HEADER *) (ScsiCmd + 1);\r
 \r
   ZeroMem (ScsiCmd, Length);\r
@@ -2102,10 +1888,10 @@ Returns:
   ScsiCmd->CmdSN            = NTOHL (Tcb->CmdSN);\r
   ScsiCmd->ExpStatSN        = NTOHL (Tcb->Conn->ExpStatSN);\r
 \r
-  CopyMem (ScsiCmd->CDB, Packet->Cdb, sizeof (ScsiCmd->CDB));\r
+  CopyMem (ScsiCmd->Cdb, Packet->Cdb, sizeof (ScsiCmd->Cdb));\r
 \r
   if (Packet->CdbLength > 16) {\r
-    Header->Length  = NTOHS (Packet->CdbLength - 15);\r
+    Header->Length  = NTOHS ((UINT16) (Packet->CdbLength - 15));\r
     Header->Type    = ISCSI_AHS_TYPE_EXT_CDB;\r
 \r
     CopyMem (Header + 1, (UINT8 *) Packet->Cdb + 16, Packet->CdbLength - 16);\r
@@ -2139,7 +1925,7 @@ Returns:
       goto ON_EXIT;\r
     }\r
 \r
-    NbufList = AllocatePool (sizeof (LIST_ENTRY    ));\r
+    NbufList = AllocatePool (sizeof (LIST_ENTRY));\r
     if (NbufList == NULL) {\r
       NetbufFree (PduHeader);\r
       NetbufFree (DataSeg);\r
@@ -2164,7 +1950,7 @@ Returns:
       ) {\r
     //\r
     // Unsolicited data out sequence is not allowed,\r
-    // or FirstBustLength data is already sent out by immediate data\r
+    // or FirstBurstLength data is already sent out by immediate data\r
     // or all the OUT data accompany this SCSI packet is sent as\r
     // immediate data, the final flag should be set on this SCSI Command\r
     // PDU.\r
@@ -2177,6 +1963,18 @@ ON_EXIT:
   return Pdu;\r
 }\r
 \r
+/**\r
+  Create a new iSCSI SCSI Data Out PDU.\r
+\r
+  @param[in]  Data   The data to put into the Data Out PDU.\r
+  @param[in]  Len    Length of the data.\r
+  @param[in]  DataSN The DataSN of the Data Out PDU.\r
+  @param[in]  Tcb    The task control block of this Data Out PDU.\r
+  @param[in]  Lun    The LUN.\r
+\r
+  @return The net buffer wrapping the Data Out PDU.\r
+  @retval NULL Other errors as indicated.\r
+**/\r
 NET_BUF *\r
 IScsiNewDataOutPdu (\r
   IN UINT8      *Data,\r
@@ -2185,25 +1983,6 @@ IScsiNewDataOutPdu (
   IN ISCSI_TCB  *Tcb,\r
   IN UINT64     Lun\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Create a new iSCSI SCSI Data Out PDU.\r
-\r
-Arguments:\r
-\r
-  Data   - The data to put into the Data Out PDU.\r
-  Len    - Length of the data.\r
-  DataSN - The DataSN of the Data Out PDU.\r
-  Tcb    - The task control block of this Data Out PDU.\r
-  Lun    - The LUN.\r
-\r
-Returns:\r
-\r
-  The net buffer wrapping the Data Out PDU.\r
-\r
---*/\r
 {\r
   LIST_ENTRY          *NbufList;\r
   NET_BUF             *PduHdr;\r
@@ -2212,7 +1991,7 @@ Returns:
   ISCSI_SCSI_DATA_OUT *DataOutHdr;\r
   ISCSI_XFER_CONTEXT  *XferContext;\r
 \r
-  NbufList = AllocatePool (sizeof (LIST_ENTRY    ));\r
+  NbufList = AllocatePool (sizeof (LIST_ENTRY));\r
   if (NbufList == NULL) {\r
     return NULL;\r
   }\r
@@ -2224,7 +2003,7 @@ Returns:
   //\r
   PduHdr = NetbufAlloc (sizeof (ISCSI_SCSI_DATA_OUT));\r
   if (PduHdr == NULL) {\r
-    gBS->FreePool (NbufList);\r
+    FreePool (NbufList);\r
     return NULL;\r
   }\r
   //\r
@@ -2233,6 +2012,7 @@ Returns:
   InsertTailList (NbufList, &PduHdr->List);\r
 \r
   DataOutHdr  = (ISCSI_SCSI_DATA_OUT *) NetbufAllocSpace (PduHdr, sizeof (ISCSI_SCSI_DATA_OUT), NET_BUF_TAIL);\r
+  ASSERT (DataOutHdr != NULL);\r
   XferContext = &Tcb->XferContext;\r
 \r
   ZeroMem (DataOutHdr, sizeof (ISCSI_SCSI_DATA_OUT));\r
@@ -2273,29 +2053,22 @@ Returns:
   return Pdu;\r
 }\r
 \r
-LIST_ENTRY     *\r
+/**\r
+  Generate a consecutive sequence of iSCSI SCSI Data Out PDUs.\r
+\r
+  @param[in]  Data The data  which will be carried by the sequence of iSCSI SCSI Data Out PDUs.\r
+  @param[in]  Tcb  The task control block of the data to send out.\r
+  @param[in]  Lun  The LUN the data will be sent to.\r
+\r
+  @return A list of net buffers with each of them wraps an iSCSI SCSI Data Out PDU.\r
+  @retval NULL Other errors as indicated.\r
+**/\r
+LIST_ENTRY *\r
 IScsiGenerateDataOutPduSequence (\r
   IN UINT8      *Data,\r
   IN ISCSI_TCB  *Tcb,\r
   IN UINT64     Lun\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Generate a consecutive sequence of iSCSI SCSI Data Out PDUs.\r
-\r
-Arguments:\r
-\r
-  Data - The data  which will be carried by the sequence of iSCSI SCSI Data Out PDUs.\r
-  Tcb  - The task control block of the data to send out.\r
-  Lun  - The LUN the data will be sent to.\r
-\r
-Returns:\r
-\r
-  A list of net buffers with each of them wraps an iSCSI SCSI Data Out PDU.\r
-\r
---*/\r
 {\r
   LIST_ENTRY          *PduList;\r
   UINT32              DataSN;\r
@@ -2303,8 +2076,9 @@ Returns:
   NET_BUF             *DataOutPdu;\r
   ISCSI_CONNECTION    *Conn;\r
   ISCSI_XFER_CONTEXT  *XferContext;\r
+  UINT8               *DataOutPacket;\r
 \r
-  PduList = AllocatePool (sizeof (LIST_ENTRY    ));\r
+  PduList = AllocatePool (sizeof (LIST_ENTRY));\r
   if (PduList == NULL) {\r
     return NULL;\r
   }\r
@@ -2321,7 +2095,6 @@ Returns:
     // Determine the length of data this Data Out PDU can carry.\r
     //\r
     DataLen = MIN (XferContext->DesiredLength, Conn->MaxRecvDataSegmentLength);\r
-    Data += DataLen;\r
 \r
     //\r
     // Create a Data Out PDU.\r
@@ -2342,41 +2115,42 @@ Returns:
     XferContext->Offset += DataLen;\r
     XferContext->DesiredLength -= DataLen;\r
     DataSN++;\r
+    Data += DataLen;\r
   }\r
   //\r
   // Set the F bit for the last data out PDU in this sequence.\r
   //\r
-  ISCSI_SET_FLAG (NetbufGetByte (DataOutPdu, 0, NULL), ISCSI_BHS_FLAG_FINAL);\r
+  DataOutPacket = NetbufGetByte (DataOutPdu, 0, NULL);\r
+  if (DataOutPacket == NULL) {\r
+    IScsiFreeNbufList (PduList);\r
+    PduList = NULL;\r
+    goto ON_EXIT;\r
+  }\r
+\r
+  ISCSI_SET_FLAG (DataOutPacket, ISCSI_BHS_FLAG_FINAL);\r
 \r
 ON_EXIT:\r
 \r
   return PduList;\r
 }\r
 \r
+/**\r
+  Send the Data in a sequence of Data Out PDUs one by one.\r
+\r
+  @param[in]  Data            The data to carry by Data Out PDUs.\r
+  @param[in]  Lun             The LUN the data will be sent to.\r
+  @param[in]  Tcb             The task control block.\r
+\r
+  @retval EFI_SUCCES           The data is sent out to the LUN.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiSendDataOutPduSequence (\r
   IN UINT8      *Data,\r
   IN UINT64     Lun,\r
   IN ISCSI_TCB  *Tcb\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Send the Data in a sequence of Data Out PDUs one by one.\r
-\r
-Arguments:\r
-\r
-  Data - The data to carry by Data Out PDUs.\r
-  Lun  - The LUN the data will be sent to.\r
-  Tcb  - The task control block.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCES           - The data is sent out to the LUN.\r
-  EFI_OUT_OF_RESOURCES - Failed to allocate memory.\r
-\r
---*/\r
 {\r
   LIST_ENTRY      *DataOutPduList;\r
   LIST_ENTRY      *Entry;\r
@@ -2410,36 +2184,33 @@ Returns:
   return Status;\r
 }\r
 \r
+/**\r
+  Process the received iSCSI SCSI Data In PDU.\r
+\r
+  @param[in]        Pdu      The Data In PDU received.\r
+  @param[in]        Tcb      The task control block.\r
+  @param[in, out]   Packet   The EXT SCSI PASS THRU request packet.\r
+\r
+  @retval EFI_SUCCES           The check on the Data IN PDU is passed and some update\r
+                               actions are taken.\r
+  @retval EFI_PROTOCOL_ERROR   Some kind of iSCSI protocol errror happened.\r
+  @retval EFI_BAD_BUFFER_SIZEE The buffer was not the proper size for the request.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiOnDataInRcvd (\r
   IN NET_BUF                                         *Pdu,\r
   IN ISCSI_TCB                                       *Tcb,\r
   IN OUT EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET  *Packet\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Process the received iSCSI SCSI Data In PDU.\r
-\r
-Arguments:\r
-\r
-  Pdu    - The Data In PDU received.\r
-  Tcb    - The task control block.\r
-  Packet - The EXT SCSI PASS THRU request packet.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCES          - The check on the Data IN PDU is passed and some update\r
-                        actions are taken.\r
-  EFI_PROTOCOL_ERROR  - Some kind of iSCSI protocol errror happened.\r
-\r
---*/\r
 {\r
   ISCSI_SCSI_DATA_IN  *DataInHdr;\r
   EFI_STATUS          Status;\r
 \r
   DataInHdr                   = (ISCSI_SCSI_DATA_IN *) NetbufGetByte (Pdu, 0, NULL);\r
+  if (DataInHdr == NULL) {\r
+    return EFI_PROTOCOL_ERROR;\r
+  }\r
 \r
   DataInHdr->InitiatorTaskTag = NTOHL (DataInHdr->InitiatorTaskTag);\r
   DataInHdr->ExpCmdSN         = NTOHL (DataInHdr->ExpCmdSN);\r
@@ -2502,6 +2273,18 @@ Returns:
   return Status;\r
 }\r
 \r
+/**\r
+  Process the received iSCSI R2T PDU.\r
+\r
+  @param[in]       Pdu       The R2T PDU received.\r
+  @param[in]       Tcb       The task control block.\r
+  @param[in]       Lun       The Lun.\r
+  @param[in, out]  Packet    The EXT SCSI PASS THRU request packet.\r
+\r
+  @retval EFI_SUCCES         The R2T PDU is valid and the solicited data is sent out.\r
+  @retval EFI_PROTOCOL_ERROR Some kind of iSCSI protocol errror happened.\r
+  @retval Others             Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiOnR2TRcvd (\r
   IN NET_BUF                                         *Pdu,\r
@@ -2509,25 +2292,6 @@ IScsiOnR2TRcvd (
   IN UINT64                                          Lun,\r
   IN OUT EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET  *Packet\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Process the received iSCSI R2T PDU.\r
-\r
-Arguments:\r
-\r
-  Pdu    - The R2T PDU received.\r
-  Tcb    - The task control block.\r
-  Lun    - The Lun.\r
-  Packet - The EXT SCSI PASS THRU request packet.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCES          - The R2T PDU is valid and the solicited data is sent out.\r
-  EFI_PROTOCOL_ERROR  - Some kind of iSCSI protocol errror happened.\r
-\r
---*/\r
 {\r
   ISCSI_READY_TO_TRANSFER *R2THdr;\r
   EFI_STATUS              Status;\r
@@ -2535,11 +2299,14 @@ Returns:
   UINT8                   *Data;\r
 \r
   R2THdr = (ISCSI_READY_TO_TRANSFER *) NetbufGetByte (Pdu, 0, NULL);\r
+  if (R2THdr == NULL) {\r
+    return EFI_PROTOCOL_ERROR;\r
+  }\r
 \r
   R2THdr->InitiatorTaskTag = NTOHL (R2THdr->InitiatorTaskTag);\r
   R2THdr->TargetTransferTag = NTOHL (R2THdr->TargetTransferTag);\r
   R2THdr->StatSN = NTOHL (R2THdr->StatSN);\r
-  R2THdr->R2TSN = NTOHL (R2THdr->R2TSN);\r
+  R2THdr->R2TSeqNum = NTOHL (R2THdr->R2TSeqNum);\r
   R2THdr->BufferOffset = NTOHL (R2THdr->BufferOffset);\r
   R2THdr->DesiredDataTransferLength = NTOHL (R2THdr->DesiredDataTransferLength);\r
 \r
@@ -2549,7 +2316,7 @@ Returns:
   //\r
   // Check the sequence number.\r
   //\r
-  Status = IScsiCheckSN (&Tcb->ExpDataSN, R2THdr->R2TSN);\r
+  Status = IScsiCheckSN (&Tcb->ExpDataSN, R2THdr->R2TSeqNum);\r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
@@ -2573,30 +2340,24 @@ Returns:
   return Status;\r
 }\r
 \r
+/**\r
+  Process the received iSCSI SCSI Response PDU.\r
+\r
+  @param[in]       Pdu      The Response PDU received.\r
+  @param[in]       Tcb      The task control block.\r
+  @param[in, out]  Packet   The EXT SCSI PASS THRU request packet.\r
+\r
+  @retval EFI_SUCCES         The Response PDU is processed.\r
+  @retval EFI_PROTOCOL_ERROR Some kind of iSCSI protocol errror happened.\r
+  @retval EFI_BAD_BUFFER_SIZEE The buffer was not the proper size for the request.\r
+  @retval Others             Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiOnScsiRspRcvd (\r
   IN NET_BUF                                         *Pdu,\r
   IN ISCSI_TCB                                       *Tcb,\r
   IN OUT EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET  *Packet\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Process the received iSCSI SCSI Response PDU.\r
-\r
-Arguments:\r
-\r
-  Pdu    - The Response PDU received.\r
-  Tcb    - The task control block.\r
-  Packet - The EXT SCSI PASS THRU request packet.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCES         - The Response PDU is processed.\r
-  EFI_PROTOCOL_ERROR - Some kind of iSCSI protocol errror happened.\r
-\r
---*/\r
 {\r
   SCSI_RESPONSE     *ScsiRspHdr;\r
   ISCSI_SENSE_DATA  *SenseData;\r
@@ -2604,6 +2365,9 @@ Returns:
   UINT32            DataSegLen;\r
 \r
   ScsiRspHdr                    = (SCSI_RESPONSE *) NetbufGetByte (Pdu, 0, NULL);\r
+  if (ScsiRspHdr == NULL) {\r
+    return EFI_PROTOCOL_ERROR;\r
+  }\r
 \r
   ScsiRspHdr->InitiatorTaskTag  = NTOHL (ScsiRspHdr->InitiatorTaskTag);\r
   if (ScsiRspHdr->InitiatorTaskTag != Tcb->InitiatorTaskTag) {\r
@@ -2666,6 +2430,9 @@ Returns:
   DataSegLen = ISCSI_GET_DATASEG_LEN (ScsiRspHdr);\r
   if (DataSegLen != 0) {\r
     SenseData               = (ISCSI_SENSE_DATA *) NetbufGetByte (Pdu, sizeof (SCSI_RESPONSE), NULL);\r
+    if (SenseData == NULL) {\r
+      return EFI_PROTOCOL_ERROR;\r
+    }\r
 \r
     SenseData->Length       = NTOHS (SenseData->Length);\r
 \r
@@ -2680,34 +2447,29 @@ Returns:
   return Status;\r
 }\r
 \r
+/**\r
+  Process the received NOP In PDU.\r
+\r
+  @param[in]  Pdu            The NOP In PDU received.\r
+  @param[in]  Tcb            The task control block.\r
+\r
+  @retval EFI_SUCCES         The NOP In PDU is processed and the related sequence\r
+                             numbers are updated.\r
+  @retval EFI_PROTOCOL_ERROR Some kind of iSCSI protocol errror happened.\r
+**/\r
 EFI_STATUS\r
 IScsiOnNopInRcvd (\r
   IN NET_BUF    *Pdu,\r
   IN ISCSI_TCB  *Tcb\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Process the received NOP In PDU.\r
-\r
-Arguments:\r
-\r
-  Pdu    - The NOP In PDU received.\r
-  Tcb    - The task control block.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCES          - The NOP In PDU is processed and the related sequence\r
-                        numbers are updated.\r
-  EFI_PROTOCOL_ERROR  - Some kind of iSCSI protocol errror happened.\r
-\r
---*/\r
 {\r
   ISCSI_NOP_IN  *NopInHdr;\r
   EFI_STATUS    Status;\r
 \r
   NopInHdr            = (ISCSI_NOP_IN *) NetbufGetByte (Pdu, 0, NULL);\r
+  if (NopInHdr == NULL) {\r
+    return EFI_PROTOCOL_ERROR;\r
+  }\r
 \r
   NopInHdr->StatSN    = NTOHL (NopInHdr->StatSN);\r
   NopInHdr->ExpCmdSN  = NTOHL (NopInHdr->ExpCmdSN);\r
@@ -2729,6 +2491,21 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Execute the SCSI command issued through the EXT SCSI PASS THRU protocol.\r
+\r
+  @param[in]       PassThru  The EXT SCSI PASS THRU protocol.\r
+  @param[in]       Target    The target ID.\r
+  @param[in]       Lun       The LUN.\r
+  @param[in, out]  Packet    The request packet containing IO request, SCSI command\r
+                             buffer and buffers to read/write.\r
+                             \r
+  @retval EFI_SUCCES           The SCSI command is executed and the result is updated to \r
+                               the Packet.\r
+  @retval EFI_DEVICE_ERROR     Session state was not as required.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiExecuteScsiCommand (\r
   IN EFI_EXT_SCSI_PASS_THRU_PROTOCOL                 *PassThru,\r
@@ -2736,27 +2513,6 @@ IScsiExecuteScsiCommand (
   IN UINT64                                          Lun,\r
   IN OUT EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET  *Packet\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Execute the SCSI command issued through the EXT SCSI PASS THRU protocol.\r
-\r
-Arguments:\r
-\r
-  PassThru - The EXT SCSI PASS THRU protocol.\r
-  Target   - The target ID.\r
-  Lun      - The LUN.\r
-  Packet   - The request packet containing IO request, SCSI command buffer and\r
-             buffers to read/write.\r
-\r
-Returns:\r
-\r
-  EFI_SUCCES       - The SCSI command is executed and the result is updated to \r
-                     the Packet.\r
-  EFI_DEVICE_ERROR - Some unexpected error happened.\r
-\r
---*/\r
 {\r
   EFI_STATUS              Status;\r
   ISCSI_DRIVER_DATA       *Private;\r
@@ -2769,13 +2525,14 @@ Returns:
   UINT8                   *Data;\r
   ISCSI_IN_BUFFER_CONTEXT InBufferContext;\r
   UINT64                  Timeout;\r
-  UINT8                   *Buffer;\r
+  UINT8                   *PduHdr;\r
 \r
   Private       = ISCSI_DRIVER_DATA_FROM_EXT_SCSI_PASS_THRU (PassThru);\r
   Session       = &Private->Session;\r
   Status        = EFI_SUCCESS;\r
   Tcb           = NULL;\r
   TimeoutEvent  = NULL;\r
+  Timeout       = 0;\r
 \r
   if (Session->State != SESSION_STATE_LOGGED_IN) {\r
     return EFI_DEVICE_ERROR;\r
@@ -2790,15 +2547,6 @@ Returns:
 \r
   if (Packet->Timeout != 0) {\r
     Timeout = MultU64x32 (Packet->Timeout, 2);\r
-    //\r
-    // Start the timeout timer.\r
-    //\r
-    Status = gBS->SetTimer (Conn->TimeoutEvent, TimerRelative, Timeout);\r
-    if (EFI_ERROR (Status)) {\r
-      goto ON_EXIT;\r
-    }\r
-\r
-    TimeoutEvent = Conn->TimeoutEvent;\r
   }\r
 \r
   Status = IScsiNewTcb (Conn, &Tcb);\r
@@ -2815,8 +2563,13 @@ Returns:
   }\r
 \r
   XferContext         = &Tcb->XferContext;\r
-  Buffer              = NetbufGetByte (Pdu, 0, NULL);\r
-  XferContext->Offset = ISCSI_GET_DATASEG_LEN (Buffer);\r
+  PduHdr              = NetbufGetByte (Pdu, 0, NULL);\r
+  if (PduHdr == NULL) {\r
+    Status = EFI_PROTOCOL_ERROR;\r
+    NetbufFree (Pdu);\r
+    goto ON_EXIT;\r
+  }\r
+  XferContext->Offset = ISCSI_GET_DATASEG_LEN (PduHdr);\r
 \r
   //\r
   // Transmit the SCSI Command PDU.\r
@@ -2854,6 +2607,16 @@ Returns:
   InBufferContext.InDataLen = Packet->InTransferLength;\r
 \r
   while (!Tcb->StatusXferd) {\r
+    //\r
+    // Start the timeout timer.\r
+    //\r
+    if (Timeout != 0) {\r
+      Status = gBS->SetTimer (Conn->TimeoutEvent, TimerRelative, Timeout);\r
+      if (EFI_ERROR (Status)) {\r
+        goto ON_EXIT;\r
+      }\r
+      TimeoutEvent = Conn->TimeoutEvent; \r
+    }\r
     //\r
     // try to receive PDU from target.\r
     //\r
@@ -2862,7 +2625,13 @@ Returns:
       goto ON_EXIT;\r
     }\r
 \r
-    switch (ISCSI_GET_OPCODE (NetbufGetByte (Pdu, 0, NULL))) {\r
+    PduHdr = NetbufGetByte (Pdu, 0, NULL);\r
+    if (PduHdr == NULL) {\r
+      Status = EFI_PROTOCOL_ERROR;\r
+      NetbufFree (Pdu);\r
+      goto ON_EXIT;\r
+    }\r
+    switch (ISCSI_GET_OPCODE (PduHdr)) {\r
     case ISCSI_OPCODE_SCSI_DATA_IN:\r
       Status = IScsiOnDataInRcvd (Pdu, Tcb, Packet);\r
       break;\r
@@ -2921,26 +2690,18 @@ ON_EXIT:
   return Status;\r
 }\r
 \r
-EFI_STATUS\r
-IScsiSessionReinstatement (\r
-  IN ISCSI_DRIVER_DATA  *Private\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Reinstate the session on some error.\r
 \r
-Arguments:\r
-\r
-  Private - The iSCSI driver data.\r
+  @param[in, out]  Private The iSCSI driver data.\r
 \r
-Returns:\r
-\r
-  EFI_SUCCES - The session is reinstated from some error.\r
-  other      - Reinstatement failed.\r
-\r
---*/\r
+  @retval EFI_SUCCES  The session is reinstated from some error.\r
+  @retval Other       Reinstatement failed.\r
+**/\r
+EFI_STATUS\r
+IScsiSessionReinstatement (\r
+  IN OUT ISCSI_DRIVER_DATA  *Private\r
+  )\r
 {\r
   ISCSI_SESSION *Session;\r
   EFI_STATUS    Status;\r
@@ -2962,52 +2723,31 @@ Returns:
   return Status;\r
 }\r
 \r
+/**\r
+  Initialize some session parameters before login.\r
+\r
+  @param[in, out]  Session  The iSCSI session.\r
+  @param[in]       Recovery Whether the request is from a fresh new start or recovery.\r
+**/\r
 VOID\r
 IScsiSessionInit (\r
-  IN ISCSI_SESSION  *Session,\r
-  IN BOOLEAN        Recovery\r
+  IN OUT ISCSI_SESSION  *Session,\r
+  IN BOOLEAN            Recovery\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Initialize some session parameters before login.\r
-\r
-Arguments:\r
-\r
-  Session  - The iSCSI session.\r
-  Recovery - Whether the request is from a fresh new start or recovery.\r
-\r
-Returns:\r
-\r
-  None.\r
-\r
---*/\r
 {\r
-  UINT32  Random;\r
-\r
   if (!Recovery) {\r
     Session->Signature  = ISCSI_SESSION_SIGNATURE;\r
     Session->State      = SESSION_STATE_FREE;\r
 \r
-    Random              = NET_RANDOM (NetRandomInitSeed ());\r
-\r
-    Session->ISID[0]    = ISID_BYTE_0;\r
-    Session->ISID[1]    = ISID_BYTE_1;\r
-    Session->ISID[2]    = ISID_BYTE_2;\r
-    Session->ISID[3]    = ISID_BYTE_3;\r
-    Session->ISID[4]    = (UINT8) Random;\r
-    Session->ISID[5]    = (UINT8) (Random >> 8);\r
-\r
     InitializeListHead (&Session->Conns);\r
     InitializeListHead (&Session->TcbList);\r
   }\r
 \r
-  Session->TSIH                 = 0;\r
+  Session->Tsih                 = 0;\r
 \r
   Session->CmdSN                = 1;\r
   Session->InitiatorTaskTag     = 1;\r
-  Session->NextCID              = 1;\r
+  Session->NextCid              = 1;\r
 \r
   Session->TargetPortalGroupTag = 0;\r
   Session->MaxConnections       = ISCSI_MAX_CONNS_PER_SESSION;\r
@@ -3023,26 +2763,18 @@ Returns:
   Session->ErrorRecoveryLevel   = 0;\r
 }\r
 \r
-EFI_STATUS\r
-IScsiSessionAbort (\r
-  IN ISCSI_SESSION  *Session\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
+/**\r
   Abort the iSCSI session, that is, reset all the connection and free the\r
   resources.\r
 \r
-Arguments:\r
-\r
-  Session - The iSCSI session.\r
-\r
-Returns:\r
+  @param[in, out]  Session The iSCSI session.\r
 \r
-  EFI_SUCCES - The session is aborted.\r
-\r
---*/\r
+  @retval EFI_SUCCES  The session is aborted.\r
+**/\r
+EFI_STATUS\r
+IScsiSessionAbort (\r
+  IN OUT ISCSI_SESSION  *Session\r
+  )\r
 {\r
   ISCSI_DRIVER_DATA *Private;\r
   ISCSI_CONNECTION  *Conn;\r