sravan/system76-edk2
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

MdeModulePkg/HiiDatabaseDxe: ASSERT StringPtr

Browse Source 
The caller of CompareAndMergeDefaultString has checked that
AltCfgResp must contain AltConfigHdr. So we add ASSERT to assume
StringPtr is not NULL.

Cc: Dandan Bi <dandan.bi@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
Reviewed-by: Dandan Bi <dandan.bi@intel.com>
This commit is contained in:
Shenglei Zhang
2019-10-15 10:44:14 +08:00
committed by Liming Gao
parent dd6d07bda9
commit e34022dcbe
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c
View File

@ -909,6 +909,7 @@ CompareAndMergeDefaultString (
// To find the <AltResp> with AltConfigHdr in AltCfgResp, ignore other <AltResp> which follow it. // To find the <AltResp> with AltConfigHdr in AltCfgResp, ignore other <AltResp> which follow it.
// //
StringPtr = StrStr (*AltCfgResp, AltConfigHdr); StringPtr = StrStr (*AltCfgResp, AltConfigHdr);
ASSERT (StringPtr != NULL);
StringPtrNext = StrStr (StringPtr + 1, L"&GUID"); StringPtrNext = StrStr (StringPtr + 1, L"&GUID");
if (StringPtrNext != NULL) { if (StringPtrNext != NULL) {
TempCharA = *StringPtrNext; TempCharA = *StringPtrNext;