diff options
author | Andrew Fish <afish@apple.com> | 2014-08-21 09:23:58 +0000 |
---|---|---|
committer | oliviermartin <oliviermartin@6f19259b-4bc3-4df7-8a09-765794883524> | 2014-08-21 09:23:58 +0000 |
commit | 66c6d4d67ed870e9135c05a1f4deb03843c9faec (patch) | |
tree | 2db510ae63dbc6e2ccf162f566da41e8b9984be7 | |
parent | 0923aa1c3ad1a68f4781a98897889f3976d1b83d (diff) | |
download | edk2-66c6d4d67ed870e9135c05a1f4deb03843c9faec.zip edk2-66c6d4d67ed870e9135c05a1f4deb03843c9faec.tar.gz edk2-66c6d4d67ed870e9135c05a1f4deb03843c9faec.tar.bz2 |
MdePkg: Introduced BaseStackCheckLib (cont.)
Liming proposed different changes compare to the latest patch.
This commit ensures the changes goes in his direction.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Andrew Fish <afish@apple.com>
Signed-off-by: Olivier Martin <olivier.martin@arm.com
Reviewed-by: Gao, Liming <liming.gao@intel.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@15866 6f19259b-4bc3-4df7-8a09-765794883524
-rw-r--r-- | MdePkg/Library/BaseStackCheckLib/BaseStackCheckGcc.c | 2 | ||||
-rw-r--r-- | MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf | 1 | ||||
-rw-r--r-- | MdePkg/MdePkg.dec | 4 | ||||
-rw-r--r-- | MdePkg/MdePkg.dsc | 3 |
4 files changed, 4 insertions, 6 deletions
diff --git a/MdePkg/Library/BaseStackCheckLib/BaseStackCheckGcc.c b/MdePkg/Library/BaseStackCheckLib/BaseStackCheckGcc.c index ecf77f0..4cd0d4c 100644 --- a/MdePkg/Library/BaseStackCheckLib/BaseStackCheckGcc.c +++ b/MdePkg/Library/BaseStackCheckLib/BaseStackCheckGcc.c @@ -23,7 +23,7 @@ #include <Library/PcdLib.h>
/// "canary" value that is inserted by the compiler into the stack frame.
-VOID *__stack_chk_guard = (VOID*)FixedPcdGet64 (PcdBaseStackCanary);
+VOID *__stack_chk_guard = (VOID*)0x0AFF;
// If ASLR was enabled we could use
//void (*__stack_chk_guard)(void) = __stack_chk_fail;
diff --git a/MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf b/MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf index 3304284..1f33d5b 100644 --- a/MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf +++ b/MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf @@ -38,5 +38,4 @@ DebugLib
[FixedPcd]
- gEfiMdePkgTokenSpaceGuid.PcdBaseStackCanary
gEfiMdePkgTokenSpaceGuid.PcdDebugPropertyMask
diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec index 743e7bd..bd5ce07 100644 --- a/MdePkg/MdePkg.dec +++ b/MdePkg/MdePkg.dec @@ -1553,10 +1553,6 @@ # The required memory space is decided by the value of PcdMaximumGuidedExtractHandler.
gEfiMdePkgTokenSpaceGuid.PcdGuidedExtractHandlerTableAddress|0x1000000|UINT64|0x30001015
- ## Canary value for the stack overflow protection. This PCD can be used by a firmware vendor
- # or for debugging purposes to change the recommended value.
- gEfiMdePkgTokenSpaceGuid.PcdBaseStackCanary|0x0AFF|UINT64|0x0000002A
-
[PcdsFixedAtBuild.IPF]
## The base address of IO port space for IA64 arch
gEfiMdePkgTokenSpaceGuid.PcdIoBlockBaseAddressForIpf|0x0ffffc000000|UINT64|0x0000000f
diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc index 9532c1e..8971d76 100644 --- a/MdePkg/MdePkg.dsc +++ b/MdePkg/MdePkg.dsc @@ -169,3 +169,6 @@ [Components.EBC]
MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
+
+[Components.ARM, Components.AARCH64]
+ MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf
|