mirror of https://github.com/acidanthera/audk.git
Add checking for memory allocation in UI modules.
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@2219 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
parent
b12dbde369
commit
41a907e423
|
@ -94,11 +94,13 @@ Returns:
|
||||||
// Need some memory for OptionList. Allow for up to 8 options.
|
// Need some memory for OptionList. Allow for up to 8 options.
|
||||||
//
|
//
|
||||||
OptionList = AllocateZeroPool (sizeof (IFR_OPTION) * 8);
|
OptionList = AllocateZeroPool (sizeof (IFR_OPTION) * 8);
|
||||||
|
ASSERT (OptionList != NULL);
|
||||||
|
|
||||||
//
|
//
|
||||||
// Allocate space for creation of Buffer
|
// Allocate space for creation of Buffer
|
||||||
//
|
//
|
||||||
UpdateData = AllocateZeroPool (0x1000);
|
UpdateData = AllocateZeroPool (0x1000);
|
||||||
|
ASSERT (UpdateData != NULL);
|
||||||
|
|
||||||
//
|
//
|
||||||
// Remove all the op-codes starting with Label 0x2222 to next Label (second label is for convenience
|
// Remove all the op-codes starting with Label 0x2222 to next Label (second label is for convenience
|
||||||
|
@ -170,11 +172,13 @@ Returns:
|
||||||
// Need some memory for OptionList. Allow for up to 8 options.
|
// Need some memory for OptionList. Allow for up to 8 options.
|
||||||
//
|
//
|
||||||
OptionList = AllocateZeroPool (sizeof (IFR_OPTION) * 8);
|
OptionList = AllocateZeroPool (sizeof (IFR_OPTION) * 8);
|
||||||
|
ASSERT (OptionList != NULL);
|
||||||
|
|
||||||
//
|
//
|
||||||
// Allocate space for creation of Buffer
|
// Allocate space for creation of Buffer
|
||||||
//
|
//
|
||||||
UpdateData = AllocateZeroPool (0x1000);
|
UpdateData = AllocateZeroPool (0x1000);
|
||||||
|
ASSERT (UpdateData != NULL);
|
||||||
|
|
||||||
//
|
//
|
||||||
// Remove all the op-codes starting with Label 0x2222 to next Label (second label is for convenience
|
// Remove all the op-codes starting with Label 0x2222 to next Label (second label is for convenience
|
||||||
|
@ -244,6 +248,7 @@ Returns:
|
||||||
0x1000,
|
0x1000,
|
||||||
(VOID **) &UpdateData
|
(VOID **) &UpdateData
|
||||||
);
|
);
|
||||||
|
ASSERT_EFI_ERROR (Status);
|
||||||
|
|
||||||
ZeroMem (UpdateData, 0x1000);
|
ZeroMem (UpdateData, 0x1000);
|
||||||
|
|
||||||
|
@ -297,6 +302,7 @@ Returns:
|
||||||
0x1000,
|
0x1000,
|
||||||
(VOID **)&UpdateData
|
(VOID **)&UpdateData
|
||||||
);
|
);
|
||||||
|
ASSERT_EFI_ERROR (Status);
|
||||||
|
|
||||||
ZeroMem (UpdateData, 0x1000);
|
ZeroMem (UpdateData, 0x1000);
|
||||||
|
|
||||||
|
@ -366,6 +372,7 @@ Returns:
|
||||||
sizeof (EFI_HII_CALLBACK_PACKET) + sizeof (SAMPLE_STRING) + 2,
|
sizeof (EFI_HII_CALLBACK_PACKET) + sizeof (SAMPLE_STRING) + 2,
|
||||||
(VOID **) Packet
|
(VOID **) Packet
|
||||||
);
|
);
|
||||||
|
ASSERT_EFI_ERROR (Status);
|
||||||
|
|
||||||
ZeroMem (*Packet, sizeof (EFI_HII_CALLBACK_PACKET) + sizeof (SAMPLE_STRING) + 2);
|
ZeroMem (*Packet, sizeof (EFI_HII_CALLBACK_PACKET) + sizeof (SAMPLE_STRING) + 2);
|
||||||
|
|
||||||
|
@ -384,6 +391,7 @@ Returns:
|
||||||
sizeof (EFI_HII_CALLBACK_PACKET) + 2,
|
sizeof (EFI_HII_CALLBACK_PACKET) + 2,
|
||||||
(VOID **) Packet
|
(VOID **) Packet
|
||||||
);
|
);
|
||||||
|
ASSERT_EFI_ERROR (Status);
|
||||||
|
|
||||||
ZeroMem (*Packet, sizeof (EFI_HII_CALLBACK_PACKET) + 2);
|
ZeroMem (*Packet, sizeof (EFI_HII_CALLBACK_PACKET) + 2);
|
||||||
|
|
||||||
|
@ -585,6 +593,7 @@ DriverSampleInit (
|
||||||
0x1000,
|
0x1000,
|
||||||
(VOID **) &UpdateData
|
(VOID **) &UpdateData
|
||||||
);
|
);
|
||||||
|
ASSERT_EFI_ERROR (Status);
|
||||||
|
|
||||||
ZeroMem (UpdateData, 0x1000);
|
ZeroMem (UpdateData, 0x1000);
|
||||||
|
|
||||||
|
|
|
@ -1409,6 +1409,8 @@ Returns:
|
||||||
TempBufferSize += PackageInstance->IfrSize + PackageInstance->StringSize;
|
TempBufferSize += PackageInstance->IfrSize + PackageInstance->StringSize;
|
||||||
|
|
||||||
TempBuffer = AllocateZeroPool (TempBufferSize);
|
TempBuffer = AllocateZeroPool (TempBufferSize);
|
||||||
|
ASSERT (TempBuffer != NULL);
|
||||||
|
|
||||||
OrigTempBuffer = TempBuffer;
|
OrigTempBuffer = TempBuffer;
|
||||||
|
|
||||||
//
|
//
|
||||||
|
|
|
@ -1233,6 +1233,7 @@ GetSelectionInputPopUp (
|
||||||
//
|
//
|
||||||
if (StrLen (StringPtr) > (PopUpWidth - 1)) {
|
if (StrLen (StringPtr) > (PopUpWidth - 1)) {
|
||||||
TempStringPtr = AllocateZeroPool (sizeof (CHAR16) * (PopUpWidth - 1));
|
TempStringPtr = AllocateZeroPool (sizeof (CHAR16) * (PopUpWidth - 1));
|
||||||
|
ASSERT (TempStringPtr != NULL);
|
||||||
CopyMem (TempStringPtr, StringPtr, (sizeof (CHAR16) * (PopUpWidth - 5)));
|
CopyMem (TempStringPtr, StringPtr, (sizeof (CHAR16) * (PopUpWidth - 5)));
|
||||||
gBS->FreePool (StringPtr);
|
gBS->FreePool (StringPtr);
|
||||||
StringPtr = TempStringPtr;
|
StringPtr = TempStringPtr;
|
||||||
|
|
|
@ -3003,7 +3003,10 @@ Returns:
|
||||||
FileFormTags->VariableDefinitions->VariableFakeSize = (UINT16) (FileFormTags->VariableDefinitions->VariableFakeSize + FileFormTags->VariableDefinitions->VariableSize);
|
FileFormTags->VariableDefinitions->VariableFakeSize = (UINT16) (FileFormTags->VariableDefinitions->VariableFakeSize + FileFormTags->VariableDefinitions->VariableSize);
|
||||||
|
|
||||||
FileFormTags->VariableDefinitions->NvRamMap = AllocateZeroPool (FileFormTags->VariableDefinitions->VariableSize);
|
FileFormTags->VariableDefinitions->NvRamMap = AllocateZeroPool (FileFormTags->VariableDefinitions->VariableSize);
|
||||||
|
ASSERT (FileFormTags->VariableDefinitions->NvRamMap != NULL);
|
||||||
|
|
||||||
FileFormTags->VariableDefinitions->FakeNvRamMap = AllocateZeroPool (NvMapSize + FileFormTags->VariableDefinitions->VariableFakeSize);
|
FileFormTags->VariableDefinitions->FakeNvRamMap = AllocateZeroPool (NvMapSize + FileFormTags->VariableDefinitions->VariableFakeSize);
|
||||||
|
ASSERT (FileFormTags->VariableDefinitions->FakeNvRamMap != NULL);
|
||||||
|
|
||||||
CopyMem (FileFormTags->VariableDefinitions->NvRamMap, NvMap, NvMapSize);
|
CopyMem (FileFormTags->VariableDefinitions->NvRamMap, NvMap, NvMapSize);
|
||||||
gBS->FreePool (NvMapListHead);
|
gBS->FreePool (NvMapListHead);
|
||||||
|
|
Loading…
Reference in New Issue