Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hao Wu <hao.a...@intel.com>
Reviewed-by: Jaben Carsey <jaben.car...@intel.com>
---
 PerformancePkg/Dp_App/DpUtilities.c | 27 ++++++++++++++++++---------
 1 file changed, 18 insertions(+), 9 deletions(-)

diff --git a/PerformancePkg/Dp_App/DpUtilities.c 
b/PerformancePkg/Dp_App/DpUtilities.c
index 73666aa..38d2293 100644
--- a/PerformancePkg/Dp_App/DpUtilities.c
+++ b/PerformancePkg/Dp_App/DpUtilities.c
@@ -1,7 +1,7 @@
 /** @file
   Utility functions used by the Dp application.
 
-  Copyright (c) 2009 - 2014, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2009 - 2015, Intel Corporation. All rights reserved.<BR>
   This program and the accompanying materials
   are licensed and made available under the terms and conditions of the BSD 
License
   which accompanies this distribution.  The full text of the license may be 
found at
@@ -159,7 +159,7 @@ GetShortPdbFileName (
   ZeroMem (UnicodeBuffer, DXE_PERFORMANCE_STRING_LENGTH * sizeof (CHAR16));
 
   if (PdbFileName == NULL) {
-    StrCpy (UnicodeBuffer, L" ");
+    StrCpyS (UnicodeBuffer, DXE_PERFORMANCE_STRING_SIZE, L" ");
   } else {
     StartIndex = 0;
     for (EndIndex = 0; PdbFileName[EndIndex] != 0; EndIndex++)
@@ -290,8 +290,11 @@ GetNameFromHandle (
                                );
     SafeFreePool (BestLanguage);
     if (!EFI_ERROR (Status)) {
-      StrnCpy (mGaugeString, StringPtr, DP_GAUGE_STRING_LENGTH);
-      mGaugeString[DP_GAUGE_STRING_LENGTH] = 0;
+      StrCpyS (
+        mGaugeString,
+        DP_GAUGE_STRING_LENGTH + 1,
+        StringPtr
+        );
       return;
     }
   }
@@ -334,8 +337,11 @@ GetNameFromHandle (
         //
         // Method 3. Get the name string from FFS UI section
         //
-        StrnCpy (mGaugeString, NameString, DP_GAUGE_STRING_LENGTH);
-        mGaugeString[DP_GAUGE_STRING_LENGTH] = 0;
+        StrCpyS (
+          mGaugeString,
+          DP_GAUGE_STRING_LENGTH + 1,
+          NameString
+          );
         FreePool (NameString);
       } else {
         //
@@ -350,8 +356,11 @@ GetNameFromHandle (
       //
       NameString = ConvertDevicePathToText (LoadedImageDevicePath, TRUE, 
FALSE);
       if (NameString != NULL) {
-        StrnCpy (mGaugeString, NameString, DP_GAUGE_STRING_LENGTH);
-        mGaugeString[DP_GAUGE_STRING_LENGTH] = 0;
+        StrCpyS (
+          mGaugeString,
+          DP_GAUGE_STRING_LENGTH + 1,
+          NameString
+          );
         FreePool (NameString);
         return;
       }
@@ -363,7 +372,7 @@ GetNameFromHandle (
   //
   StringPtr = HiiGetString (gHiiHandle, STRING_TOKEN (STR_DP_ERROR_NAME), 
NULL);
   ASSERT (StringPtr != NULL);
-  StrCpy (mGaugeString, StringPtr);
+  StrCpyS (mGaugeString, DP_GAUGE_STRING_LENGTH + 1, StringPtr);
   FreePool (StringPtr);
   return;
 }
-- 
1.9.5.msysgit.0


------------------------------------------------------------------------------
Monitor 25 network devices or servers for free with OpManager!
OpManager is web-based network management software that monitors 
network devices and physical & virtual servers, alerts via email & sms 
for fault. Monitor 25 devices for free with no restriction. Download now
http://ad.doubleclick.net/ddm/clk/292181274;119417398;o
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to