Code logic ensures that both pointers 'DriverInfoData' and 'AllocInfoData' will not be NULL when they are used.
Add ASSERTs as warning for cases that will not happen. Cc: Star Zeng <[email protected]> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu <[email protected]> --- MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c | 3 +++ MdeModulePkg/Core/PiSmmCore/SmramProfileRecord.c | 3 +++ 2 files changed, 6 insertions(+) diff --git a/MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c b/MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c index 14c4959..30c0df4 100644 --- a/MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c +++ b/MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c @@ -1283,6 +1283,9 @@ CoreUpdateProfileFree ( } } + ASSERT (DriverInfoData != NULL); + ASSERT (AllocInfoData != NULL); + Found = TRUE; Context = &ContextData->Context; diff --git a/MdeModulePkg/Core/PiSmmCore/SmramProfileRecord.c b/MdeModulePkg/Core/PiSmmCore/SmramProfileRecord.c index 281e382..ac832ce 100644 --- a/MdeModulePkg/Core/PiSmmCore/SmramProfileRecord.c +++ b/MdeModulePkg/Core/PiSmmCore/SmramProfileRecord.c @@ -1403,6 +1403,9 @@ SmmCoreUpdateProfileFree ( } } + ASSERT (DriverInfoData != NULL); + ASSERT (AllocInfoData != NULL); + Found = TRUE; Context = &ContextData->Context; -- 1.9.5.msysgit.0 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
