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]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to