]> git.proxmox.com Git - mirror_edk2.git/commitdiff
MdeModulePkg/HiiDatabaseDxe: ASSERT StringPtr
authorShenglei Zhang <shenglei.zhang@intel.com>
Tue, 15 Oct 2019 02:44:14 +0000 (10:44 +0800)
committerLiming Gao <liming.gao@intel.com>
Wed, 6 Nov 2019 05:26:21 +0000 (13:26 +0800)
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>
MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c

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