Posting from the "wrong" email:
Apologies. As some of you might have noticed, I have had email issues recently,
which wasn't helped by them starting while I was away for 4 weeks.

This looks like a clear bugfix to me. I'm happy for it being included in the 
stable tag.

/
    Leif

> -----Original Message-----
> From: Michael D Kinney <[email protected]>
> Sent: 17 August 2022 18:43
> To: [email protected]
> Cc: Leif Lindholm <[email protected]>; Andrew Fish
> <[email protected]>; Konstantin Aladyshev <[email protected]>; Bob
> Feng <[email protected]>; Liming Gao <[email protected]>;
> Yuwei Chen <[email protected]>
> Subject: [Patch V2 edk2-stable202208 1/1] BaseTools/Source/C/GenSec: Fix
> EFI_SECTION_FREEFORM_SUBTYPE_GUID header
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4021
> 
> When the size of a EFI_SECTION_FREEFORM_SUBTYPE_GUID section
> required
> the use of EFI_FREEFORM_SUBTYPE_GUID_SECTION2 header, set the
> section
> type to EFI_SECTION_FREEFORM_SUBTYPE_GUID.
> 
> Cc: Leif Lindholm <[email protected]>
> Cc: Andrew Fish <[email protected]>
> Cc: Konstantin Aladyshev <[email protected]>
> Cc: Bob Feng <[email protected]>
> Cc: Liming Gao <[email protected]>
> Cc: Yuwei Chen <[email protected]>
> Signed-off-by: Michael D Kinney <[email protected]>
> Acked-by: Konstantin Aladyshev <[email protected]>
> Reviewed-by: Bob Feng <[email protected]>
> ---
>  BaseTools/Source/C/GenSec/GenSec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/BaseTools/Source/C/GenSec/GenSec.c
> b/BaseTools/Source/C/GenSec/GenSec.c
> index d86cc197cc26..a4c2d19aa6f4 100644
> --- a/BaseTools/Source/C/GenSec/GenSec.c
> +++ b/BaseTools/Source/C/GenSec/GenSec.c
> @@ -1112,7 +1112,7 @@ Routine Description:
>    //
>    if (TotalLength >= MAX_SECTION_SIZE) {
>      SubtypeGuidSect2 = (EFI_FREEFORM_SUBTYPE_GUID_SECTION2 *)
> FileBuffer;
> -    SubtypeGuidSect2->CommonHeader.Type     =
> EFI_SECTION_GUID_DEFINED;
> +    SubtypeGuidSect2->CommonHeader.Type     =
> EFI_SECTION_FREEFORM_SUBTYPE_GUID;
>      SubtypeGuidSect2->CommonHeader.Size[0]  = (UINT8) 0xff;
>      SubtypeGuidSect2->CommonHeader.Size[1]  = (UINT8) 0xff;
>      SubtypeGuidSect2->CommonHeader.Size[2]  = (UINT8) 0xff;
> --
> 2.37.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#92548): https://edk2.groups.io/g/devel/message/92548
Mute This Topic: https://groups.io/mt/93087383/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to