Changed the code style. it could pass EBC and ICC build.
authorvanjeff <vanjeff@6f19259b-4bc3-4df7-8a09-765794883524>
Thu, 6 Sep 2007 02:53:47 +0000 (02:53 +0000)
committervanjeff <vanjeff@6f19259b-4bc3-4df7-8a09-765794883524>
Thu, 6 Sep 2007 02:53:47 +0000 (02:53 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@3781 6f19259b-4bc3-4df7-8a09-765794883524

MdeModulePkg/Include/Library/IpIoLib.h
MdeModulePkg/Library/DxeIpIoLib/DxeIpIoLib.c

index e2c9f67..682036f 100644 (file)
@@ -68,12 +68,10 @@ typedef enum {
 } ICMP_ERROR;\r
 \r
 typedef struct _ICMP_ERROR_INFO {\r
 } ICMP_ERROR;\r
 \r
 typedef struct _ICMP_ERROR_INFO {\r
-  EFI_STATUS  Error;\r
   BOOLEAN     IsHard;\r
   BOOLEAN     Notify;\r
 } ICMP_ERROR_INFO;\r
 
   BOOLEAN     IsHard;\r
   BOOLEAN     Notify;\r
 } ICMP_ERROR_INFO;\r
 
-
 #define EFI_IP4_HEADER_LEN(HdrPtr) ((HdrPtr)->HeaderLength << 2)
 
 extern EFI_IP4_CONFIG_DATA  mIpIoDefaultIpConfigData;
 #define EFI_IP4_HEADER_LEN(HdrPtr) ((HdrPtr)->HeaderLength << 2)
 
 extern EFI_IP4_CONFIG_DATA  mIpIoDefaultIpConfigData;
index cc86971..608bdd2 100644 (file)
@@ -58,6 +58,19 @@ EFI_IP4_CONFIG_DATA  mIpIoDefaultIpConfigData = {
   0\r
 };\r
 \r
   0\r
 };\r
 \r
+STATIC ICMP_ERROR_INFO  mIcmpErrMap[10] = {\r
+  {FALSE, TRUE},\r
+  {FALSE, TRUE},\r
+  {TRUE, TRUE},\r
+  {TRUE, TRUE},\r
+  {TRUE, TRUE},\r
+  {FALSE, TRUE},\r
+  {FALSE, TRUE},\r
+  {FALSE, TRUE},\r
+  {FALSE, FALSE},\r
+  {FALSE, TRUE}\r
+};\r
+\r
 STATIC\r
 VOID\r
 EFIAPI\r
 STATIC\r
 VOID\r
 EFIAPI\r
@@ -1271,58 +1284,42 @@ IpIoGetIcmpErrStatus (
   OUT BOOLEAN     *Notify OPTIONAL\r
   )\r
 {\r
   OUT BOOLEAN     *Notify OPTIONAL\r
   )\r
 {\r
-  ICMP_ERROR_INFO  IcmpErrMap[10];\r
-\r
-  IcmpErrMap[0].Error  = EFI_NETWORK_UNREACHABLE;\r
-  IcmpErrMap[0].IsHard = FALSE;\r
-  IcmpErrMap[0].Notify = TRUE;\r
-\r
-  IcmpErrMap[1].Error = EFI_HOST_UNREACHABLE;\r
-  IcmpErrMap[1].IsHard = FALSE;\r
-  IcmpErrMap[1].Notify = TRUE;\r
-\r
-  IcmpErrMap[2].Error = EFI_PROTOCOL_UNREACHABLE;\r
-  IcmpErrMap[2].IsHard = TRUE;\r
-  IcmpErrMap[2].Notify = TRUE;\r
-\r
-  IcmpErrMap[3].Error = EFI_PORT_UNREACHABLE;\r
-  IcmpErrMap[3].IsHard = TRUE;\r
-  IcmpErrMap[3].Notify = TRUE;\r
-\r
-  IcmpErrMap[4].Error = EFI_ICMP_ERROR;\r
-  IcmpErrMap[4].IsHard = TRUE;\r
-  IcmpErrMap[4].Notify = TRUE;\r
-\r
-  IcmpErrMap[5].Error = EFI_ICMP_ERROR;\r
-  IcmpErrMap[5].IsHard = FALSE;\r
-  IcmpErrMap[5].Notify = TRUE;\r
+  ASSERT ((IcmpError >= ICMP_ERR_UNREACH_NET) && (IcmpError <= ICMP_ERR_PARAMPROB));\r
 \r
 \r
-  IcmpErrMap[6].Error = EFI_HOST_UNREACHABLE;\r
-  IcmpErrMap[6].IsHard = FALSE;\r
-  IcmpErrMap[6].Notify = TRUE;\r
+  if (IsHard != NULL) {\r
+    *IsHard = mIcmpErrMap[IcmpError].IsHard;\r
+  }\r
 \r
 \r
-  IcmpErrMap[7].Error = EFI_HOST_UNREACHABLE;\r
-  IcmpErrMap[7].IsHard = FALSE;\r
-  IcmpErrMap[7].Notify = TRUE;\r
+  if (Notify != NULL) {\r
+    *Notify = mIcmpErrMap[IcmpError].Notify;\r
+  }\r
 \r
 \r
-  IcmpErrMap[8].Error = EFI_ICMP_ERROR;\r
-  IcmpErrMap[8].IsHard = FALSE;\r
-  IcmpErrMap[8].Notify = FALSE;\r
+  switch (IcmpError) {\r
+  case ICMP_ERR_UNREACH_NET:\r
+    return  EFI_NETWORK_UNREACHABLE;\r
 \r
 \r
-  IcmpErrMap[9].Error = EFI_ICMP_ERROR;\r
-  IcmpErrMap[9].IsHard = FALSE;\r
-  IcmpErrMap[9].Notify = TRUE;\r
+  case ICMP_ERR_TIMXCEED_INTRANS:\r
+  case ICMP_ERR_TIMXCEED_REASS:\r
+  case ICMP_ERR_UNREACH_HOST:\r
+    return  EFI_HOST_UNREACHABLE;\r
 \r
 \r
-  ASSERT ((IcmpError >= ICMP_ERR_UNREACH_NET) && (IcmpError <= ICMP_ERR_PARAMPROB));\r
+  case ICMP_ERR_UNREACH_PROTOCOL:\r
+    return  EFI_PROTOCOL_UNREACHABLE;\r
 \r
 \r
-  if (IsHard != NULL) {\r
-    *IsHard = IcmpErrMap[IcmpError].IsHard;\r
-  }\r
+  case ICMP_ERR_UNREACH_PORT:\r
+    return  EFI_PORT_UNREACHABLE;\r
 \r
 \r
-  if (Notify != NULL) {\r
-    *Notify = IcmpErrMap[IcmpError].Notify;\r
+  case ICMP_ERR_MSGSIZE:\r
+  case ICMP_ERR_UNREACH_SRCFAIL:\r
+  case ICMP_ERR_QUENCH:\r
+  case ICMP_ERR_PARAMPROB:\r
+    return  EFI_ICMP_ERROR;\r
   }\r
 \r
   }\r
 \r
-  return IcmpErrMap[IcmpError].Error;\r
+  //\r
+  // will never run here!\r
+  //\r
+  ASSERT (FALSE);\r
+  return EFI_UNSUPPORTED;\r
 }\r
 \r
 }\r
 \r