Reviewed-by: Star Zeng <star.z...@intel.com>

-----Original Message-----
From: Qiu, Shumin 
Sent: Thursday, June 25, 2015 3:47 PM
To: edk2-devel@lists.sourceforge.net
Cc: Zeng, Star; Fan, Jeff; Gao, Liming; Ni, Ruiyu; Tian, Feng
Subject: [PATCH 02/13] MdeModulePkg/Library/DxeCorePerformanceLib: Use safe 
string functions to refine code.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Qiu Shumin <shumin....@intel.com>
---
 .../Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c      | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c 
b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c
index 90053e7..7ebdb29 100644
--- a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c
+++ b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c
@@ -10,7 +10,7 @@
   This library is mainly used by DxeCore to start performance logging to 
ensure that
   Performance Protocol is installed at the very beginning of DXE phase.
 
-Copyright (c) 2006 - 2012, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2006 - 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 @@ -191,10 +191,10 
@@ StartGaugeEx (
   GaugeEntryExArray[Index].Handle = (EFI_PHYSICAL_ADDRESS) (UINTN) Handle;
 
   if (Token != NULL) {
-    AsciiStrnCpy (GaugeEntryExArray[Index].Token, Token, 
DXE_PERFORMANCE_STRING_LENGTH);
+    AsciiStrCpyS (GaugeEntryExArray[Index].Token, 
+ DXE_PERFORMANCE_STRING_SIZE, Token);
   }
   if (Module != NULL) {
-    AsciiStrnCpy (GaugeEntryExArray[Index].Module, Module, 
DXE_PERFORMANCE_STRING_LENGTH);
+    AsciiStrCpyS (GaugeEntryExArray[Index].Module, 
+ DXE_PERFORMANCE_STRING_SIZE, Module);
   }
 
   GaugeEntryExArray[Index].EndTimeStamp = 0; @@ -463,8 +463,8 @@ 
InternalGetPeiPerformance (
     NumberOfEntries = LogHob->NumberOfEntries;
     for (Index = 0; Index < NumberOfEntries; Index++) {
       GaugeEntryExArray[Index].Handle         = LogEntryArray[Index].Handle;
-      AsciiStrnCpy (GaugeEntryExArray[Index].Token,  
LogEntryArray[Index].Token,  DXE_PERFORMANCE_STRING_LENGTH);
-      AsciiStrnCpy (GaugeEntryExArray[Index].Module, 
LogEntryArray[Index].Module, DXE_PERFORMANCE_STRING_LENGTH);
+      AsciiStrCpyS (GaugeEntryExArray[Index].Token,  
DXE_PERFORMANCE_STRING_SIZE, LogEntryArray[Index].Token);
+      AsciiStrCpyS (GaugeEntryExArray[Index].Module, 
+ DXE_PERFORMANCE_STRING_SIZE, LogEntryArray[Index].Module);
       GaugeEntryExArray[Index].StartTimeStamp = 
LogEntryArray[Index].StartTimeStamp;
       GaugeEntryExArray[Index].EndTimeStamp   = 
LogEntryArray[Index].EndTimeStamp;
       GaugeEntryExArray[Index].Identifier     = 0;
--
1.9.5.msysgit.1



------------------------------------------------------------------------------
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