Fix regression issue by b41fac8e
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
.../Source/C/FMMT/FirmwareModuleManagement.c | 1 +
BaseTools/Source/C/FMMT/FmmtLib.c | 20 +++
2 files changed, 8 insertions(+), 13 deletions(-)
diff --git a/BaseTools/So
Some RAW ffs parse section for check UI section will encounter dead loop.
Actually when FFS is raw FFS, not include UI section.
Change-Id: I40e1cbb988bed30d8757008bad9074bc11eae135
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/FMMT/FirmwareModuleManagement.c | 8
Fmmt replace case like as below:
One Fv include one GUID section, and the GUID section include multiple sub
GUID section.
This patch will support the replace case.
Change-Id: I78f015e00f2b73e4e0e9ee36d03d24c697783a9c
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2624
When EFI file come from binary file, not generate .map file, so need ignore the
map file.
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/Python/GenFds/EfiSection.py | 2 +-
1 file changed, 1 insertion(+), 1
.
-Original Message-
From: Ni, Ray
Sent: Saturday, May 9, 2020 4:28 PM
To: Feng, YunhuaX ; devel@edk2.groups.io
Cc: Gao, Liming ; Feng, Bob C
Subject: RE: [edk2-staging][PATCH v2] BaseTools/Fmmt: Fix found the next FFS
issue
Yunhua,
I understand that if the ffs size 0 should be
if the ffs size is invalid, break the iteration and return NOT_FOUND.
Cc: Ray Ni< ray...@intel.com>
Cc: Bob Feng
Cc: Liming Gao
Change-Id: I9e320d6176af350ff208901209f3f6c89e4e1924
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/FMMT/FmmtLib.c | 7 +++
1 file changed, 7 insertions(+)
d
if the ffs size is invalid, break the iteration and return NOT_FOUND.
Cc: Ray Ni< ray...@intel.com>
Cc: Bob Feng
Cc: Liming Gao
Change-Id: I9e320d6176af350ff208901209f3f6c89e4e1924
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/FMMT/FmmtLib.c | 7 +++
1 file changed, 7 insertions(+)
d
Yes , you are right, the condition of break should think about Fv's erase
policy, I will update this patch.
Thank you
-Original Message-
From: devel@edk2.groups.io On Behalf Of Ni, Ray
Sent: Saturday, May 9, 2020 11:28 AM
To: Feng, YunhuaX ; devel@edk2.groups.io
Cc: Feng, Bob C
];
EFI_FFS_FILE_STATE State;
} EFI_FFS_FILE_HEADER
-Original Message-
From: Ni, Ray
Sent: Saturday, May 9, 2020 10:46 AM
To: devel@edk2.groups.io; Feng, YunhuaX
Cc: Feng, Bob C ; Gao, Liming
Subject: RE: [edk2-staging][PATCH] BaseTools/Fmmt: Fix found the next FFS issue
Yunhua,
Why the comparison
if found the next FFS size equal 0xFF, return NOT_FOUND
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/FMMT/FmmtLib.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/BaseTools/Source/C/FMMT/FmmtLib.c
b/BaseTools/Source/C/FMMT/FmmtLib.c
index cdbee3d6
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2686
The build tool will give an incorrect GUID value if the GUID includes character
' or " ASCII value.
This patch is going to fix this issue.
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/Python/Common/Misc.p
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2686
some GUID include character ' or " ascii value, transfer to string will
catch the wrong value.
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/Python/Common/Misc.py | 10 +-
1 file changed, 5 insertio
when Fv not include any FFS, will encounter not enough space to add issue.
Change-Id: Ie401f0edcf3f9b6bcae706a877b284e840f61fe9
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/FMMT/FirmwareModuleManagement.c | 12
1 file changed, 12 insertions(+)
dif
Correct the extract function help message
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/FMMT/FirmwareModuleManagement.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/BaseTools/Source/C/FMMT/FirmwareModuleManagement.c
b/BaseTools/Sour
Enhance for check input FD if empty file or not.
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/FMMT/FirmwareModuleManagement.c | 2 +-
BaseTools/Source/C/FMMT/FmmtLib.c | 5 +
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/Bas
Update the UTILITY_MINOR_VERSION from 59 to 60
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
Silicon/Intel/Tools/FitGen/FitGen.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Silicon/Intel/Tools/FitGen/FitGen.h
b/Silicon/Intel/Tools/FitGen/FitGen.h
index fdf
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2117
avoid path traversal attack check.
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
Silicon/Intel/Tools/FitGen/FitGen.c | 47 +
1 file changed, 47 insertions(+)
diff --git a/Silicon/Intel/T
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2527
One BIOS image may have more than FVs with SEC module.
FCE tool should find the latest FV with SEC module as BFV image.
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/BfmLib/BinFileManager.c | 11 +
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2521
1. If FFS file has no UI section, its FILE_GUID will be used as its name.
FMMT tool -d/-a/-r option can be updated to support such FFS file.
2. -v option will print FILE_GUID if no UI section
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yun
19 matches
Mail list logo
