[edk2] [PATCH] MdePkg: Fix a comment typo in Include/Ppi/MpServices.h

2015-07-13 Thread Hao Wu
The summary of EFI_PEI_MP_SERVICES_WHOAMI is modified to show the correct description of this service. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- MdePkg/Include/Ppi/MpServices.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdePkg

[edk2] [PATCH] MdePkg: Add BMC device path definition and its node/text conversion

2015-07-13 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Feng Tian --- MdePkg/Include/Protocol/DevicePath.h | 22 ++ .../Library/UefiDevicePathLib/DevicePathFromText.c | 35 ++ .../Library/UefiDevicePathLib

[edk2] [PATCH v2] MdePkg UefiLib: Code style alignment with IntelFrameworkPkg

2015-07-09 Thread Hao Wu
MdePkg/Library/UefiLib/UefiLibPrint.c is modified to keep code style consistency with IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLibPrint.c Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- MdePkg/Library/UefiLib/UefiLibPrint.c | 4 +++- 1 file changed, 3

[edk2] [PATCH v2] PerformancePkg Dp_App: Fix ASSERT in GetNameFromHandle

2015-07-09 Thread Hao Wu
(mGaugeString, DP_GAUGE_STRING_LENGTH + 1, NameString); (twice) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Star Zeng --- PerformancePkg/Dp_App/DpUtilities.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a

[edk2] [PATCH v2] UefiCpuPkg S3Resume2Pei: Fix ASSERT in WriteToOsS3PerformanceData

2015-07-09 Thread Hao Wu
noCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c b/UefiCpuPkg/Universa

[edk2] [PATCH v2 4/5] IntelFrameworkModulePkg BootMngr: Fix potential read over memory boundary

2015-07-09 Thread Hao Wu
ory region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Universal/BdsDxe/BootMngr/BootManager.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/IntelFrameworkModulePkg

[edk2] [PATCH v2 0/5] Resolve issues brought by r17732-r17739

2015-07-09 Thread Hao Wu
when buffer overflow occurs, the patches will use StrnCpyS/StrnCatS instead to resolve this issue. Hao Wu (5): IntelFrameworkModulePkg GenericBdsLib: Potential read over memory boudary IntelFrameworkModulePkg BdsDxe: Fix ASSERT in BdsMemoryTest IntelFrameworkModulePkg BootMaint: Fix potent

[edk2] [PATCH v2 5/5] IntelFrameworkModulePkg DeviceMngr: Potential read over memory boundary

2015-07-09 Thread Hao Wu
ontributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/IntelFrameworkModulePkg/Univer

[edk2] [PATCH v2 2/5] IntelFrameworkModulePkg BdsDxe: Fix ASSERT in BdsMemoryTest

2015-07-09 Thread Hao Wu
(StrTotalMemory, StrTotalMemorySize / sizeof (CHAR16), TmpStr); Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Universal/BdsDxe/MemoryTest.c | 14 -- 1 file changed, 12 insertions

[edk2] [PATCH v2 3/5] IntelFrameworkModulePkg BootMaint: Fix potential read over memory boundary

2015-07-09 Thread Hao Wu
Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Universal/BdsDxe/BootMaint/BootOption.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/IntelFrameworkModulePkg/Universal/BdsDxe/BootMaint/BootOption.c

[edk2] [PATCH v2 1/5] IntelFrameworkModulePkg GenericBdsLib: Potential read over memory boudary

2015-07-09 Thread Hao Wu
out of the scope of the constant string. Potential risk for the constant string allocated at the boundary of memory region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- .../Library/GenericBdsLib

[edk2] [PATCH v2] IntelFrameworkPkg FrameworkUefiLib: Fix ASSERT in CatVSPrint

2015-07-08 Thread Hao Wu
read contents out of the scope of 'String'. Potential risk for 'String' allocated at the boundary of memory region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLibPrint.c | 8 +++- 1 f

[edk2] [PATCH] PerformancePkg Dp_App: Resolve issue brought by r17745

2015-07-07 Thread Hao Wu
); (twice) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Star Zeng --- PerformancePkg/Dp_App/DpUtilities.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/PerformancePkg/Dp_App/DpUtilities.c b/PerformancePkg/Dp_App

[edk2] [PATCH] IntelFrameworkPkg FrameworkUefiLib: Resolve issue brought by r17740

2015-07-07 Thread Hao Wu
7;. Potential risk for 'String' allocated at the boundary of memory region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin --- IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLibPrint.c | 6 +- 1 file changed, 5 in

[edk2] [PATCH] UefiCpuPkg S3Resume2Pei: Resolve issue brought by r17744

2015-07-06 Thread Hao Wu
AsciiStrCpyS (PerfData->Token, PERF_TOKEN_SIZE, Token); The above using of AsciiStrCpyS will cause ASSERT if Token is longer than PerfData->Token. Therefore, AsciiStrnCatS is used here to resolve the issue. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Re

[edk2] [PATCH 1/5] IntelFrameworkModulePkg GenericBdsLib: Resolve issue brought by r17733

2015-07-06 Thread Hao Wu
sk for the constant string allocated at the boundary of memory region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- .../Library/GenericBdsLib/BdsMisc.c | 20 1 file c

[edk2] [PATCH 3/5] IntelFrameworkModulePkg BootMaint: Resolve issue brought by r17736

2015-07-06 Thread Hao Wu
Str = AllocateCopyPool (MaxLen * sizeof (CHAR16), Str1); The above using of AllocateCopyPool() will read contents out of the scope of Str1. Potential risk for Str1 allocated at the boundary of memory region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu

[edk2] [PATCH 2/5] IntelFrameworkModulePkg BdsDxe: Resolve issue brought by r17735

2015-07-06 Thread Hao Wu
), TmpStr); Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Universal/BdsDxe/MemoryTest.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a

[edk2] [PATCH 0/5] Resolve issues brought by r17732-r17739

2015-07-06 Thread Hao Wu
Potential risk when 'Buffer' is allocated at the boundary of memory region. 2. Some replacement of StrnCpy/StrnCat with StrCpyS/StrCatS functions These changes will cause ASSERT when buffer overflow occurs, the patches will use StrnCpyS/StrnCatS instead to resolve th

[edk2] [PATCH 5/5] IntelFrameworkModulePkg DeviceMngr: Resolve issue brought by r17738

2015-07-06 Thread Hao Wu
Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/IntelFrameworkModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.c b/IntelFramewor

[edk2] [PATCH 4/5] IntelFrameworkModulePkg BootMngr: Resolve issue brought by r17737

2015-07-06 Thread Hao Wu
Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Qiu Shumin Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Universal/BdsDxe/BootMngr/BootManager.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/IntelFrameworkModulePkg/Universal/BdsDxe/BootMngr/BootMa

[edk2] [PATCH] FatPkg EnhancedFatDxe: Use safe string functions

2015-07-01 Thread Hao Wu
Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Ruiyu Ni --- EnhancedFatDxe/DirectoryManage.c | 18 +++--- EnhancedFatDxe/Fat.h | 5 +++-- EnhancedFatDxe/FileName.c| 14 +++--- EnhancedFatDxe/Hash.c| 9 ++--- 4 files changed, 31 insertions(+), 15

[edk2] [PATCH 8/9] EdkCompatibilityPkg: Remove ZeroGuid definition in Datahub2SmbiosThunk

2015-06-25 Thread Hao Wu
MdeModulePkg has defined gZeroGuid in 'Guid/ZeroGuid.h', therefore, the ZeroGuid defined in \Compatibility\PiSmbiosRecordOnDataHubSmbiosRecordThunk is redundant. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Ja

[edk2] [PATCH 9/9] EdkCompatibilityPkg: Remove gZeroGuid def in FrameworkHiiOnUefiHiiThunk

2015-06-25 Thread Hao Wu
MdeModulePkg has defined gZeroGuid in 'Guid/ZeroGuid.h', therefore, the gZeroGuid defined in Compatibility/FrameworkHiiOnUefiHiiThunk is redundent. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Ja

[edk2] [PATCH 4/9] NetworkPkg: Remove mZeroGuid definition in IpSecDxe

2015-06-25 Thread Hao Wu
mZeroGuid is defined but not used in IpSecDxe. All the header files in IpSecDxe which expose mZeroGuid are not included externally as well. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- NetworkPkg/IpSecDxe

[edk2] [PATCH 0/9] Remove redundant ZeroGuid definitions

2015-06-25 Thread Hao Wu
MdeModulePkg has defined gZeroGuid in Include/Guid/ZeroGuid.h, therefore, the ZeroGuid/gZeroGuid/mZeroGuid/mTrEEZeroGuid defined in current code is redundent. Hao Wu (9): IntelFrameworkModulePkg: Remove gZeroGuid definition in DataHubDxe MdeModulePkg: Remove gZeroGuid definition in

[edk2] [PATCH 1/9] IntelFrameworkModulePkg: Remove gZeroGuid definition in DataHubDxe

2015-06-25 Thread Hao Wu
MdeModulePkg has defined gZeroGuid in 'Guid/ZeroGuid.h', therefore, the gZeroGuid defined in DataHubDxe is redundant. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- IntelFrameworkModulePkg

[edk2] [PATCH 5/9] SecurityPkg: Remove mTrEEZeroGuid definition in DxeTpm2MeasureBootLib

2015-06-25 Thread Hao Wu
MdeModulePkg has defined gZeroGuid in 'Guid/ZeroGuid.h', therefore, the mTrEEZeroGuid defined in DxeTpm2MeasureBootLib is redundant. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- SecurityP

[edk2] [PATCH 2/9] MdeModulePkg: Remove gZeroGuid definition in SetupBrowserDxe

2015-06-25 Thread Hao Wu
MdeModulePkg has defined gZeroGuid in 'Guid/ZeroGuid.h', therefore, the gZeroGuid defined in SetupBrowserDxe is redundant. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- MdeModulePkg/Universal/Setup

[edk2] [PATCH 6/9] SecurityPkg: Remove mZeroGuid definition in DxeTpmMeasureBootLib

2015-06-25 Thread Hao Wu
MdeModulePkg has defined gZeroGuid in 'Guid/ZeroGuid.h', therefore, the mZeroGuid defined in DxeTpmMeasureBootLib is redundant. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- SecurityP

[edk2] [PATCH 3/9] MdeModulePkg: Remove mZeroGuid definition in Universal/Variable/RuntimeDxe

2015-06-25 Thread Hao Wu
MdeModulePkg has defined gZeroGuid in 'Guid/ZeroGuid.h', therefore, the mZeroGuid defined in Universal/Variable/RuntimeDxe is redundant. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- MdeModulePkg

[edk2] [PATCH 7/9] SecurityPkg: Remove mZeroGuid def in VariableAuthenticated/RuntimeDxe

2015-06-25 Thread Hao Wu
MdeModulePkg has defined gZeroGuid in 'Guid/ZeroGuid.h', therefore, the mZeroGuid defined in VariableAuthenticated/RuntimeDxe is redundant. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- S

[edk2] [PATCH] UefiCpuPkg S3Resume2Pei: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jaben Carsey --- UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c b/UefiCpuPkg

[edk2] [PATCH 2/2] PerformancePkg Dp_App: Resolve buffer size mismatch

2015-06-25 Thread Hao Wu
introduce a potential risk of buffer overflow. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Star Zeng --- PerformancePkg/Dp_App/DpUtilities.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/PerformancePkg/Dp_App

[edk2] [PATCH 0/2] PerformancePkg Dp_App: Use safe string functions

2015-06-25 Thread Hao Wu
The first patch is to use safe string functions in Dp_App. The second patch is to resolve a buffer size mismatch problem in DpUtilities.c. Details are available in the log message in each commit. Hao Wu (2): PerformancePkg Dp_App: Use safe string functions PerformancePkg Dp_App: Resolve

[edk2] [PATCH 1/2] PerformancePkg Dp_App: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jaben Carsey --- PerformancePkg/Dp_App/DpUtilities.c | 27 ++- 1 file changed, 18 insertions(+), 9 deletions(-) diff --git a/PerformancePkg/Dp_App/DpUtilities.c b/PerformancePkg

[edk2] [PATCH v2 0/2] MdePkg: Remove unsafe string functions

2015-06-25 Thread Hao Wu
. It's been fixed in this patch. Hao Wu (2): MdePkg UefiDevicePathLib: Remove unsafe string functions MdePkg UefiLib: Use safe string functions MdePkg/Library/UefiDevicePathLib/DevicePathFromText.c | 17 + MdePkg/Library/UefiLib/Console.c

[edk2] [PATCH v2] IntelFrameworkPkg FrameworkUefiLib: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLibPrint.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/IntelFrameworkPkg/Library

[edk2] [PATCH v2 2/2] MdePkg UefiLib: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- MdePkg/Library/UefiLib/Console.c | 6 +++--- MdePkg/Library/UefiLib/UefiLibPrint.c | 8 ++-- 2 files changed, 5 insertions(+), 9 deletions(-) diff --git

[edk2] [PATCH v2 1/2] MdePkg UefiDevicePathLib: Remove unsafe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Jaben Carsey --- MdePkg/Library/UefiDevicePathLib/DevicePathFromText.c | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/MdePkg/Library

[edk2] [PATCH v2 4/8] IntelFrameworkModulePkg BdsDxe: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan Reviewed-by: Jaben Carsey --- IntelFrameworkModulePkg/Universal/BdsDxe/FrontPage.c | 8 IntelFrameworkModulePkg/Universal/BdsDxe/MemoryTest.c | 6 +++--- 2 files changed, 7 insertions

[edk2] [PATCH v2 2/8] IntelFrameworkModulePkg GenericBdsLib: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan Reviewed-by: Jaben Carsey --- IntelFrameworkModulePkg/Library/GenericBdsLib/BdsMisc.c| 14 +- .../Library/GenericBdsLib/Performance.c| 10 +- 2 files

[edk2] [PATCH v2 0/8] IntelFrameworkModulePkg: Use safe string functions

2015-06-25 Thread Hao Wu
/BootOption.c Hao Wu (8): IntelFrameworkModulePkg IsaFloppyDxe: Use safe string functions IntelFrameworkModulePkg GenericBdsLib: Use safe string functions IntelFrameworkModulePkg PeiDxeDebugLib: Use safe string functions IntelFrameworkModulePkg BdsDxe: Use safe string functions

[edk2] [PATCH v2 8/8] IntelFrameworkModulePkg UpdateDriverDxe: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan Reviewed-by: Jaben Carsey --- .../Universal/FirmwareVolume/UpdateDriverDxe/ParseUpdateProfile.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a

[edk2] [PATCH v2 5/8] IntelFrameworkModulePkg BootMaint: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan Reviewed-by: Jaben Carsey --- .../Universal/BdsDxe/BootMaint/BootOption.c| 31 +++--- .../Universal/BdsDxe/BootMaint/FormGuid.h | 16 +++ .../Universal

[edk2] [PATCH v2 1/8] IntelFrameworkModulePkg IsaFloppyDxe: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan Reviewed-by: Jaben Carsey --- IntelFrameworkModulePkg/Bus/Isa/IsaFloppyDxe/ComponentName.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/IntelFrameworkModulePkg/Bus/Isa

[edk2] [PATCH v2 6/8] IntelFrameworkModulePkg BootMngr: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan Reviewed-by: Jaben Carsey --- .../Universal/BdsDxe/BootMngr/BootManager.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a

[edk2] [PATCH v2 3/8] IntelFrameworkModulePkg PeiDxeDebugLib: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan Reviewed-by: Jaben Carsey --- .../Library/PeiDxeDebugLibReportStatusCode/DebugLib.c| 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a

[edk2] [PATCH v2 7/8] IntelFrameworkModulePkg DeviceMngr: Use safe string functions

2015-06-25 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan Reviewed-by: Jaben Carsey --- .../Universal/BdsDxe/DeviceMngr/DeviceManager.c| 25 +++--- 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a

[edk2] [PATCH 4/8] IntelFrameworkModulePkg BdsDxe: Use safe string functions

2015-06-22 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Universal/BdsDxe/FrontPage.c | 8 IntelFrameworkModulePkg/Universal/BdsDxe/MemoryTest.c | 16 +--- 2 files changed, 17 insertions(+), 7

[edk2] [PATCH 0/8] IntelFrameworkModulePkg: Use safe string functions

2015-06-22 Thread Hao Wu
Replace unsafe string functions: (Ascii)StrCpy (Ascii)StrnCpy (Ascii)StrCat (Ascii)StrnCat with new added safe string functions: (Ascii)StrCpyS (Ascii)StrnCpyS (Ascii)StrCatS (Ascii)StrnCatS in IntelFrameworkModulePkg. Hao Wu (8): IntelFrameworkModulePkg IsaFloppyDxe: Use safe string functions

[edk2] [PATCH 5/8] IntelFrameworkModulePkg BootMaint: Use safe string functions

2015-06-22 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan --- .../Universal/BdsDxe/BootMaint/BootOption.c| 60 +- .../Universal/BdsDxe/BootMaint/FormGuid.h | 16 -- .../Universal/BdsDxe/BootMaint/UpdatePage.c

[edk2] [PATCH 2/8] IntelFrameworkModulePkg GenericBdsLib: Use safe string functions

2015-06-22 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Library/GenericBdsLib/BdsMisc.c | 6 +++--- IntelFrameworkModulePkg/Library/GenericBdsLib/Performance.c | 10 +- 2 files changed, 8 insertions(+), 8

[edk2] [PATCH 8/8] IntelFrameworkModulePkg UpdateDriverDxe: Use safe string functions

2015-06-22 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan --- .../Universal/FirmwareVolume/UpdateDriverDxe/ParseUpdateProfile.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/IntelFrameworkModulePkg/Universal/FirmwareVolume

[edk2] [PATCH 1/8] IntelFrameworkModulePkg IsaFloppyDxe: Use safe string functions

2015-06-22 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan --- IntelFrameworkModulePkg/Bus/Isa/IsaFloppyDxe/ComponentName.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/IntelFrameworkModulePkg/Bus/Isa/IsaFloppyDxe

[edk2] [PATCH 3/8] IntelFrameworkModulePkg PeiDxeDebugLib: Use safe string functions

2015-06-22 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan --- .../Library/PeiDxeDebugLibReportStatusCode/DebugLib.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/IntelFrameworkModulePkg/Library

[edk2] [PATCH 7/8] IntelFrameworkModulePkg DeviceMngr: Use safe string functions

2015-06-22 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan --- .../Universal/BdsDxe/DeviceMngr/DeviceManager.c| 69 ++ 1 file changed, 57 insertions(+), 12 deletions(-) diff --git a/IntelFrameworkModulePkg/Universal/BdsDxe

[edk2] [PATCH 6/8] IntelFrameworkModulePkg BootMngr: Use safe string functions

2015-06-22 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan --- .../Universal/BdsDxe/BootMngr/BootManager.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/IntelFrameworkModulePkg/Universal/BdsDxe/BootMngr

[edk2] [PATCH] IntelFspPkg BaseFspDebugLibSerialPort: Use safe string functions

2015-06-18 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jiewen Yao --- IntelFspPkg/Library/BaseFspDebugLibSerialPort/DebugLib.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/IntelFspPkg/Library/BaseFspDebugLibSerialPort/DebugLib.c

[edk2] [PATCH 1/2] MdePkg UefiDevicePathLib: Use safe string functions

2015-06-18 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Library/UefiDevicePathLib/DevicePathFromText.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/MdePkg/Library/UefiDevicePathLib

[edk2] [PATCH 0/2] MdePkg: Use safe string functions

2015-06-18 Thread Hao Wu
Replace unsafe string functions Str*() with safe ones Str*S(). Hao Wu (2): MdePkg UefiDevicePathLib: Use safe string functions MdePkg UefiLib: Use safe string functions MdePkg/Library/UefiDevicePathLib/DevicePathFromText.c | 18 ++ MdePkg/Library/UefiLib/Console.c

[edk2] [PATCH 2/2] MdePkg UefiLib: Use safe string functions

2015-06-18 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Library/UefiLib/Console.c | 6 +++--- MdePkg/Library/UefiLib/UefiLibPrint.c | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/MdePkg/Library/UefiLib

[edk2] [PATCH] IntelFrameworkPkg FrameworkUefiLib: Use safe string functions

2015-06-18 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLibPrint.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLibPrint.c

[edk2] [PATCH] MdePkg: Add EFI REST Protocol definitions

2015-06-07 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Include/Protocol/Rest.h | 94 ++ MdePkg/MdePkg.dec | 3 ++ 2 files changed, 97 insertions(+) create mode 100644 MdePkg/Include

[edk2] [PATCH 0/2] Add header files for EAP Protocols

2015-05-24 Thread Hao Wu
Add definitions for EFI EAP Management2 Protocol and EFI EAP Configuration Protocol. Hao Wu (2): MdePkg: Add EFI EAP Management2 Protocol definitions MdePkg: Add EFI EAP Configuration Protocol definitions MdePkg/Include/Protocol/Eap.h | 8 +- MdePkg/Include/Protocol

[edk2] [PATCH 2/2] MdePkg: Add EFI EAP Configuration Protocol definitions

2015-05-24 Thread Hao Wu
Add UEFI2.5 EFI EAP Configuration Protocol definitions. MdePkg/Include/Protocol/Eap.h is also modified for backward compatibility. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Include/Protocol/Eap.h | 8

[edk2] [PATCH 1/2] MdePkg: Add EFI EAP Management2 Protocol definitions

2015-05-24 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Include/Protocol/EapManagement2.h | 87 MdePkg/MdePkg.dec| 3 ++ 2 files changed, 90 insertions(+) create mode 100644

[edk2] [PATCH v2] MdePkg: Resolve type mismatch in node/text conversion for Wifi device

2015-05-20 Thread Hao Wu
Function DevPathFromTextWiFi() in use UINT8* and CHAR8* interchangeably, which breaks the build on ARM. Pointer type conversion is added to resolved this problem. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Ard Biesheuvel --- MdePkg/Library

[edk2] [PATCH] MdePkg: Resolve type mismatch in node/text conversion for Wifi device

2015-05-20 Thread Hao Wu
issue. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- MdePkg/Include/Protocol/DevicePath.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdePkg/Include/Protocol/DevicePath.h b/MdePkg/Include/Protocol/DevicePath.h index 7b9a4e5..121b952

[edk2] [PATCH 0/2] Header files and device support for Wi-Fi protocol

2015-05-14 Thread Hao Wu
UEFI 2.5 introduces EFI wireless MAC connection protocol. The header file will be added into MdePkg. Wi-Fi device path definition and its node/text conversion are added as well. Hao Wu (2): MdePkg: Add EFI Wireless MAC Connection Protocol definitions MdePkg: Add WiFi device path definition

[edk2] [PATCH 2/2] MdePkg: Add WiFi device path definition and its node/text conversion

2015-05-14 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Ye Ting --- MdePkg/Include/Protocol/DevicePath.h | 14 ++ .../Library/UefiDevicePathLib/DevicePathFromText.c | 31 ++ .../Library/UefiDevicePathLib

[edk2] [PATCH 1/2] MdePkg: Add EFI Wireless MAC Connection Protocol definitions

2015-05-14 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Ye Ting --- MdePkg/Include/Protocol/WiFi.h | 1144 MdePkg/MdePkg.dec |3 + 2 files changed, 1147 insertions(+) create mode 100644 MdePkg/Include

[edk2] [PATCH] MdePkg: Fix PI Multi-processor PPI header file include issue

2015-05-13 Thread Hao Wu
rocessor service now, function prototype EFI_AP_PROCEDURE should be moved to 'Pi/PiMultiPhase.h'. The move is a compatible change and will not affect other codes. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Jeff Fan Reviewed-by: Liming Gao --

[edk2] [PATCH] MdePkg: Modify specification number encoding

2015-05-04 Thread Hao Wu
Change the PEI, DXE, and SMM service table revisions to 1.4. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Include/Pi/PiDxeCis.h | 4 ++-- MdePkg/Include/Pi/PiPeiCis.h | 4 ++-- MdePkg/Include/Pi/PiSmmCis.h | 8 3

[edk2] [PATCH] MdeModulePkg: Remove GetBootMode() at FirmwarePerformancePeiEntryPoint

2015-05-04 Thread Hao Wu
Boot mode maybe changed at any point during the PEI phase, therefore, boot mode should be checked later rather at entry point. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- .../FirmwarePerformancePei.c | 78

[edk2] [PATCH] MdePkg: EFI_FIRMWARE_MANAGEMENT_PROTOCOL.CheckImage() Description Update

2015-05-04 Thread Hao Wu
Update description comments for IMAGE_UPDATABLE_VALID. Add new definition IMAGE_UPDATABLE_VALID_WITH_VENDOR_CODE to FMP CheckImage. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Include/Protocol/FirmwareManagement.h | 10

[edk2] [PATCH] MdePkg: Update the UEFI version to reflect new revision

2015-05-04 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Include/Uefi/UefiSpec.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/MdePkg/Include/Uefi/UefiSpec.h b/MdePkg/Include/Uefi/UefiSpec.h index 64105c2..b862d1b

[edk2] [PATCH 0/2] MdePkg: Add ESRT Interface Definitions

2015-04-26 Thread Hao Wu
Add EFI System Resource Table (ESRT) interface (API only). Also, trailing whitespaces are removed for modified files. Hao Wu (2): MdePkg: Add ESRT Interface Definitions MdePkg: Trim trailing whitespaces for ESRT related files MdePkg/Include/Guid/FmpCapsule.h | 33 ++-- MdePkg

[edk2] [PATCH 2/2] MdePkg: Trim trailing whitespaces for ESRT related files

2015-04-26 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Chao Zhang --- MdePkg/Include/Guid/FmpCapsule.h | 14 +- MdePkg/Include/Protocol/FirmwareManagement.h | 214 +-- 2 files changed, 114

[edk2] [PATCH 1/2] MdePkg: Add ESRT Interface Definitions

2015-04-26 Thread Hao Wu
Add EFI System Resource Table (ESRT) interface (API only). Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao Reviewed-by: Chao Zhang --- MdePkg/Include/Guid/FmpCapsule.h | 19 ++-- MdePkg/Include/Guid/SystemResourceTable.h

[edk2] [PATCH] MdePkg: Add the header file for UEFI PI Multi-processor PPI.

2015-04-26 Thread Hao Wu
The MP Services PPI is installed by some platform or chipset-specific PEIM that abstracts handling multiprocessor support. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Include/Ppi/MpServices.h | 283

[edk2] [PATCH] MdePkg: Add header files to support graphics in PEI.

2015-04-26 Thread Hao Wu
Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Liming Gao --- MdePkg/Include/Guid/GraphicsInfoHob.h | 36 +++ MdePkg/Include/Ppi/Graphics.h | 85 +++ MdePkg/MdePkg.dec | 13 ++ 3 files changed, 134

[edk2] [PATCH] IntelFrameworkModulePkg: Modify IsaSerialDxe to follow driver rules

2015-04-24 Thread Hao Wu
: Hao Wu --- .../Bus/Isa/IsaSerialDxe/ComponentName.c | 64 +- 1 file changed, 37 insertions(+), 27 deletions(-) diff --git a/IntelFrameworkModulePkg/Bus/Isa/IsaSerialDxe/ComponentName.c b/IntelFrameworkModulePkg/Bus/Isa/IsaSerialDxe/ComponentName.c index 4b17c54

[edk2] [PATCH 1/2] MdePkg: Add a macro which converts an ASCII sting to UNICODE format

2015-03-06 Thread Hao Wu
This macro is used as a general solution to remove duplicated hardcode strings when both the ASCII and UNICODE format of a string is needed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- MdePkg/Include/Base.h | 16 +++- 1 file changed, 15

[edk2] [PATCH 2/2] IntelFrameworkModulePkg: Removed a duplicated hardcode string

2015-03-06 Thread Hao Wu
emove the duplicated hardcode string. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- IntelFrameworkModulePkg/Bus/Isa/IsaSerialDxe/ComponentName.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/IntelFrameworkModulePkg/Bus/Isa/IsaSerialDx

[edk2] [PATCH 0/2] MdePkg, IntelFrameworkModulePkg: Removed duplicated hardcode string

2015-03-06 Thread Hao Wu
Macros are added which return the UNICODE format of a given ASCII string. Therefore, duplicated hardcode string when both the ASCII and UNICODE format of a string constant can be removed generally. Hao Wu (2): MdePkg: Add a macro which converts an ASCII sting to UNICODE format

[edk2] [PATCH 1/2] MdePkg: Add a macro which converts an ASCII sting to UNICODE format

2015-03-05 Thread Hao Wu
This macro is used as a general solution to remove duplicated hardcode strings when both the ASCII and UNICODE format of a string is needed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- MdePkg/Include/Base.h | 16 +++- 1 file changed, 15

[edk2] [PATCH 2/2] IntelFrameworkModulePkg: Removed a duplicated hardcode string

2015-03-05 Thread Hao Wu
emove the duplicated hardcode string. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu --- IntelFrameworkModulePkg/Bus/Isa/IsaSerialDxe/ComponentName.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/IntelFrameworkModulePkg/Bus/Isa/IsaSerialDx

[edk2] [PATCH 0/2] MdePkg, IntelFrameworkModulePkg: Removed duplicated hardcode string

2015-03-05 Thread Hao Wu
Macros are added which return the UNICODE format of a given ASCII string. Therefore, duplicated hardcode string when both the ASCII and UNICODE format of a string constant can be removed generally. Hao Wu (2): MdePkg: Add a macro which converts an ASCII sting to UNICODE format