related fields in BmmOldFakeNVData,
or it will result in incorrect comparison in BootMaintRouteConfig function,
then resulting in incorrect UI behaviors.
Cc: Laszlo Ersek
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
sult in incorrect UI behaviors. When the BootOptionOrder has not been
saved, we should use the BootOptionOrder in current BmmFakeNvData.
Cc: Laszlo Ersek
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Library/BootMai
the logic here.
Cc: Laszlo Ersek
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Library/BootMaintenanceManagerUiLib/UpdatePage.c | 12
1 file changed, 12 deletions(-)
diff --git a/MdeModuleP
anoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/BdsDxe/BootMaint/UpdatePage.c| 22 +++---
1 file changed, 19 insertions(+), 3 deletions(-)
diff --git a/IntelFrameworkModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c
b/IntelFrameworkModule
: Dandan Bi
---
ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.dsc
index 3bd0cbd..bfa4dbe 100644
--- a/ArmVirtPkg/ArmVirtQemuKernel.dsc
+++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc
@@ -1,9
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Library/BootMaintenanceManagerUiLib/Variable.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MdeModulePkg/Library/BootMaintenanceManagerUiLib/Variable.c
b
: Laszlo Ersek
Cc: Ard Biesheuvel
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
ArmVirtPkg/ArmVirtQemu.dsc | 5 -
ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 -
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/ArmVirtPkg
Because SecureBootConfigDxe use FileExplorerLib now, but
FileExplorerLib is not in the dsc file of the package
that use SecureBootConfigDxe. Now add it to pass build.
Cc: Jordan Justen
Cc: Laszlo Ersek
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan
x27;t need previous patch.
- ArmVirtPkg/ArmVirtQemuKernel.dsc has been added, so add FileExplorerLib.inf
to it.
Cc: Laszlo Ersek
Cc: Ard Biesheuvel
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Dandan Bi (5):
ArmVirPkg: Add FileExplorerLib.i
Because SecureBootConfigDxe use FileExplorerLib now, but
FileExplorerLib is not in the dsc file of the package
that use SecureBootConfigDxe. Now add it to pass build.
Cc: David Wei
Cc: Tim He
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Because SecureBootConfigDxe use FileExplorerLib now, but
FileExplorerLib is not in the dsc file of the package
that use SecureBootConfigDxe. Now add it to pass build.
Cc: Chao Zhang
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric
Agreement 1.0
Signed-off-by: Dandan Bi
Acked-by: Laszlo Ersek
Reviewed-by: Eric Dong
Reviewed-by: Chao Zhang
---
.../SecureBootConfigDxe/SecureBootConfig.vfr | 109 +-
.../SecureBootConfigDxe/SecureBootConfigDxe.inf|3 +-
.../SecureBootConfigFileExplorer.c | 1127
when free up resource allocated for a MenuEntry, there exists
the case that the DevicePath and DisplayString are NULL,
so before calling FreePool function, need to check.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Cc: Qiu Shumin
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Library/FileExplorerLib/FileExplorer.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MdeModulePkg/Library/FileExplorerLib/FileExplorer.c
b/MdeModulePkg
Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Library/FileExplorerLib/FormGuid.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/MdeModulePkg/Library/FileExplorerLib/FormGuid.h
b/MdeModulePkg/Library/FileExplorerLib/FormGuid.h
index 011ae22..d0a43a9
The function AllocateCopyPool may return NULL, so need to do check
after calling it. This patch is to enhance the related logic.
Cc: Chao Zhang
Cc: Qiu Shumin
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../SecureBootConfigDxe
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../SecureBootConfigFileExplorer.c | 30 +++---
1 file changed, 21 insertions(+), 9 deletions(-)
diff --git
a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe
previous version:
- Add a Pcd to control this feature, default is enabled.
- Export the date after ReadyToBoot.
Cc: Liming Gao
Cc: Eric Dong
Cc: Brian J. Johnson
Cc: Andrew Fish
Cc: El-Haj-Mahmoud, Samer
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/MdeModulePkg.dec | 8 ++
MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 144 -
.../Universal/HiiDatabaseDxe/HiiDatabase.h | 30 +
.../Universal
Remove the empty line to conform with current coding style.
Cc: Qiu Shumin
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/MdeModulePkg
Since PcdHiiOsRuntimeSupport has added in the MdeModulePkg.dec file,
now add the usage information in the uni file.
Cc: Qiu Shumin
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/MdeModulePkg.uni | 8
1 file changed, 8
Remove the ASSERT code that may be triggered in LegacyBootMaintUiLib.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../LegacyBootMaintUiLib/LegacyBootMaintUi.c | 30 ++
1 file changed, 25 insertions
Remove the ASSERT in UI code that may be triggered,
and clean up the useless code.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Application/UiApp/FrontPage.c | 7 ++-
.../BootMaintenanceManagerUiLib
In DriverSampleInit, has installed the configAccess protocol to
the DriverHandle[1], but don't uninstall it in DriverSampleUnload.
Now uninstall the configAccess protocol.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dand
The type casting is not necessary and now remove it.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootMaintenance.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions
handlers: CreateBootOptionFromFile, CreateDriverOptionFromFile,
BootFromFile. Now specifies the calling convention for those functions.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Library/BootMaintenanceManagerUiLib
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c
b
under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/HiiDatabaseDxe/ConfigRouting.c | 517 +
1 file changed, 517 insertions(+)
diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c
b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRout
.
Cc: Liming Gao
Cc: Eric Dong
Cc: Laszlo Ersek
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/HiiDatabaseDxe/ConfigRouting.c | 534 +
1 file changed, 534 insertions(+)
diff --git a/MdeModulePkg/Universal/HiiDataba
SetupBrowser to fix the issue described above.
Patch 2 is to add a sample case for orderedlist to get default value
through Callback() function.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Dandan Bi (2):
MdeModulePkg
Add the sample case for orderedlist to get standard
default value from Callback function.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/DriverSampleDxe/DriverSample.c | 53
1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
index b357e29..c36588e 100644
--- a
: Dandan Bi
Dandan Bi (2):
MdeModulePkg/SetupBrowserDxe: Get default from callback for
orderedList
MdeModulePkg/DriverSampleDxe: Add a sample case
.../Universal/DriverSampleDxe/DriverSample.c | 53 +-
.../Universal/DriverSampleDxe/NVDataStruc.h| 1
1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
index b357e29..c36588e 100644
--- a
Add the sample case for orderedlist to get standard
default value from Callback function.
Notes:
V2:
- Refine the sample case.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/DriverSampleDxe/DriverSample.c
ction FindSameBlockElement ().
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/HiiDatabaseDxe/ConfigRouting.c | 541 +
1 file changed, 541 insertions(+)
diff --git a/MdeModulePkg/Universal/HiiDataba
odes.
Cc: Eric Dong
Cc: Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/Confi
: Dandan Bi
Dandan Bi (2):
MdeModulePkg/BootMaintUiLib: Update menus when open BMM form
IntelFrameworkModulePkg/LegacyUi: Get legacy options when open legacy
form
.../LegacyBootMaintUiLib/LegacyBootMaintUi.c | 42 ++
.../BootMaintenanceManagerUiLib
option) when opening the BMM form. When opening BMM form, the
LeagcyBootMaintUiLib must have been initialized.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
.../BootMaintenanceManagerUiLib
legacy forminstead in its the constructor function. Because
when opening the legacy form, the LegacyBootManagerLib must have been
initialized.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
Patch 1 is to enhance the error handling codes in function
HiiGetFormSetFromHiiHandle
Patch 2 is enhance the logic in the caller functions of
HiiGetFormSetFromHiiHandle.
Cc: Eric Dong
Cc: Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Dandan Bi (2
Cc: Eric Dong
Cc: Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Library/UefiHiiLib/HiiLib.c | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/MdeModulePkg/Library/UefiHiiLib/HiiLib.c
b/MdeModulePkg
Pointer "Buffer" is returned by function HiiGetFormSetFromHiiHandle,
when the function HiiGetFormSetFromHiiHandle return success, the
"Buffer" can not be NULL, so we add the ASSET codes.
Cc: Eric Dong
Cc: Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0
Sign
w the
reason of failure.
Cc: Liming Gao
Cc: Eric Dong
Cc: Chao Zhang
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/MdeModu
Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
.../BootMaintenanceManagerUiLib/BootMaintenance.c | 5 ++-
.../BootMaintenanceManagerUiLib/BootOption.c | 42 --
.../BootMaintenanceManagerUiLib/UpdatePage.c | 14
3 files changed, 35 insertions
option) when opening the BMM form. We think when opening BMM form,
the LeagcyBootMaintUiLib must have been initialized.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
.../BootMaintenanceManagerUiLib
orm is open in the callback function.
-Patch 2:compare the question id when the form is open in the callback function.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Dandan Bi (2):
MdeModulePkg/BootMaintUiLib: Update menus when o
forminstead in its the constructor function. Because
when opening the legacy form, the LegacyBootManagerLib must have been
initialized.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
ted-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdePkg/Include/Protocol/HiiImageEx.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MdePkg/Include/Protocol/HiiImageEx.h
b/MdePkg/Include/Protocol/HiiImageEx.h
index 9905ecd..b92fc3a 100644
--- a/Mde
patch 1 is to fix a bug in current code.
patch 2 is to add two compile option for VfrCompile.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Dandan Bi (2):
BaseTool/VfrCompile: Add missing question opcode
BaseTools/VfrCompile
The function CheckQuestionOpCode is to check whether the opcode
is question opcode, but it misses two question opcodes:
'EFI_IFR_REF_OP' and 'EFI_IFR_RESET_BUTTON'. Now add them.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Sign
default.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
BaseTools/Source/C/VfrCompile/VfrCompiler.cpp | 56 +-
BaseTools/Source/C/VfrCompile/VfrCompiler.h | 3 +-
BaseTools/Source/C/VfrCompile
m the existing
defaults, and share its value to other default id which has no
default value.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Dandan Bi (2):
MdeModulePkg/HiiDB: Share default if some default value are not spec
share its value to other default id which has no
default value.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
.../Universal/HiiDatabaseDxe/ConfigRouting.c | 115 ++---
.../Univ
share its value to other default id which has no
default value.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 66 +-
MdeModu
, and share its value to other default id which has no
default value.
Notes:
v1->V2:
Generate DefaultStoreList as ascending order directly instead
of using function GetDefaultIdArray().
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: D
, and share its value to other default id which has no
default value.
Notes:
v1->V2:
Clarify that when the type of DefaultValueData is
DefaultValueFromOtherDefault,it can be overrode by itself.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: D
f
using function GetDefaultIdArray().
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Dandan Bi (2):
MdeModulePkg/HiiDB: Share default if some default value are not
specified
MdeModulePkg/Browser: Share default if some defau
When comparing the L"OFFSET=" with the ConfigResp string to find
all the "OFFSET=" in ConfigResp, should skip the character "&"
before "OFFSET=" in ConfigResp string.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
StorageWidth of the question.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 54 +-
1 file changed, 28 insertions(+), 26 deletions
"EFI_IFR_RESET_BUTTON_OP" is a statement, not a question,
so remove it from function CheckQuestionOpCode.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
BaseTools/Source/C/VfrCompile/VfrFormPkg.cpp | 1 -
1 file
StorageWidth of the question.
Notes:
V2:
- Add some comments to make the logic clear.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
Reviewed-by: Eric Dong
---
MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 61
checkbox conflicts with the new added policy,
so now we move this behavior to the end of getting default form other
default id.
Cc: Liming Gao
Cc: Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 7
ff-by: Dandan Bi
---
ShellPkg/Application/Shell/ShellProtocol.c | 4
1 file changed, 4 insertions(+)
diff --git a/ShellPkg/Application/Shell/ShellProtocol.c
b/ShellPkg/Application/Shell/ShellProtocol.c
index 0e5d954..6f29250 100644
--- a/ShellPkg/Application/Shell/ShellProtocol.c
+++ b/She
ution Agreement 1.0
Signed-off-by: Dandan Bi
---
ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c | 4
ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.uni | 3 ++-
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/ShellPkg/Library/UefiShel
tly.
Cc: Ruiyu Ni
Cc: Jaben Carsey
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
ShellPkg/Library/UefiShellDriver1CommandsLib/DrvCfg.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/ShellPkg/Library/UefiShellDriver1CommandsLib/DrvCfg.c
b/S
The first parameter of match2 opcode should be the pattern
and the second one should be the string.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/DriverSampleDxe/Vfr.vfr | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
When mismatch happens,there exists one case that exit current
form and display last form.Assert code don't cover this case.
Now add check to handle this situation.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/DisplayEngi
to ignore."
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/DisplayEngineDxe/FormDisplay.c| 3 ++-
.../Universal/DisplayEngineDxe/FormDisplayStr.uni | Bin 17032 -> 17048 bytes
2 files changed, 2 insertions(+), 1 deletion(-)
di
Enhance the setupbrowserdxe to output debug message when DisplayEngineDxe is
not installed and this will be easy for user to find the reason why can not
enter Setup page.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/SetupBrowserDxe
;ESC' to ignore."
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/DisplayEngineDxe/FormDisplay.c| 3 ++-
.../Universal/DisplayEngineDxe/FormDisplayStr.uni | Bin 17032 -> 17056 bytes
2 files changed, 2 insertions(+
the configuration string in SetupBrowserDxe to lowercase to fix
this issue.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c | 1 +
MdeModulePkg/Universal/SetupBrowserDxe/Setup.h| 13 +
2
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Application/UiApp/BootMaint/BootMaint.c| 1 -
.../Application/UiApp/DeviceMngr/DeviceManager.h | 4 +-
MdeModulePkg/Application/UiApp/FrontPage.c | 4 ++
MdeModulePkg/Application/UiApp
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c | 4 +---
.../Library/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf| 12 ++--
2 files changed, 7 insertions(+), 9 deletions(-)
diff
Once the question is refreshed,the processing should happen in the callback
function
of EFI_BROWSER_ACTION_RETRIEVE case,rather than EFI_BROWSER_ACTION_CHANGING in
DriverSample.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Universal/DriverSampleDxe
Refine the code in LegacyBootMaintUiLib to prevent the potential risk.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Library/LegacyBootMaintUiLib/LegacyBootMaintUi.c| 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git
Refine the code in UiApp to prevent the potential risk.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Application/UiApp/BootMaint/FileExplorer.c | 1 -
MdeModulePkg/Application/UiApp/BootMaint/Variable.c | 12 ++--
2 files changed
Replace the unsafe string functions with the safe one in UiApp.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Application/UiApp/BootMaint/BootOption.c | 30 ++
.../Application/UiApp/BootMaint/FormGuid.h | 2
TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Application/UiApp/BootMaint/Variable.c | 20
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/MdeModulePkg/Application/UiApp/BootMaint/Variable.c
b/MdeModulePkg/Application/UiApp/BootMaint/
In current UiApp/Boot Maintenance manager,some pages don't support F10,
they use Commit Changes and Exit menu to save changes.Now support F10
in these pages.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../Application/UiApp/BootMaint/Bmstrin
In current UiApp/Boot Maintenance manager,some pages don't support F10,
they use Commit Changes and Exit menu to save changes.Now support F10
in these pages.Compared with previous patch,mainly update the console page.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: D
Check the local variable 'NameSpace' before use it.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/Md
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
BaseTools/Source/C/VfrCompile/VfrFormPkg.h | 1 +
BaseTools/Source/C/VfrCompile/VfrSyntax.g | 6 --
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/BaseTools/Source/C/VfrCompile/VfrFormPkg.h
Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Universal/DriverSampleDxe/Vfr.vfr | 1 +
.../Universal/DriverSampleDxe/VfrStrings.uni | Bin 63554 -> 63834 bytes
.../Universal/HiiDatabaseDxe/ConfigRouting.c | 23 ++---
3 files changed,
Our tool can support OrderedList which has default value,but doesn't update the
source code in HiiDatabase when parse the ifr data.Now update the code and add
test case in DriverSample.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
MdeModulePkg/Univ
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi
---
.../RegularExpressionDxe/RegularExpressionDxe.c| 146 +
.../RegularExpressionDxe/RegularExpressionDxe.h| 108 +++
.../RegularExpressionDxe/RegularExpressionDxe.inf | 10
The pointer to the destination string changed,the max length also
changed.Previous change neglect this point.
And base on the code logic,we can use StrCatS to replace StrCpyS.Now this patch
is to fix this bug.
Signed-off-by: Dandan Bi
---
MdeModulePkg/Library/UefiHiiLib/HiiLib.c
1.1
Signed-off-by: Dandan Bi
---
BaseTools/Source/C/VfrCompile/VfrError.cpp | 2 +-
BaseTools/Source/C/VfrCompile/VfrError.h | 2 +-
BaseTools/Source/C/VfrCompile/VfrSyntax.g | 10 +-
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/BaseTools/Source/C/VfrCompile
Contribution Agreement 1.1
Signed-off-by: Dandan Bi
---
.../NonDiscoverablePciDeviceDxe/NonDiscoverablePciDeviceIo.c| 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/MdeModulePkg/Bus/Pci/NonDiscoverablePciDeviceDxe/NonDiscoverablePciDeviceIo.c
b/MdeModulePkg/Bus/Pci
e not 0. So we can remove the redundant
check "if (Attributes)".
Cc: Vladimir Olovyannikov
Cc: Jian J Wang
Cc: Hao Wu
Cc: Ard Biesheuvel
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi
Reviewed-by: Star Zeng
---
.../NonDiscoverablePciDeviceIo.c
g
function PeCoffLoaderImageAddress which will cover more
check and validation.
Cc: Michael D Kinney
Cc: Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi
---
MdePkg/Library/BasePeCoffLib/BasePeCoff.c | 29 +++
1 file changed, 24 insertions(+), 5
l consideration,
it's not safe enough.
So this patch is to replace the same code logic with function
PeCoffLoaderImageAddress which will cover more validation.
Cc: Michael D Kinney
Cc: Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi
---
MdePkg/L
>Size - 1
not
ImageBase + RelocDir->VirtualAddress + RelocDir->Size
This patch is to fix this issue when call PeCoffLoaderImageAddress
function to calculate the address of RelocBaseEnd.
Cc: Michael D Kinney
Cc: Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1
Signed
ted
...\Parsers\Pptt\PpttParser.c(193): warning C4244: '=':
conversion from 'UINT32' to 'UINT8', possible loss of data
Cc: Krzysztof Koch
Cc: Jaben Carsey
Cc: Ray Ni
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi
---
.../UefiShellAcpiViewCo
Cc: Ard Biesheuvel
Cc: Liming Gao
Cc: Jian J Wang
Cc: Hao Wu
Dandan Bi (2):
MdeModulePkg: Fix coding style issues
MdePkg: Fix coding style issues
.../FaultTolerantWriteSmmCommon.h| 4 ++--
.../FaultTolerantWriteStandaloneMm.c | 2
ned-off-by: Dandan Bi
---
.../FaultTolerantWriteDxe/FaultTolerantWriteSmmCommon.h | 4 ++--
.../FaultTolerantWriteDxe/FaultTolerantWriteStandaloneMm.c | 2 +-
.../FaultTolerantWriteDxe/FaultTolerantWriteTraditionalMm.c | 2 +-
.../Universal/Variable/RuntimeDxe/PrivilegePolymorphi
bution Agreement 1.1
Signed-off-by: Dandan Bi
---
.../Library/StandaloneMmDriverEntryPoint.h | 16
.../StandaloneMmDriverEntryPoint.c | 4 ++--
.../StandaloneMmServicesTableLib.c | 4 ++--
3 files changed, 12 insertions(+), 12 deletions(-)
diff -
avoid using AllocatePool.
Cc: Jian J Wang
Cc: Hao Wu
Cc: Michael Turner
Cc: Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi
---
.../Library/DxeReportStatusCodeLib/ReportStatusCodeLib.c | 9 +++--
.../RuntimeDxeReportStatusCodeLib
to avoid using AllocatePool.
Cc: Jian J Wang
Cc: Hao Wu
Cc: Michael Turner
Cc: Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi
---
.../Library/DxeReportStatusCodeLib/ReportStatusCodeLib.c | 9 +++--
.../RuntimeDxeReportStatusCodeLib/ReportStatus
: Jian J Wang
Cc: Hao Wu
Cc: Ruiyu Ni
Cc: Laszlo Ersek
Cc: Sean Brogan
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi
---
.../Library/UefiBootManagerLib/BmBoot.c | 22 ++-
1 file changed, 16 insertions(+), 6 deletions(-)
diff --git a
501 - 600 of 664 matches
Mail list logo