Thanks Gerd, I will refine the code logic.
> -----Original Message----- > From: Gerd Hoffmann <kra...@redhat.com> > Sent: Friday, January 13, 2023 5:51 PM > To: Wu, Jiaxin <jiaxin...@intel.com> > Cc: devel@edk2.groups.io; Dong, Eric <eric.d...@intel.com>; Ni, Ray > <ray...@intel.com>; Zeng, Star <star.z...@intel.com>; Laszlo Ersek > <ler...@redhat.com>; Kumar, Rahul R <rahul.r.ku...@intel.com> > Subject: Re: [PATCH v1 2/4] UefiCpuPkg/PiSmmCpuDxeSmm: Consume SMM > Base Hob for SmBase info > > Hi, > > > + } else { > > + mSmmInitialized = (BOOLEAN *)AllocateZeroPool (sizeof (BOOLEAN) * > mMaxNumberOfCpus); > > + ASSERT (mSmmInitialized != NULL); > > ASSERT is not sufficient error handling. > > take care, > Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#98467): https://edk2.groups.io/g/devel/message/98467 Mute This Topic: https://groups.io/mt/96241702/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-