On 9/27/19 9:46 AM, Liming Gao wrote:
> Signed-off-by: Liming Gao <liming....@intel.com>
> ---
>  BaseTools/Source/C/GenFw/GenFw.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/BaseTools/Source/C/GenFw/GenFw.c 
> b/BaseTools/Source/C/GenFw/GenFw.c
> index c99782b78e..d8d3360c24 100644
> --- a/BaseTools/Source/C/GenFw/GenFw.c
> +++ b/BaseTools/Source/C/GenFw/GenFw.c
> @@ -653,7 +653,11 @@ PeCoffConvertImageToXip (
>      //
>      // Make the size of raw data in section header alignment.
>      //
> -    SectionHeader->SizeOfRawData = (SectionHeader->Misc.VirtualSize + 
> PeHdr->Pe32.OptionalHeader.FileAlignment - 1) & 
> (~(PeHdr->Pe32.OptionalHeader.FileAlignment - 1));
> +    SectionSize = (SectionHeader->Misc.VirtualSize + 
> PeHdr->Pe32.OptionalHeader.FileAlignment - 1) & 
> (~(PeHdr->Pe32.OptionalHeader.FileAlignment - 1));
> +    if (SectionSize < SectionHeader->SizeOfRawData) {
> +      SectionHeader->SizeOfRawData = SectionSize;
> +    }
> +
>      SectionHeader->PointerToRawData = SectionHeader->VirtualAddress;
>    }
>
> @@ -999,7 +1003,7 @@ Returns:
>      CopyMem (
>        FileBuffer + SectionHeader->PointerToRawData,
>        (VOID*) (UINTN) (ImageContext.ImageAddress + 
> SectionHeader->VirtualAddress),
> -      SectionHeader->SizeOfRawData
> +      SectionHeader->SizeOfRawData < SectionHeader->Misc.VirtualSize ? 
> SectionHeader->SizeOfRawData : SectionHeader->Misc.VirtualSize
>        );
>    }
>
> 

Reviewed-by: Philippe Mathieu-Daude <phi...@redhat.com>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#48218): https://edk2.groups.io/g/devel/message/48218
Mute This Topic: https://groups.io/mt/34309056/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to