[edk2-devel] [PATCH] BaseTools/VolInfo: Increase GUID base name string

[Bob Feng]

Reviewed-by: Bob Feng <bob.c.feng at intel.com>

-----Original Message-----
From: Konstantin Aladyshev <aladyshev22 at gmail.com> 
Sent: Thursday, July 21, 2022 9:12 PM
To: devel at edk2.groups.io
Cc: Feng, Bob C <bob.c.feng at intel.com>; Gao, Liming <gaoliming at byosoft.com.cn>; Chen, Christine <yuwei.chen at intel.com>; Konstantin Aladyshev <aladyshev22 at 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 <aladyshev22 at 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/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-