From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by mx.groups.io with SMTP id smtpd.web10.10425.1584518906952512377 for ; Wed, 18 Mar 2020 01:08:27 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.120, mailfrom: guomin.jiang@intel.com) IronPort-SDR: USGFhL2mcxCKLZZNmqOYYk5rCMvhFuDYFaqRvtYgH+l0oljh55wRTSv6XWBHPAxsOBZ1vmrJ2g bxinnk2ni4Ag== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Mar 2020 01:08:26 -0700 IronPort-SDR: NIahXYeVC8HPdCXuY36wtaQSZifRo5B4RcSa9NvXtifX33IDP7QlC6nlO4C4rxEA8z6WNxkpX7 jZF1fPpqGZfQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,566,1574150400"; d="scan'208";a="445784711" Received: from guominji-mobl.ccr.corp.intel.com ([10.238.5.173]) by fmsmga006.fm.intel.com with ESMTP; 18 Mar 2020 01:08:25 -0700 From: "Jiang, Guomin" To: devel@edk2.groups.io Cc: Guomin Jiang , Ray Ni , Rangasai V Chaganty Subject: [edk2-platform][PATCH] IntelSiliconPkg/IntelPciDeviceSecurityDxe: Replace ASSERT_EFI_ERROR Date: Wed, 18 Mar 2020 16:08:24 +0800 Message-Id: <20200318080824.1160-1-guomin.jiang@intel.com> X-Mailer: git-send-email 2.25.1.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Guomin Jiang REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2589 According to code logic, it just judge the condition rather than the status, it should use the ASSERT rather than ASSERT_EFI_ERROR. Cc: Ray Ni Cc: Rangasai V Chaganty Signed-off-by: Guomin Jiang --- .../IntelPciDeviceSecurityDxe/IntelPciDeviceSecurityDxe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Silicon/Intel/IntelSiliconPkg/Feature/PcieSecurity/IntelPciDev= iceSecurityDxe/IntelPciDeviceSecurityDxe.c b/Silicon/Intel/IntelSiliconPkg/= Feature/PcieSecurity/IntelPciDeviceSecurityDxe/IntelPciDeviceSecurityDxe.c index 29b6916f61..aa2bf14fa8 100644 --- a/Silicon/Intel/IntelSiliconPkg/Feature/PcieSecurity/IntelPciDeviceSecu= rityDxe/IntelPciDeviceSecurityDxe.c +++ b/Silicon/Intel/IntelSiliconPkg/Feature/PcieSecurity/IntelPciDeviceSecu= rityDxe/IntelPciDeviceSecurityDxe.c @@ -69,7 +69,7 @@ RecordPciDeviceInList( ASSERT_EFI_ERROR(Status);=0D =0D NewPciDevice =3D AllocateZeroPool(sizeof(*NewPciDevice));=0D - ASSERT_EFI_ERROR(NewPciDevice !=3D NULL);=0D + ASSERT(NewPciDevice !=3D NULL);=0D =0D NewPciDevice->Signature =3D PCI_DEVICE_INSTANCE_SIGNATURE;=0D NewPciDevice->PciSegment =3D PciSegment;=0D --=20 2.25.1.windows.1