Fix the comments to follow UEFI Spec regarding how to check an EFI_HANDLE is valid...

[mirror_edk2.git] / MdeModulePkg / Universal / Network / ArpDxe / ArpDriver.c

diff --git a/MdeModulePkg/Universal/Network/ArpDxe/ArpDriver.c b/MdeModulePkg/Universal/Network/ArpDxe/ArpDriver.c
index d2e7a59ca6f46079d5512c14035ea4b6fb112e11..37042d11cd1ebfa6dcc2253307011ab344e30e9a 100644 (file)
--- a/MdeModulePkg/Universal/Network/ArpDxe/ArpDriver.c
+++ b/MdeModulePkg/Universal/Network/ArpDxe/ArpDriver.c
@@ -1,8 +1,8 @@
 /** @file\r
   ARP driver functions.\r
   \r
-Copyright (c) 2006 - 2009, Intel Corporation.<BR>\r
-All rights reserved. This program and the accompanying materials\r
+Copyright (c) 2006 - 2011, 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<BR>\r
 http://opensource.org/licenses/bsd-license.php\r
@@ -54,6 +54,14 @@ ArpCreateService (
 \r
   ArpService->Signature = ARP_SERVICE_DATA_SIGNATURE;\r
 \r
+  //\r
+  // Init the lists.\r
+  //\r
+  InitializeListHead (&ArpService->ChildrenList);\r
+  InitializeListHead (&ArpService->PendingRequestTable);\r
+  InitializeListHead (&ArpService->DeniedCacheTable);\r
+  InitializeListHead (&ArpService->ResolvedCacheTable);\r
+\r
   //\r
   // Init the servicebinding protocol members.\r
   //\r
@@ -168,17 +176,6 @@ ArpCreateService (
                   TimerPeriodic,\r
                   ARP_PERIODIC_TIMER_INTERVAL\r
                   );\r
-  if (EFI_ERROR (Status)) {\r
-    goto ERROR_EXIT;\r
-  }\r
-\r
-  //\r
-  // Init the lists.\r
-  //\r
-  InitializeListHead (&ArpService->ChildrenList);\r
-  InitializeListHead (&ArpService->PendingRequestTable);\r
-  InitializeListHead (&ArpService->DeniedCacheTable);\r
-  InitializeListHead (&ArpService->ResolvedCacheTable);\r
 \r
 ERROR_EXIT:\r
 \r
@@ -651,7 +648,7 @@ ERROR:
   @retval EFI_SUCCES            The protocol was removed from ChildHandle.\r
   @retval EFI_UNSUPPORTED       ChildHandle does not support the protocol that is \r
                                 being removed.\r
-  @retval EFI_INVALID_PARAMETER Child handle is not a valid UEFI Handle.\r
+  @retval EFI_INVALID_PARAMETER Child handle is NULL.\r
   @retval EFI_ACCESS_DENIED     The protocol could not be removed from the ChildHandle\r
                                 because its services are being used.\r
   @retval other                 The child handle was not destroyed\r