Align the sections we're loading, and check for validity /after/ discarding.

Turns out a) the codegen on aarch64 generates code that has real
alignment needs, and b) if we check the length of discardable sections
before discarding them, we error for no reason.

So do the error checking in the right order, and always enforce some
alignment because we know we have to.

Signed-off-by: Peter Jones <pjones@redhat.com>

1 files changed, 1 insertions, 0 deletions

diff --git a/include/PeImage.h b/include/PeImage.h
index 133e11e6..05f32ea2 100644
--- a/include/PeImage.h
+++ b/include/PeImage.h
@@ -778,6 +778,7 @@ typedef struct {
 	UINTN SizeOfHeaders;

 	UINT16 ImageType;

 	UINT16 NumberOfSections;

+	UINT32 SectionAlignment;

 	EFI_IMAGE_SECTION_HEADER *FirstSection;

 	EFI_IMAGE_DATA_DIRECTORY *RelocDir;

 	EFI_IMAGE_DATA_DIRECTORY *SecDir;