Use string pointer instead string buffer to avoid string copy operation.

[mirror_edk2.git] / NetworkPkg / Dhcp6Dxe / ComponentName.c

diff --git a/NetworkPkg/Dhcp6Dxe/ComponentName.c b/NetworkPkg/Dhcp6Dxe/ComponentName.c
index 352df087f91b11afd0e881b783de6cf8555467dc..927a7fe7ecc7468fc7c50c39a33ca0dd8c3aa308 100644 (file)
--- a/NetworkPkg/Dhcp6Dxe/ComponentName.c
+++ b/NetworkPkg/Dhcp6Dxe/ComponentName.c
@@ -1,7 +1,7 @@
 /** @file\r
   UEFI Component Name(2) protocol implementation for Dhcp6 driver.\r
 \r
-  Copyright (c) 2009 - 2012, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2009 - 2014, 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
@@ -259,7 +259,7 @@ UpdateName (
 {\r
   EFI_STATUS                       Status;\r
   EFI_DHCP6_MODE_DATA              Dhcp6ModeData;\r
-  CHAR16                           HandleName[64];\r
+  CHAR16                           *HandleName;\r
 \r
   if (Dhcp6 == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -279,9 +279,12 @@ UpdateName (
   }\r
   \r
   if (Dhcp6ModeData.Ia == NULL) {\r
-    UnicodeSPrint (HandleName, sizeof (HandleName), L"DHCPv6 (No configured IA)");\r
+    HandleName = L"DHCPv6 (No configured IA)";\r
   } else {\r
-    StrCpy (HandleName, mDhcp6ControllerName[Dhcp6ModeData.Ia->State]);\r
+    if (Dhcp6ModeData.Ia->State > Dhcp6Rebinding) {\r
+      return EFI_DEVICE_ERROR;\r
+    }\r
+    HandleName = mDhcp6ControllerName[Dhcp6ModeData.Ia->State];\r
   }\r
   \r
   Status = AddUnicodeString2 (\r