OrgACRN
/
slimbootloader
mirror of https://github.com/slimbootloader/slimbootloader.git
1
0
Fork 0
Code Issues Releases Wiki Activity

[LiteVariable] Added Data NULL conditional check in InternalGetVariable() (#1699) 

Argument DataLen of function InternalGetVariable() inside Reclaim()
function is not initialized. This uninitialized value is assigned
to another variable and compared, resulting in EFI_BUFFER_TOO_SMALL
error when Data is NULL. Hence added Data NULL conditional check with
DataLen to overcome EFI_BUFFER_TOO_SMALL error when Data is NULL.

Signed-off-by: M Karuppasamy <karuppasamy.m@intel.com>
Signed-off-by: Sachin Kamat <sachin.kamat@intel.com>

Signed-off-by: M Karuppasamy <karuppasamy.m@intel.com>
Signed-off-by: Sachin Kamat <sachin.kamat@intel.com>
This commit is contained in:
Karuppa-samy 2022-09-30 18:36:47 +05:30 committed by GitHub
parent b44ef69ca4
commit 90406ffbac
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
BootloaderCommonPkg/Library/LiteVariableLib/LiteVariableLib.c
View File

@ -412,7 +412,7 @@ InternalGetVariable (
DataSizeIn = *DataSize;
VariableDataLen = FindVarHdrPtr->DataSize - VariableNameLen;
*DataSize = VariableDataLen;
if (DataSizeIn < VariableDataLen) {
if ((Data != NULL) && (DataSizeIn < VariableDataLen)) {
return EFI_BUFFER_TOO_SMALL;
}