[edk2-devel] [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate in some case
Current implementation of the Elf loader reuses the same memory range if
the desired memory range is covered by [file base, file base + file size].
However, there is a potentil bug, for example:
If the first segment is located at 0x1000, and the desired address is
0x2000. While the second segment is located at 0x2000, and the desired
address is 0x2000. When we parse and try to load the elf file, current
implementation may load the first segment at 0x2000, and override second
segment.
Cc: Guo Dong
Cc: Ray Ni
Cc: Maurice Ma
Cc: Benjamin You
Cc: Sean Rhodes
Signed-off-by: Zhiguang Liu
---
UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c
b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c
index 2a6305c67b..c7dfae14af 100644
--- a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c
+++ b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c
@@ -312,6 +312,13 @@ ParseElfImage (
ElfCt->ReloadRequired = TRUE;
}
+if (SegInfo.MemAddr != ((UINTN)ElfCt->FileBase + SegInfo.Offset)) {
+ //
+ // Need to relocate if the desired address is not the current address
+ //
+ ElfCt->ReloadRequired = TRUE;
+}
+
if (Base > (SegInfo.MemAddr & ~(EFI_PAGE_SIZE - 1))) {
Base = SegInfo.MemAddr & ~(EFI_PAGE_SIZE - 1);
}
--
2.32.0.windows.2
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#88456): https://edk2.groups.io/g/devel/message/88456
Mute This Topic: https://groups.io/mt/90289885/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate in some case
> -Original Message- > From: Liu, Zhiguang > Sent: Wednesday, April 6, 2022 9:52 PM > To: devel@edk2.groups.io > Cc: Dong, Guo ; Ni, Ray ; Ma, Maurice > ; You, Benjamin > ; Rhodes, Sean > Subject: [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate in > some case > > Current implementation of the Elf loader reuses the same memory range if > the desired memory range is covered by [file base, file base + file size]. > However, there is a potentil bug, for example: > If the first segment is located at 0x1000, and the desired address is > 0x2000. While the second segment is located at 0x2000, and the desired > address is 0x2000. When we parse and try to load the elf file, current Why the desired address for both segments is 0x2000? -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#88484): https://edk2.groups.io/g/devel/message/88484 Mute This Topic: https://groups.io/mt/90289885/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate in some case
Sorry, it is a typo. the example should be The first segment is located at 0x1000, and the desired address is 0x2000. The second segment is located at 0x2000, and the desired address is 0x1000. The current ELF loader may copy first segment to 0x2000, which will erase the second segment. Thanks Zhiguang > -Original Message- > From: Ni, Ray > Sent: Thursday, April 7, 2022 8:06 AM > To: Liu, Zhiguang ; devel@edk2.groups.io > Cc: Dong, Guo ; Ma, Maurice ; > You, Benjamin ; Rhodes, Sean > > Subject: RE: [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate > in > some case > > > > > -Original Message- > > From: Liu, Zhiguang > > Sent: Wednesday, April 6, 2022 9:52 PM > > To: devel@edk2.groups.io > > Cc: Dong, Guo ; Ni, Ray ; Ma, > > Maurice ; You, Benjamin > > ; Rhodes, Sean > > Subject: [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't > > relocate in some case > > > > Current implementation of the Elf loader reuses the same memory range > > if the desired memory range is covered by [file base, file base + file > > size]. > > However, there is a potentil bug, for example: > > If the first segment is located at 0x1000, and the desired address is > > 0x2000. While the second segment is located at 0x2000, and the desired > > address is 0x2000. When we parse and try to load the elf file, current > > Why the desired address for both segments is 0x2000? -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#88490): https://edk2.groups.io/g/devel/message/88490 Mute This Topic: https://groups.io/mt/90289885/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate in some case
Reviewed-by: Guo Dong
-Original Message-
From: Liu, Zhiguang
Sent: Wednesday, April 6, 2022 6:52 AM
To: devel@edk2.groups.io
Cc: Dong, Guo ; Ni, Ray ; Ma, Maurice
; You, Benjamin ; Rhodes, Sean
Subject: [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate in
some case
Current implementation of the Elf loader reuses the same memory range if the
desired memory range is covered by [file base, file base + file size].
However, there is a potentil bug, for example:
If the first segment is located at 0x1000, and the desired address is 0x2000.
While the second segment is located at 0x2000, and the desired address is
0x2000. When we parse and try to load the elf file, current implementation may
load the first segment at 0x2000, and override second segment.
Cc: Guo Dong
Cc: Ray Ni
Cc: Maurice Ma
Cc: Benjamin You
Cc: Sean Rhodes
Signed-off-by: Zhiguang Liu
---
UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c
b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c
index 2a6305c67b..c7dfae14af 100644
--- a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c
+++ b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/ElfLib.c
@@ -312,6 +312,13 @@ ParseElfImage (
ElfCt->ReloadRequired = TRUE; } +if (SegInfo.MemAddr !=
((UINTN)ElfCt->FileBase + SegInfo.Offset)) {+ //+ // Need to relocate
if the desired address is not the current address+ //+
ElfCt->ReloadRequired = TRUE;+}+ if (Base > (SegInfo.MemAddr &
~(EFI_PAGE_SIZE - 1))) { Base = SegInfo.MemAddr & ~(EFI_PAGE_SIZE - 1);
}--
2.32.0.windows.2
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#88666): https://edk2.groups.io/g/devel/message/88666
Mute This Topic: https://groups.io/mt/90289885/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate in some case
With the updated commit message, Reviewed-by: Ray Ni > -Original Message- > From: Liu, Zhiguang > Sent: Thursday, April 7, 2022 8:57 AM > To: Ni, Ray ; devel@edk2.groups.io > Cc: Dong, Guo ; Ma, Maurice ; You, > Benjamin ; > Rhodes, Sean > Subject: RE: [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't relocate > in some case > > Sorry, it is a typo. the example should be > The first segment is located at 0x1000, and the desired address is 0x2000. > The second segment is located at 0x2000, and the desired address is 0x1000. > The current ELF loader may copy first segment to 0x2000, which will erase the > second segment. > > Thanks > Zhiguang > > > > -Original Message- > > From: Ni, Ray > > Sent: Thursday, April 7, 2022 8:06 AM > > To: Liu, Zhiguang ; devel@edk2.groups.io > > Cc: Dong, Guo ; Ma, Maurice ; > > You, Benjamin ; Rhodes, Sean > > > > Subject: RE: [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't > > relocate in > > some case > > > > > > > > > -Original Message- > > > From: Liu, Zhiguang > > > Sent: Wednesday, April 6, 2022 9:52 PM > > > To: devel@edk2.groups.io > > > Cc: Dong, Guo ; Ni, Ray ; Ma, > > > Maurice ; You, Benjamin > > > ; Rhodes, Sean > > > Subject: [PATCH] UefiPayloadPkg: Fix bug that Elf loader doesn't > > > relocate in some case > > > > > > Current implementation of the Elf loader reuses the same memory range > > > if the desired memory range is covered by [file base, file base + file > > > size]. > > > However, there is a potentil bug, for example: > > > If the first segment is located at 0x1000, and the desired address is > > > 0x2000. While the second segment is located at 0x2000, and the desired > > > address is 0x2000. When we parse and try to load the elf file, current > > > > Why the desired address for both segments is 0x2000? -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#88676): https://edk2.groups.io/g/devel/message/88676 Mute This Topic: https://groups.io/mt/90289885/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
