Reviewed-by: Bob Feng <bob.c.f...@intel.com> -----Original Message----- From: Konstantin Aladyshev <aladyshe...@gmail.com> Sent: Thursday, July 21, 2022 9:12 PM To: devel@edk2.groups.io Cc: Feng, Bob C <bob.c.f...@intel.com>; Gao, Liming <gaolim...@byosoft.com.cn>; Chen, Christine <yuwei.c...@intel.com>; Konstantin Aladyshev <aladyshe...@gmail.com> Subject: [PATCH] BaseTools/VolInfo: Increase GUID base name string
The current string lenght (=60) is not enough for cases where basename is a path to Build folder. Drop custom define and use MAX_LINE_LEN from the BaseTools codebase instead. Signed-off-by: Konstantin Aladyshev <aladyshe...@gmail.com> --- BaseTools/Source/C/VolInfo/VolInfo.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/BaseTools/Source/C/VolInfo/VolInfo.c b/BaseTools/Source/C/VolInfo/VolInfo.c index 4628e756d7..d9be3d8c18 100644 --- a/BaseTools/Source/C/VolInfo/VolInfo.c +++ b/BaseTools/Source/C/VolInfo/VolInfo.c @@ -51,15 +51,13 @@ EFI_GUID gEfiCrc32GuidedSectionExtractionProtocolGuid = EFI_CRC32_GUIDED_SECTIO #define EFI_SECTION_ERROR EFIERR (100) -#define MAX_BASENAME_LEN 60 // not good to hardcode, but let's be reasonable- // // Structure to keep a list of guid-to-basenames // typedef struct _GUID_TO_BASENAME { struct _GUID_TO_BASENAME *Next; INT8 Guid[PRINTED_GUID_BUFFER_SIZE];- INT8 BaseName[MAX_BASENAME_LEN];+ INT8 BaseName[MAX_LINE_LEN]; } GUID_TO_BASENAME; static GUID_TO_BASENAME *mGuidBaseNameList = NULL;-- 2.25.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94616): https://edk2.groups.io/g/devel/message/94616 Mute This Topic: https://groups.io/mt/92526015/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-