Re: [edk2-devel] [PATCH v2 2/4] MdeModulePkg/HiiDatabaseDxe: Add check for StringPtr

2019-10-31 Thread Dandan Bi
Hi Shenglei, Please update the commit message and subject before commit, since we have added ASSERT code for this case that the StringPtr cannot be NULL instead of adding check. With the commit message updated, Reviewed-by: Dandan Bi . Thanks, Dandan > -Original Message- > From:

[edk2-devel] [PATCH v2 2/4] MdeModulePkg/HiiDatabaseDxe: Add check for StringPtr

2019-10-30 Thread Zhang, Shenglei
If the target string doesn't appear in the searched string, StringPtr will be NULL. So add a check for that. Cc: Dandan Bi Cc: Eric Dong Signed-off-by: Shenglei Zhang --- v2: Instead of returning a value, we add ASSERT to ensure StringPtr is not NULL.