tyler.durden
/
audk
mirror of https://github.com/acidanthera/audk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

MdePkg/BasePeCoffLib: Deal with broken debug directories 

Older versions of GenFw put the wrong value in the debug directory size
field in the PE/COFF header: instead of putting the combined size of all
the entries, it puts the size of the only entry it creates, but adds the
size of the NB10 payload that the entry points to. This confuses the
loader now that we started using additional debug directory entries to
describe DLL characteristics.

GenFw was fixed in commit 60e85a39fe, but the binaries that were
generated with it still need to be supported.

So let's detect this condition, and check whether the size of the debug
directory is consistent with the NB10 payload: if we should expect
additional directory entries where we observe the NB10 payload, the size
field is clearly wrong, and we can break from the loop.

BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4425
Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
Tested-by: Liming Gao <gaoliming@byosoft.com.cn>
Acked-by: Michael Kubacki <michael.kubacki@microsoft.com>
This commit is contained in:
Ard Biesheuvel 2023-05-04 16:40:18 +02:00 committed by mergify[bot]
parent ff7cb2d7c9
commit 94c802e108
1 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
MdePkg/Library/BasePeCoffLib/BasePeCoff.c
View File

@ -585,6 +585,7 @@ PeCoffLoaderGetImageInfo (
UINTN Size;
UINTN ReadSize;
UINTN Index;
UINTN NextIndex;
UINTN DebugDirectoryEntryRva;
UINTN DebugDirectoryEntryFileOffset;
UINTN SectionHeaderOffset;
@ -755,6 +756,20 @@ PeCoffLoaderGetImageInfo (
ImageContext->ImageSize += DebugEntry.SizeOfData;
}
//
// Implementations of GenFw before commit 60e85a39fe49071 will
// concatenate the debug directory entry and the codeview entry,
// and erroneously put the combined size into the debug directory's
// size field. If this is the case, no other relevant directory
// entries can exist, and we can terminate here.
//
NextIndex = Index + sizeof (EFI_IMAGE_DEBUG_DIRECTORY_ENTRY);
if ((NextIndex < DebugDirectoryEntry->Size) &&
(DebugEntry.FileOffset == (DebugDirectoryEntryFileOffset + NextIndex)))
{
break;
}
continue;
}