diff options
author | qwang12 <qwang12@6f19259b-4bc3-4df7-8a09-765794883524> | 2009-01-20 01:52:02 +0000 |
---|---|---|
committer | qwang12 <qwang12@6f19259b-4bc3-4df7-8a09-765794883524> | 2009-01-20 01:52:02 +0000 |
commit | 96ff65a1fb627823efaf4dc7cd01a424fa5297e8 (patch) | |
tree | 32254272c819993e9e12235ac421e06148cc7825 /MdeModulePkg/Universal/HiiDatabaseDxe/String.c | |
parent | 5a829f56880a176dc9a90e9d3653c961f9350aa0 (diff) | |
download | edk2-platforms-96ff65a1fb627823efaf4dc7cd01a424fa5297e8.tar.xz |
K8:
1) Fix the bug the assert conditions. || should be &&.
2) Add in code path and check-for-null-pointer to do error handling.
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@7308 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'MdeModulePkg/Universal/HiiDatabaseDxe/String.c')
-rw-r--r-- | MdeModulePkg/Universal/HiiDatabaseDxe/String.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/String.c b/MdeModulePkg/Universal/HiiDatabaseDxe/String.c index 8c207f9476..64148fe131 100644 --- a/MdeModulePkg/Universal/HiiDatabaseDxe/String.c +++ b/MdeModulePkg/Universal/HiiDatabaseDxe/String.c @@ -124,7 +124,7 @@ ConvertToUnicodeText ( ASSERT (StringSrc != NULL && BufferSize != NULL);
StringSize = AsciiStrSize (StringSrc) * 2;
- if (*BufferSize < StringSize) {
+ if (*BufferSize < StringSize || StringDest == NULL) {
*BufferSize = StringSize;
return EFI_BUFFER_TOO_SMALL;
}
|