diff options
| author | Dandan Bi <dandan.bi@intel.com> | 2016-06-24 13:19:14 +0800 |
|---|---|---|
| committer | Star Zeng <star.zeng@intel.com> | 2016-06-28 18:17:30 +0800 |
| commit | 763cfa739b1733715fe6700a29ec078c36453f5e (patch) | |
| tree | 78b4783a3140c34f9b0535347c247c91d69d94e5 /MdeModulePkg | |
| parent | 631c942726640615d53e4a358c078bb915e1bdd4 (diff) | |
| download | edk2-platforms-763cfa739b1733715fe6700a29ec078c36453f5e.tar.xz | |
MdeModulePkg/HiiDatabaseDxe: Add ASSERT before using the pointer 'String'
The 'Sting' is returned by the function GetUnicodeStringTextAndSize.
If it is NULL, function GetUnicodeStringTextAndSize will return
EFI_OUT_OF_RESOURCES, and error handling codes will cover it.
So the pointer 'Sting' can not be NULL when using it.
So we can add the ASSERT codes.
Cc: Eric Dong <eric.dong@intel.com>
Cc: Hao Wu <hao.a.wu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Hao Wu <hao.a.wu@intel.com>
Diffstat (limited to 'MdeModulePkg')
| -rw-r--r-- | MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c index 03f8141199..6682319711 100644 --- a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c +++ b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c @@ -875,7 +875,7 @@ GetStringIdFromString ( if (EFI_ERROR (Status)) {
goto Done;
}
-
+ ASSERT (String != NULL);
if (StrCmp(KeywordValue, String) == 0) {
*StringId = CurrentStringId;
goto Done;
@@ -895,7 +895,7 @@ GetStringIdFromString ( if (EFI_ERROR (Status)) {
goto Done;
}
-
+ ASSERT (String != NULL);
if (StrCmp(KeywordValue, String) == 0) {
*StringId = CurrentStringId;
goto Done;
@@ -914,7 +914,7 @@ GetStringIdFromString ( if (EFI_ERROR (Status)) {
goto Done;
}
-
+ ASSERT (String != NULL);
BlockSize += StringSize;
if (StrCmp(KeywordValue, String) == 0) {
*StringId = CurrentStringId;
@@ -939,7 +939,7 @@ GetStringIdFromString ( if (EFI_ERROR (Status)) {
goto Done;
}
-
+ ASSERT (String != NULL);
BlockSize += StringSize;
if (StrCmp(KeywordValue, String) == 0) {
*StringId = CurrentStringId;
|