From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mx.groups.io with SMTP id smtpd.web11.10474.1584519869635871846 for ; Wed, 18 Mar 2020 01:24:29 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.31, mailfrom: guomin.jiang@intel.com) IronPort-SDR: pO0P2IaYoqI++uXwlCCe1tp3G9z3POe+7L+6f3Jr+IevPLUCc2+bMOW+nSt/2k7pgUrPsC+V1b Joh1VbmNOdgg== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Mar 2020 01:24:29 -0700 IronPort-SDR: tQfDtAe1BFmgWQldd9XrJNUJreZnewgZzdcq7pTHZgSps6EaM63tBk+pxd3eEbvWctrla9NMlV WIuGThvmGtxA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,566,1574150400"; d="scan'208";a="445788998" Received: from guominji-mobl.ccr.corp.intel.com ([10.238.5.173]) by fmsmga006.fm.intel.com with ESMTP; 18 Mar 2020 01:24:28 -0700 From: "Jiang, Guomin" To: devel@edk2.groups.io Subject: [edk2-platform][PATCH] IntelSiliconPkg/IntelPciDeviceSecurityDxe: Replace ASSERT_EFI_ERROR Date: Wed, 18 Mar 2020 16:24:27 +0800 Message-Id: <20200318082427.1295-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