fixed icc build bug
authoreric_tian <eric_tian@de2fecce-e211-0410-80a6-f3fac2684e05>
Tue, 22 Jan 2008 07:14:48 +0000 (07:14 +0000)
committereric_tian <eric_tian@de2fecce-e211-0410-80a6-f3fac2684e05>
Tue, 22 Jan 2008 07:14:48 +0000 (07:14 +0000)
git-svn-id: https://edk2.tianocore.org/svn/edk2/trunk@4609 de2fecce-e211-0410-80a6-f3fac2684e05

edk2/MdeModulePkg/Universal/iScsi/IScsiCHAP.c
edk2/MdeModulePkg/Universal/iScsi/IScsiConfig.c
edk2/MdeModulePkg/Universal/iScsi/IScsiConfig.h
edk2/MdeModulePkg/Universal/iScsi/IScsiDhcp.c
edk2/MdeModulePkg/Universal/iScsi/IScsiDhcp.h
edk2/MdeModulePkg/Universal/iScsi/IScsiIbft.c
edk2/MdeModulePkg/Universal/iScsi/IScsiImpl.h
edk2/MdeModulePkg/Universal/iScsi/IScsiMisc.c
edk2/MdeModulePkg/Universal/iScsi/IScsiProto.c
edk2/MdeModulePkg/Universal/iScsi/IScsiProto.h

index e6ea8b5..3231e61 100644 (file)
@@ -119,7 +119,6 @@ Returns:
 {\r
   EFI_STATUS                Status;\r
   ISCSI_SESSION             *Session;\r
-  ISCSI_SESSION_CONFIG_DATA *ConfigData;\r
   ISCSI_CHAP_AUTH_DATA      *AuthData;\r
   CHAR8                     *Value;\r
   UINT8                     *Data;\r
@@ -137,7 +136,6 @@ Returns:
   ASSERT (Conn->RspQue.BufNum != 0);\r
 \r
   Session     = Conn->Session;\r
-  ConfigData  = &Session->ConfigData;\r
   AuthData    = &Session->AuthData;\r
 \r
   Len         = Conn->RspQue.BufSize;\r
@@ -153,7 +151,7 @@ Returns:
   //\r
   // Build the key-value list from the data segment of the Login Response.\r
   //\r
-  KeyValueList = IScsiBuildKeyValueList (Data, Len);\r
+  KeyValueList = IScsiBuildKeyValueList ((CHAR8 *) Data, Len);\r
   if (KeyValueList == NULL) {\r
     Status = EFI_OUT_OF_RESOURCES;\r
     goto ON_EXIT;\r
@@ -319,7 +317,6 @@ Returns:
   EFI_STATUS                Status;\r
   ISCSI_SESSION             *Session;\r
   ISCSI_LOGIN_REQUEST       *LoginReq;\r
-  ISCSI_SESSION_CONFIG_DATA *ConfigData;\r
   ISCSI_CHAP_AUTH_DATA      *AuthData;\r
   CHAR8                     *Value;\r
   CHAR8                     ValueStr[256];\r
@@ -331,7 +328,6 @@ Returns:
   ASSERT (Conn->CurrentStage == ISCSI_SECURITY_NEGOTIATION);\r
 \r
   Session     = Conn->Session;\r
-  ConfigData  = &Session->ConfigData;\r
   AuthData    = &Session->AuthData;\r
   LoginReq    = (ISCSI_LOGIN_REQUEST *) NetbufGetByte (Pdu, 0, 0);\r
   Status      = EFI_SUCCESS;\r
@@ -387,7 +383,7 @@ Returns:
     //\r
     // CHAP_N=<N>\r
     //\r
-    IScsiAddKeyValuePair (Pdu, ISCSI_KEY_CHAP_NAME, (UINT8 *) &AuthData->AuthConfig.CHAPName);\r
+    IScsiAddKeyValuePair (Pdu, ISCSI_KEY_CHAP_NAME, (CHAR8 *) &AuthData->AuthConfig.CHAPName);\r
     //\r
     // CHAP_R=<R>\r
     //\r
index 67dcb8b..a4c581e 100644 (file)
@@ -866,7 +866,7 @@ Returns:
       ConfigFormEntry->PortTitleToken,\r
       ConfigFormEntry->PortTitleHelpToken,\r
       EFI_IFR_FLAG_CALLBACK,\r
-      KEY_DEVICE_ENTRY_BASE + FormIndex,\r
+      (UINT16)(KEY_DEVICE_ENTRY_BASE + FormIndex),\r
       &UpdateData\r
       );\r
 \r
@@ -913,7 +913,7 @@ Returns:
   EFI_HII_PACKAGE_LIST_HEADER *PackageList;\r
   ISCSI_FORM_CALLBACK_INFO    *CallbackInfo;\r
 \r
-  Status = gBS->LocateProtocol (&gEfiHiiDatabaseProtocolGuid, NULL, &HiiDatabase);\r
+  Status = gBS->LocateProtocol (&gEfiHiiDatabaseProtocolGuid, NULL, (VOID **)&HiiDatabase);\r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
@@ -931,7 +931,7 @@ Returns:
   CallbackInfo->ConfigAccess.RouteConfig = IScsiFormRouteConfig;\r
   CallbackInfo->ConfigAccess.Callback = IScsiFormCallback;\r
 \r
-  Status = gBS->LocateProtocol (&gEfiHiiConfigRoutingProtocolGuid, NULL, &CallbackInfo->ConfigRouting);\r
+  Status = gBS->LocateProtocol (&gEfiHiiConfigRoutingProtocolGuid, NULL, (VOID **)&CallbackInfo->ConfigRouting);\r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
index aab488f..32f9502 100644 (file)
@@ -21,18 +21,10 @@ Abstract:
 #ifndef _ISCSI_CONFIG_H_\r
 #define _ISCSI_CONFIG_H_\r
 \r
-//#include "Tiano.h"\r
-//#include "EfiDriverLib.h"\r
-//#include "Base.h"\r
 #include <Library/HiiLib.h>\r
 #include <Library/IfrSupportLib.h>\r
 #include <Library/DebugLib.h>\r
 #include <Library/BaseLib.h>\r
-//#include "EfiPrintLib.h"\r
-//#include EFI_PROTOCOL_DEFINITION (Hii)\r
-//#include EFI_PROTOCOL_DEFINITION (FormBrowser)\r
-//#include EFI_PROTOCOL_DEFINITION (FormCallback)\r
-\r
 #include <Library/NetLib.h>\r
 #include "IScsiConfigNVDataStruc.h"\r
 \r
index b9b8cea..ec80ff4 100644 (file)
@@ -230,7 +230,7 @@ Returns:
 \r
   Status      = This->Parse (This, Packet, &OptionCount, NULL);\r
   if (Status != EFI_BUFFER_TOO_SMALL) {\r
-    \rreturn EFI_NOT_READY;\r
+    return EFI_NOT_READY;\r
   }\r
 \r
   OptionList = NetAllocatePool (OptionCount * sizeof (EFI_DHCP4_PACKET_OPTION *));\r
@@ -437,7 +437,7 @@ Returns:
   // Ask the server to reply with Netmask, Router, DNS and RootPath options.\r
   //\r
   ParaList->OpCode  = DHCP4_TAG_PARA_LIST;\r
-  ParaList->Length  = ConfigData->NvData.TargetInfoFromDhcp ? 4 : 3;\r
+  ParaList->Length  = (UINT8) (ConfigData->NvData.TargetInfoFromDhcp ? 4 : 3);\r
   ParaList->Data[0] = DHCP4_TAG_NETMASK;\r
   ParaList->Data[1] = DHCP4_TAG_ROUTER;\r
   ParaList->Data[2] = DHCP4_TAG_DNS;\r
index 9312b54..7111b87 100644 (file)
@@ -20,9 +20,7 @@ Abstract:
 #ifndef _ISCSI_DHCP_H_\r
 #define _ISCSI_DHCP_H_\r
 \r
-//#include "Tiano.h"\r
-//#include EFI_PROTOCOL_CONSUMER (Dhcp4)\r
-#include "protocol\Dhcp4.h"\r
+#include <protocol/Dhcp4.h>\r
 \r
 #define DHCP4_TAG_PARA_LIST             55\r
 #define DHCP4_TAG_NETMASK               1\r
index e5d6324..1a60a35 100644 (file)
@@ -101,7 +101,7 @@ Returns:
     // Need expand the control section if more than 2 NIC/Target sections\r
     // exist.\r
     //\r
-    Control->Header.Length += (UINT16) (NumOffset - 4) * sizeof (UINT16);\r
+    Control->Header.Length = (UINT16) (Control->Header.Length + (NumOffset - 4) * sizeof (UINT16));\r
   }\r
 }\r
 \r
@@ -191,7 +191,7 @@ Returns:
   //\r
   // Get the identifier from the handle.\r
   //\r
-  Status = gBS->HandleProtocol (Handle, &mIScsiPrivateGuid, &IScsiIdentifier);\r
+  Status = gBS->HandleProtocol (Handle, &mIScsiPrivateGuid, (VOID **) &IScsiIdentifier);\r
   if (EFI_ERROR (Status)) {\r
     ASSERT (FALSE);\r
     return ;\r
@@ -278,7 +278,7 @@ Returns:
   Status = gBS->HandleProtocol (\r
                   Controller,\r
                   &gEfiDevicePathProtocolGuid,\r
-                  &DevicePath\r
+                  (VOID **)&DevicePath\r
                   );\r
   if (EFI_ERROR (Status)) {\r
     return 0;\r
@@ -293,7 +293,7 @@ Returns:
     return 0;\r
   }\r
 \r
-  Status = gBS->HandleProtocol (PciIoHandle, &gEfiPciIoProtocolGuid, &PciIo);\r
+  Status = gBS->HandleProtocol (PciIoHandle, &gEfiPciIoProtocolGuid, (VOID **)&PciIo);\r
   if (EFI_ERROR (Status)) {\r
     return 0;\r
   }\r
@@ -333,7 +333,7 @@ Returns:
   Status = gBS->HandleProtocol (\r
                   Controller,\r
                   &gEfiSimpleNetworkProtocolGuid,\r
-                  &Snp\r
+                  (VOID **) &Snp\r
                   );\r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -392,7 +392,7 @@ Returns:
   SectionOffset = &Control->NIC0Offset;\r
 \r
   for (Index = 0; Index < HandleCount; Index++) {\r
-    Status = gBS->HandleProtocol (Handles[Index], &mIScsiPrivateGuid, &IScsiIdentifier);\r
+    Status = gBS->HandleProtocol (Handles[Index], &mIScsiPrivateGuid, (VOID **)&IScsiIdentifier);\r
     if (EFI_ERROR (Status)) {\r
       ASSERT (FALSE);\r
       return ;\r
@@ -552,7 +552,7 @@ Returns:
   EFI_ACPI_TABLE_VERSION                    Version;\r
   UINT32                                    Signature;\r
 \r
-  Status = gBS->LocateProtocol (&gEfiAcpiSupportProtocolGuid, NULL, &AcpiSupport);\r
+  Status = gBS->LocateProtocol (&gEfiAcpiSupportProtocolGuid, NULL, (VOID **)&AcpiSupport);\r
   if (EFI_ERROR (Status)) {\r
     return ;\r
   }\r
@@ -563,7 +563,7 @@ Returns:
     Status = AcpiSupport->GetAcpiTable (\r
                             AcpiSupport,\r
                             Index,\r
-                            &Table,\r
+                            (VOID **)&Table,\r
                             &Version,\r
                             &TableHandle\r
                             );\r
@@ -613,7 +613,7 @@ Returns:
     return ;\r
   }\r
 \r
-  Heap = (CHAR8 *) Table + IBFT_HEAP_OFFSET;\r
+  Heap = (UINT8 *) Table + IBFT_HEAP_OFFSET;\r
 \r
   //\r
   // Fill in the various section of the iSCSI Boot Firmware Table.\r
index 715f814..535bc84 100644 (file)
@@ -37,7 +37,7 @@ Abstract:
 \r
 #define ISCSI_SESSION_SIGNATURE EFI_SIGNATURE_32 ('I', 'S', 'S', 'N')\r
 \r
-typedef struct _ISCSI_SESSION {\r
+struct _ISCSI_SESSION {\r
   UINT32                    Signature;\r
 \r
   ISCSI_SESSION_CONFIG_DATA ConfigData;\r
@@ -77,11 +77,11 @@ typedef struct _ISCSI_SESSION {
   BOOLEAN                   DataPDUInOrder;\r
   BOOLEAN                   DataSequenceInOrder;\r
   UINT8                     ErrorRecoveryLevel;\r
-} ISCSI_SESSION;\r
+};\r
 \r
 #define ISCSI_CONNECTION_SIGNATURE  EFI_SIGNATURE_32 ('I', 'S', 'C', 'N')\r
 \r
-typedef struct _ISCSI_CONNECTION {\r
+struct _ISCSI_CONNECTION {\r
   UINT32            Signature;\r
   NET_LIST_ENTRY    Link;\r
 \r
@@ -116,7 +116,7 @@ typedef struct _ISCSI_CONNECTION {
   UINT32            MaxRecvDataSegmentLength;\r
   ISCSI_DIGEST_TYPE HeaderDigest;\r
   ISCSI_DIGEST_TYPE DataDigest;\r
-} ISCSI_CONNECTION;\r
+};\r
 \r
 #define ISCSI_DRIVER_DATA_SIGNATURE EFI_SIGNATURE_32 ('I', 'S', 'D', 'A')\r
 \r
@@ -142,7 +142,7 @@ typedef struct _ISCSI_CONNECTION {
   ISCSI_DRIVER_DATA_SIGNATURE \\r
   )\r
 \r
-typedef struct _ISCSI_DRIVER_DATA {\r
+struct _ISCSI_DRIVER_DATA {\r
   UINT32                          Signature;\r
   EFI_HANDLE                      Image;\r
   EFI_HANDLE                      Controller;\r
@@ -156,6 +156,6 @@ typedef struct _ISCSI_DRIVER_DATA {
   EFI_DEVICE_PATH_PROTOCOL        *DevicePath;\r
 \r
   ISCSI_SESSION                   Session;\r
-} ISCSI_DRIVER_DATA;\r
+};\r
 \r
 #endif\r
index 0347d32..2c0b788 100644 (file)
@@ -167,7 +167,7 @@ Returns:
     Len++;\r
   }\r
 \r
-  return 32 - Len;\r
+  return (UINT8) (32 - Len);\r
 }\r
 \r
 EFI_STATUS\r
@@ -196,6 +196,7 @@ Returns:
   UINT32  Index;\r
   CHAR8   *LunUnitStr[4];\r
   CHAR8   Digit;\r
+  UINTN   Temp;\r
 \r
   NetZeroMem (Lun, 8);\r
   NetZeroMem (LunUnitStr, sizeof (LunUnitStr));\r
@@ -203,7 +204,7 @@ Returns:
   Index         = 0;\r
   LunUnitStr[0] = Str;\r
 \r
-  if (!IsHexDigit (&Digit, *Str)) {\r
+  if (!IsHexDigit ((UINT8 *) &Digit, *Str)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -219,13 +220,13 @@ Returns:
       Index++;\r
 \r
       if (*(Str + 1) != '\0') {\r
-        if (!IsHexDigit (&Digit, *(Str + 1))) {\r
+        if (!IsHexDigit ((UINT8 *) &Digit, *(Str + 1))) {\r
           return EFI_INVALID_PARAMETER;\r
         }\r
 \r
         LunUnitStr[Index] = Str + 1;\r
       }\r
-    } else if (!IsHexDigit (&Digit, *Str)) {\r
+    } else if (!IsHexDigit ((UINT8 *) &Digit, *Str)) {\r
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
@@ -237,7 +238,8 @@ Returns:
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
-    *((UINT16 *) &Lun[Index * 2]) = HTONS (AsciiStrHexToUintn (LunUnitStr[Index]));\r
+    Temp = AsciiStrHexToUintn (LunUnitStr[Index]);\r
+    *((UINT16 *) &Lun[Index * 2]) = HTONS (Temp);\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -470,7 +472,7 @@ Returns:
   UINT32  Index;\r
 \r
   for (Index = 0; Index < Len; Index++) {\r
-    Str[3 * Index]      = NibbleToHexChar (Mac->Addr[Index] >> 4);\r
+    Str[3 * Index]      = NibbleToHexChar ((UINT8) (Mac->Addr[Index] >> 4));\r
     Str[3 * Index + 1]  = NibbleToHexChar (Mac->Addr[Index]);\r
     Str[3 * Index + 2]  = L'-';\r
   }\r
@@ -600,7 +602,7 @@ Returns:
     } else {\r
       Byte = BinBuffer[*BinLength - 1 - Index / 2];\r
       Byte &= 0x0F;\r
-      Byte |= Digit << 4;\r
+      Byte = (UINT8) (Byte | (Digit << 4));\r
     }\r
 \r
     BinBuffer[*BinLength - 1 - Index / 2] = Byte;\r
@@ -809,7 +811,7 @@ Returns:
   Status = gBS->HandleProtocol (\r
                   Private->Controller,\r
                   &gEfiSimpleNetworkProtocolGuid,\r
-                  &Snp\r
+                  (VOID **)&Snp\r
                   );\r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
@@ -905,7 +907,7 @@ Returns:
   Status = gBS->HandleProtocol (\r
                   Tcp4Io->Handle,\r
                   &gEfiDevicePathProtocolGuid,\r
-                  &DevicePath\r
+                  (VOID **)&DevicePath\r
                   );\r
   if (EFI_ERROR (Status)) {\r
     return NULL;\r
@@ -923,7 +925,7 @@ Returns:
         ) {\r
 \r
       DPathNode->Ipv4.LocalPort       = 0;\r
-      DPathNode->Ipv4.StaticIpAddress = !Session->ConfigData.NvData.InitiatorInfoFromDhcp;\r
+      DPathNode->Ipv4.StaticIpAddress = (BOOLEAN) (!Session->ConfigData.NvData.InitiatorInfoFromDhcp);\r
       break;\r
     }\r
 \r
index 4cef3dc..54bad42 100644 (file)
@@ -546,7 +546,7 @@ Returns:
   //\r
   // Allocate the space for the key-value pair.\r
   //\r
-  Data = NetbufAllocSpace (Pdu, TotalLen, NET_BUF_TAIL);\r
+  Data = (CHAR8 *)NetbufAllocSpace (Pdu, TotalLen, NET_BUF_TAIL);\r
   if (Data == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
@@ -668,7 +668,7 @@ Returns:
     //\r
     // Check whether we will issue the stage transition signal?\r
     //\r
-    Conn->TransitInitiated = ISCSI_FLAG_ON (LoginReq, ISCSI_LOGIN_REQ_PDU_FLAG_TRANSIT);\r
+    Conn->TransitInitiated = (BOOLEAN) ISCSI_FLAG_ON (LoginReq, ISCSI_LOGIN_REQ_PDU_FLAG_TRANSIT);\r
   }\r
 \r
   return Nbuf;\r
@@ -746,7 +746,7 @@ Returns:
     // Process the TargetAddress key-value strings in the data segment to update the\r
     // target address info.\r
     //\r
-    Status = IScsiUpdateTargetAddress (Session, DataSeg, DataSegLen);\r
+    Status = IScsiUpdateTargetAddress (Session, (CHAR8 *)DataSeg, DataSegLen);\r
     if (EFI_ERROR (Status)) {\r
       return Status;\r
     }\r
@@ -765,11 +765,11 @@ Returns:
   //\r
   // The status is sucess, extract the wanted fields from the header segment.\r
   //\r
-  Transit                     = ISCSI_FLAG_ON (LoginRsp, ISCSI_LOGIN_RSP_PDU_FLAG_TRANSIT);\r
-  Continue                    = ISCSI_FLAG_ON (LoginRsp, ISCSI_LOGIN_RSP_PDU_FLAG_CONTINUE);\r
+  Transit                     = (BOOLEAN) ISCSI_FLAG_ON (LoginRsp, ISCSI_LOGIN_RSP_PDU_FLAG_TRANSIT);\r
+  Continue                    = (BOOLEAN) ISCSI_FLAG_ON (LoginRsp, ISCSI_LOGIN_RSP_PDU_FLAG_CONTINUE);\r
 \r
-  CurrentStage                = ISCSI_GET_CURRENT_STAGE (LoginRsp);\r
-  NextStage                   = ISCSI_GET_NEXT_STAGE (LoginRsp);\r
+  CurrentStage                = (UINT8) ISCSI_GET_CURRENT_STAGE (LoginRsp);\r
+  NextStage                   = (UINT8) ISCSI_GET_NEXT_STAGE (LoginRsp);\r
 \r
   LoginRsp->InitiatorTaskTag  = NTOHL (LoginRsp->InitiatorTaskTag);\r
 \r
@@ -1283,7 +1283,7 @@ Returns:
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
-  NetbufQueCopy (&Conn->RspQue, 0, Len, Data);\r
+  NetbufQueCopy (&Conn->RspQue, 0, Len, (UINT8 *) Data);\r
 \r
   Status = EFI_PROTOCOL_ERROR;\r
 \r
@@ -1352,7 +1352,7 @@ Returns:
     goto ON_ERROR;\r
   }\r
 \r
-  Session->InitialR2T = Session->InitialR2T || (BOOLEAN) (AsciiStrCmp (Value, "Yes") == 0);\r
+  Session->InitialR2T = (BOOLEAN) (Session->InitialR2T || (AsciiStrCmp (Value, "Yes") == 0));\r
 \r
   //\r
   // ImmediateData, result function is AND.\r
@@ -1362,7 +1362,7 @@ Returns:
     goto ON_ERROR;\r
   }\r
 \r
-  Session->ImmediateData = Session->ImmediateData && (BOOLEAN) (AsciiStrCmp (Value, "Yes") == 0);\r
+  Session->ImmediateData = (BOOLEAN) (Session->ImmediateData && (AsciiStrCmp (Value, "Yes") == 0));\r
 \r
   //\r
   // MaxRecvDataSegmentLength, result function is Mininum.\r
@@ -1422,7 +1422,7 @@ Returns:
     goto ON_ERROR;\r
   }\r
 \r
-  Session->DataPDUInOrder = Session->DataPDUInOrder || (BOOLEAN) (AsciiStrCmp (Value, "Yes") == 0);\r
+  Session->DataPDUInOrder = (BOOLEAN) (Session->DataPDUInOrder || (AsciiStrCmp (Value, "Yes") == 0));\r
 \r
   //\r
   // DataSequenceInorder, result function is OR.\r
@@ -1432,7 +1432,7 @@ Returns:
     goto ON_ERROR;\r
   }\r
 \r
-  Session->DataSequenceInOrder = Session->DataSequenceInOrder || (BOOLEAN) (AsciiStrCmp (Value, "Yes") == 0);\r
+  Session->DataSequenceInOrder = (BOOLEAN) (Session->DataSequenceInOrder || (AsciiStrCmp (Value, "Yes") == 0));\r
 \r
   //\r
   // DefaultTime2Wait, result function is Maximum.\r
@@ -1799,7 +1799,7 @@ Returns:
       //\r
       // Convert the upper-case characters to lower-case ones\r
       //\r
-      Name[Index] = Name[Index] - 'A' + 'a';\r
+      Name[Index] = (CHAR8) (Name[Index] - 'A' + 'a');\r
     }\r
 \r
     if (!NET_IS_LOWER_CASE_CHAR (Name[Index]) &&\r
@@ -2048,7 +2048,7 @@ Returns:
     //\r
     // The CDB exceeds 16 bytes, an extended CDB AHS is required.\r
     //\r
-    AHSLength += ISCSI_ROUNDUP (Packet->CdbLength - 16) + sizeof (ISCSI_ADDITIONAL_HEADER);\r
+    AHSLength = (UINT8) (AHSLength + (ISCSI_ROUNDUP (Packet->CdbLength - 16) + sizeof (ISCSI_ADDITIONAL_HEADER)));\r
   }\r
 \r
   Length    = sizeof (SCSI_COMMAND) + AHSLength;\r
@@ -2769,6 +2769,7 @@ Returns:
   UINT8                   *Data;\r
   ISCSI_IN_BUFFER_CONTEXT InBufferContext;\r
   UINT64                  Timeout;\r
+  UINT8                   *Buffer;\r
 \r
   Private       = ISCSI_DRIVER_DATA_FROM_EXT_SCSI_PASS_THRU (PassThru);\r
   Session       = &Private->Session;\r
@@ -2814,7 +2815,8 @@ Returns:
   }\r
 \r
   XferContext         = &Tcb->XferContext;\r
-  XferContext->Offset = ISCSI_GET_DATASEG_LEN (NetbufGetByte (Pdu, 0, NULL));\r
+  Buffer              = NetbufGetByte (Pdu, 0, NULL);\r
+  XferContext->Offset = ISCSI_GET_DATASEG_LEN (Buffer);\r
 \r
   //\r
   // Transmit the SCSI Command PDU.\r
index 59608fc..bab2ffc 100644 (file)
@@ -117,10 +117,10 @@ typedef enum {
 #define ISCSI_GET_OPCODE(PduHdr)            ((((ISCSI_BASIC_HEADER *) (PduHdr))->OpCode) & ISCSI_OPCODE_MASK)\r
 #define ISCSI_CHECK_OPCODE(PduHdr, Op)      ((((PduHdr)->OpCode) & ISCSI_OPCODE_MASK) == (Op))\r
 #define ISCSI_IMMEDIATE_ON(PduHdr)          ((PduHdr)->OpCode & ISCSI_REQ_IMMEDIATE)\r
-#define ISCSI_SET_FLAG(PduHdr, Flag)        (((ISCSI_BASIC_HEADER *) (PduHdr))->Flags |= (Flag))\r
+#define ISCSI_SET_FLAG(PduHdr, Flag)        (((ISCSI_BASIC_HEADER *) (PduHdr))->Flags |= (BOOLEAN)(Flag))\r
 #define ISCSI_CLEAR_FLAG(PduHdr, Flag)      (((ISCSI_BASIC_HEADER *) (PduHdr))->Flags &= ~(Flag))\r
 #define ISCSI_FLAG_ON(PduHdr, Flag)         ((((ISCSI_BASIC_HEADER *) (PduHdr))->Flags & (Flag)) == (Flag))\r
-#define ISCSI_SET_STAGES(PduHdr, Cur, Nxt)  ((PduHdr)->Flags |= ((Cur) << 2 | (Nxt)))\r
+#define ISCSI_SET_STAGES(PduHdr, Cur, Nxt)  ((PduHdr)->Flags = (UINT8) ((PduHdr)->Flags | ((Cur) << 2 | (Nxt))))\r
 #define ISCSI_GET_CURRENT_STAGE(PduHdr)     (((PduHdr)->Flags >> 2) & 0x3)\r
 #define ISCSI_GET_NEXT_STAGE(PduHdr)        (((PduHdr)->Flags) & 0x3)\r
 \r
@@ -129,9 +129,9 @@ typedef enum {
 \r
 #define HTON24(Dst, Src) \\r
   do { \\r
-    (Dst)[0]  = (UINT8) ((Src) >> 16) & 0xFF; \\r
-    (Dst)[1]  = (UINT8) ((Src) >> 8) & 0xFF; \\r
-    (Dst)[2]  = (UINT8) (Src) & 0xFF; \\r
+    (Dst)[0]  = (UINT8) (((Src) >> 16) & 0xFF); \\r
+    (Dst)[1]  = (UINT8) (((Src) >> 8) & 0xFF); \\r
+    (Dst)[2]  = (UINT8) ((Src) & 0xFF); \\r
   } while (0);\r
 \r
 #define NTOH24(src)                         (((src)[0] << 16) | ((src)[1] << 8) | ((src)[2]))\r