Additional Klocwork scanning fixes in BootloaderCommonPkg

This patches fixed additional Klocwork scanning reports in
BootloaderCommonPkg.

Signed-off-by: Maurice Ma <maurice.ma@intel.com>
This commit is contained in:
Maurice Ma 2019-11-01 10:41:41 -07:00
parent 0087546e83
commit e0a19d2814
2 changed files with 3 additions and 3 deletions

View File

@ -47,7 +47,7 @@ AddMemoryResourceRange (
MemoryRanges[1].Type = EfiReservedMemoryType;
MemoryRanges[2].Type = EfiMaxMemoryType;
for (Index = 0; (MemoryRanges[Index].Type != EfiMaxMemoryType) && (Index < EfiMaxMemoryType); Index++) {
for (Index = 0; (MemoryRanges[Index].Type != EfiMaxMemoryType) && (Index < ARRAY_SIZE (MemoryRanges)); Index++) {
CoreAddMemoryDescriptor (MemoryRanges[Index].Type, MemoryRanges[Index].BaseAddress, MemoryRanges[Index].NumberOfPages,
MEMORY_ATTRIBUTE_DEFAULT);
}

View File

@ -79,7 +79,7 @@ AppendSeedData (
// Get Seed List HOB
SeedListInfoHob = GetSeedListInfoHOB(&SeedListHobLen);
if ((SeedListInfoHob == NULL)&& (SeedListHobLen < 0)) {
if ((SeedListInfoHob == NULL) || (SeedListHobLen == 0)) {
return EFI_NOT_FOUND;
}
@ -91,7 +91,7 @@ AppendSeedData (
SeedEntryData = (SEED_ENTRY *)((UINT8 *)SeedListInfoHob + Offset); //Start of SeedEntryData buffer
// This is to get to the 'last' seed entry data pointer that was added so far
for( Index =0; Index <= SeedListInfoHob->TotalSeedCount; Index++) {
for (Index = 0; Index <= SeedListInfoHob->TotalSeedCount; Index++) {
SeedEntryData = (SEED_ENTRY *)((UINT8 *)SeedEntryData + SeedEntryData->SeedEntrySize);
}