From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by mx.groups.io with SMTP id smtpd.web11.14927.1584538428890160341 for ; Wed, 18 Mar 2020 06:33:49 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.24, mailfrom: guomin.jiang@intel.com) IronPort-SDR: ejaMqYsf4e8jx0w9AXl/AHqFBq5Y3aVKg0xyZOsNbnkZyBOoWEeuBEP4RVquPKCyqPjpZ4fGtJ 1pfJRx6qnaeA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Mar 2020 06:33:46 -0700 IronPort-SDR: zCKMw0u+Dv/MzR2ICacIexBtlIHyzG+9lMqULLbztx0RIJ90KtJmJq8oPN3UHFuD+nJUJFqh1Q AvcEPTZMaaDQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,567,1574150400"; d="scan'208";a="263383768" Received: from guominji-mobl.ccr.corp.intel.com ([10.254.212.104]) by orsmga002.jf.intel.com with ESMTP; 18 Mar 2020 06:33:45 -0700 From: "Jiang, Guomin" To: devel@edk2.groups.io Subject: [edk2-platforms][PATCH] IntelSiliconPkg/IntelPciDeviceSecurityDxe: Replace ASSERT_EFI_ERROR Date: Wed, 18 Mar 2020 21:33:43 +0800 Message-Id: <20200318133343.393-1-guomin.jiang@intel.com> X-Mailer: git-send-email 2.25.1.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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