Reviewed-by: Guo Dong <[email protected]>
-----Original Message----- From: Jiang, Guomin <[email protected]> Sent: Saturday, January 29, 2022 1:31 AM To: [email protected] Cc: Dong, Guo <[email protected]>; Ni, Ray <[email protected]>; Ma, Maurice <[email protected]>; You, Benjamin <[email protected]> Subject: [PATCH 1/1] UefiPayloadPkg/PayloadLoaderPeim: Replace Delta type INTN with UINTN REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3818 ProcessRelocation64 use INTN Delta. However it force it to UINTN when call it. It will have some potential issue when memory larger than 2G because the high memory address will be fill with 0xFFFFFFFF if use INTN. Cc: Guo Dong <[email protected]> Cc: Ray Ni <[email protected]> Cc: Maurice Ma <[email protected]> Cc: Benjamin You <[email protected]> Signed-off-by: Guomin Jiang <[email protected]> --- UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c index dc47a05c6e4a..ee530322d7ed 100644 --- a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c +++ b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c @@ -108,7 +108,7 @@ ProcessRelocation64 ( IN UINT64 RelaSize, IN UINT64 RelaEntrySize, IN UINT64 RelaType, - IN INTN Delta, + IN UINTN Delta, IN BOOLEAN DynamicLinking ) { -- 2.30.0.windows.2 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#86318): https://edk2.groups.io/g/devel/message/86318 Mute This Topic: https://groups.io/mt/88763343/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
