From nobody Mon Feb 9 04:45:20 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+68582+1787277+3901457@groups.io; arc=fail (BodyHash is different from the expected one); dmarc=fail(p=none dis=none) header.from=arm.com Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1607539683063555.956913358409; Wed, 9 Dec 2020 10:48:03 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id USSiYY1788612xH9m9Qj0o43; Wed, 09 Dec 2020 10:48:02 -0800 X-Received: from EUR05-DB8-obe.outbound.protection.outlook.com (EUR05-DB8-obe.outbound.protection.outlook.com [40.107.20.72]) by mx.groups.io with SMTP id smtpd.web10.454.1607539667458992399 for ; Wed, 09 Dec 2020 10:47:47 -0800 X-Received: from MR2P264CA0079.FRAP264.PROD.OUTLOOK.COM (2603:10a6:500:32::19) by AM0PR08MB5043.eurprd08.prod.outlook.com (2603:10a6:208:166::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3632.17; Wed, 9 Dec 2020 18:47:45 +0000 X-Received: from VE1EUR03FT031.eop-EUR03.prod.protection.outlook.com (2603:10a6:500:32:cafe::cd) by MR2P264CA0079.outlook.office365.com (2603:10a6:500:32::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3632.21 via Frontend Transport; Wed, 9 Dec 2020 18:47:45 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 63.35.35.123) smtp.mailfrom=arm.com; edk2.groups.io; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com;edk2.groups.io; dmarc=pass action=none header.from=arm.com; Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+68582+1787277+3901457@groups.io; helo=mail02.groups.io; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 63.35.35.123 as permitted sender) receiver=protection.outlook.com; client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com; X-Received: from 64aa7808-outbound-1.mta.getcheckrecipient.com (63.35.35.123) by VE1EUR03FT031.mail.protection.outlook.com (10.152.18.69) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3654.12 via Frontend Transport; Wed, 9 Dec 2020 18:47:44 +0000 X-Received: ("Tessian outbound eeda57fffe7b:v71"); Wed, 09 Dec 2020 18:47:44 +0000 X-CheckRecipientChecked: true X-CR-MTA-CID: 2a71889da3d9d13c X-CR-MTA-TID: 64aa7808 X-Received: from a08d53c29784.1 by 64aa7808-outbound-1.mta.getcheckrecipient.com id CFDD7EB9-D446-429C-8813-89E47202EC56.1; Wed, 09 Dec 2020 18:47:29 +0000 X-Received: from EUR03-DB5-obe.outbound.protection.outlook.com by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id a08d53c29784.1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Wed, 09 Dec 2020 18:47:29 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fDHBl8T+wFhbp6eWXAG+Hh5OnFN9dTuNidRuiNUL0GGpfGiGqsOoPaRuRB6zZH5VLgt5TR4ppgJmhH/SH5jQ6z/N+dEarv+oTdBMkZgui5qC0ogaLssk2HAt4735w8QDU5N3vN/fsfQi8t63GYU2CvD4eL7O01QbePdyxSVy/YTE+l+6TlKoHGVBCROCXXngpr+O6fbPs5L2gVT3NsBzD4QVXRtqXt3fhPS/dtdH84hyJ1Up4rgS8gRffIomu3QZdvlix/R9q4C1vTI7R9eWn7yk0VhMpO2UoMkGeM0HgM/5tlPt6sxUvXrYoMB5xh1F+DuQmRb+/t/lRnf/mX0xFw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=de11ozc+Wq4XDtVO58tZpt46ZTAphy8tW53hW6xboy8=; b=eB0aLMy8ZM50yWMn99KCTv+3JivSpwdvGGmdIhv/yKtSw7F8Mqle3MfDOWbW36F03/Hevev2CnoA3gtE+IGtyVGRRwcQwz9e1oI+FP5MdEuGfa96sOpZBF1H9AH8H1bxH1a5M3hPtDg4jmpzAgR7bXXVBli11YNQR+dk6eadQFHtOUENK90J2cr+es6c/kCBYC3V9YB1PBq92BxmbkMmgD60YpjxQreGp9fJtJ0Vlymt7VrqZCY8U2vaVukHnJYELqlsp7SMSKqqdYSVZWo1H9/gcbyX8K+xaTcqzxkViLg34ExbAkyTEMBLhRLPpUExPsqaLUmi3K9s+MYamNF7kA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 40.67.248.234) smtp.rcpttodomain=edk2.groups.io smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=none (message not signed); arc=none X-Received: from AM0PR04CA0014.eurprd04.prod.outlook.com (2603:10a6:208:122::27) by HE1PR0801MB1883.eurprd08.prod.outlook.com (2603:10a6:3:4f::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3654.12; Wed, 9 Dec 2020 18:47:27 +0000 X-Received: from AM5EUR03FT047.eop-EUR03.prod.protection.outlook.com (2603:10a6:208:122:cafe::c4) by AM0PR04CA0014.outlook.office365.com (2603:10a6:208:122::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3654.12 via Frontend Transport; Wed, 9 Dec 2020 18:47:27 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 40.67.248.234) smtp.mailfrom=arm.com; edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; dmarc=pass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 40.67.248.234 as permitted sender) receiver=protection.outlook.com; client-ip=40.67.248.234; helo=nebula.arm.com; X-Received: from nebula.arm.com (40.67.248.234) by AM5EUR03FT047.mail.protection.outlook.com (10.152.16.197) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.3654.12 via Frontend Transport; Wed, 9 Dec 2020 18:47:26 +0000 X-Received: from AZ-NEU-EX04.Arm.com (10.251.24.32) by AZ-NEU-EX03.Arm.com (10.251.24.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2044.4; Wed, 9 Dec 2020 18:47:15 +0000 X-Received: from E107187.Arm.com (10.57.26.36) by mail.arm.com (10.251.24.32) with Microsoft SMTP Server id 15.1.2044.4 via Frontend Transport; Wed, 9 Dec 2020 18:47:15 +0000 From: "Sami Mujawar" To: CC: Sami Mujawar , , , , , , , , Subject: [edk2-devel] [PATCH v1 06/25] StandaloneMmPkg: Fix ECC error 3002 in StandaloneMmCpu Date: Wed, 9 Dec 2020 18:46:51 +0000 Message-ID: <20201209184710.20912-7-sami.mujawar@arm.com> In-Reply-To: <20201209184710.20912-1-sami.mujawar@arm.com> References: <20201209184710.20912-1-sami.mujawar@arm.com> MIME-Version: 1.0 X-EOPAttributedMessage: 1 X-MS-Office365-Filtering-HT: Tenant X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: f92d3fb6-510a-44be-7ca7-08d89c72eaa1 X-MS-TrafficTypeDiagnostic: HE1PR0801MB1883:|AM0PR08MB5043: X-Microsoft-Antispam-PRVS: x-checkrecipientrouted: true NoDisclaimer: true X-MS-Oob-TLC-OOBClassifiers: OLM:8273;OLM:8273; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Untrusted: BCL:0; X-Microsoft-Antispam-Message-Info-Original: fzt0Txsaj8F0NPfv099G9vzUYQMKPhsqruR8PyZDmQKvgTg+0JYtV9jazgNLiGv7RVpiokvSWmPNbRkPc/YU3mSg9JTLhURTjEhh64wJFIL8KVUUFQvnaE5pM+dzlLocZZJdMLiHit/PckBZ99fmgl6abbJAc68f1JP4lpPtxdO3LFf0TWMLcrx6/9JWZv+vOV8h8y6w64R6o8YkAyoJj80uoxG30MRkS+0o2Az+C171k17gCDPupj5glSQlmuqxCFZ2Sl6b5bwtJx+SC6prvIe6UXm+IFrsLhEAtNS4kJI2cSe1GFpPmXTCJDvcey4r+Ti2BxmBH420V+OJOZSTfivSmQtu3d0lN3eRgkTuSrkzPqxQKlbQ3MPfr31UN+Q19mweeLt2K/Qh7KmZ+U3Zig== X-Forefront-Antispam-Report-Untrusted: CIP:40.67.248.234;CTRY:IE;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:nebula.arm.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(6029001)(4636009)(376002)(346002)(136003)(46966005)(83380400001)(426003)(1076003)(26005)(508600001)(5660300002)(8936002)(7696005)(4326008)(47076004)(82310400003)(6916009)(2616005)(86362001)(6666004)(44832011)(2906002)(70586007)(186003)(36756003)(81166007)(8676002)(336012)(356005)(54906003)(70206006);DIR:OUT;SFP:1101; X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0801MB1883 X-MS-Exchange-Transport-CrossTenantHeadersStripped: VE1EUR03FT031.eop-EUR03.prod.protection.outlook.com X-MS-Office365-Filtering-Correlation-Id-Prvs: afcd5ccf-b0aa-4718-ee84-08d89c72dfef X-Microsoft-Antispam-Message-Info: w+4UQZMDuOLBrC1OHXMVRkTr/VTsj8pVZtCxe8cDIExcKWeOc5XgxEilsE7O5PS0MQ0g1nS813yF/69pZsKe3Y1OhRZlq4WblcchUPhSpkAJbvZTZM3rGA8TeGU9MN4oF96A8tJtdrrGQo3xFy6EG26yfkemg5z+JA0wj9WZppO+4tbIbNKOaajcKTHI1rsg16iUATsV1EWqQlor5iEAcTDrDthczKhOkm3IYeTz08OrHQuoRgeCCqX4tyHRRvSf32iyocXDR+j0ZTDSAnWgCjvEkIkNJWnfK2HSy0hV9CubMYIwquyGH/SRpqiKSkOhG3wl30yFXLkPiYC53/NuaGANMoZKjKknvGvw81pQrdmFvPhStRM+aOxUxhW6UuzMTwCz6djQSNxqeDWLnJA7RA== X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Dec 2020 18:47:44.8322 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: f92d3fb6-510a-44be-7ca7-08d89c72eaa1 X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com] X-MS-Exchange-CrossTenant-AuthSource: VE1EUR03FT031.eop-EUR03.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR08MB5043 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,sami.mujawar@arm.com X-Gm-Message-State: ewbGC3U2r3TGZjDZWyFFUjh5x1787277AA= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1607539682; bh=tKEONjY8eVGuFtts5Aa+9W26ZnTD19sXjCgPRMMPT8U=; h=CC:Content-Type:Date:From:Reply-To:Subject:To; b=w+qTXOl6Ovauxb3mLAYndR93xP1tA2ZoCttsD/TcNlnz3pB7wEG1M38ZEpw+2VklwQH cLaE63KjEyrRqy9zt6MGRaoDuWO9OODOTw2YTWiR3SKx2nzNaxRgz0C4MyNVPmN5jMCpc X6roT6N+bBEJ2j/XbsCkG8YE8+3jzrUVg9c= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Fix the ECC tool reported error "[3002] Non-Boolean comparisons should use a compare operator". Also fix the following: - add curly braces for 'if' condition statements to comply with the coding standard. - The value returned by GET_GUID_HOB_DATA() is stored in *HobData. Therefore, check *HobData against NULL. The original code was checking HobData which is incorrect. Signed-off-by: Sami Mujawar --- StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/EventHandle.c | 19 +++= ++++++++++------ StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.c | 11 +++= ++++---- 2 files changed, 20 insertions(+), 10 deletions(-) diff --git a/StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/EventHandle.c = b/StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/EventHandle.c index 6a25c4c54843ccdc2d745e8e537942d49b1d1141..9738e8bd60149efb45ef1b66139= e0cec38b6c1fe 100644 --- a/StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/EventHandle.c +++ b/StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/EventHandle.c @@ -1,7 +1,7 @@ /** @file =20 Copyright (c) 2016 HP Development Company, L.P. - Copyright (c) 2016 - 2018, ARM Limited. All rights reserved. + Copyright (c) 2016 - 2020, Arm Limited. All rights reserved. =20 SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -84,12 +84,18 @@ PiMmStandaloneArmTfCpuDriverEntry ( } =20 // Perform parameter validation of NsCommBufferAddr - if (NsCommBufferAddr && (NsCommBufferAddr < mNsCommBuffer.PhysicalStart)) + if (NsCommBufferAddr =3D=3D (UINTN)NULL) { + return EFI_INVALID_PARAMETER; + } + + if (NsCommBufferAddr < mNsCommBuffer.PhysicalStart) { return EFI_ACCESS_DENIED; + } =20 if ((NsCommBufferAddr + sizeof (EFI_MM_COMMUNICATE_HEADER)) >=3D - (mNsCommBuffer.PhysicalStart + mNsCommBuffer.PhysicalSize)) + (mNsCommBuffer.PhysicalStart + mNsCommBuffer.PhysicalSize)) { return EFI_INVALID_PARAMETER; + } =20 // Find out the size of the buffer passed NsCommBufferSize =3D ((EFI_MM_COMMUNICATE_HEADER *) NsCommBufferAddr)->M= essageLength + @@ -97,9 +103,9 @@ PiMmStandaloneArmTfCpuDriverEntry ( =20 // perform bounds check. if (NsCommBufferAddr + NsCommBufferSize >=3D - mNsCommBuffer.PhysicalStart + mNsCommBuffer.PhysicalSize) + mNsCommBuffer.PhysicalStart + mNsCommBuffer.PhysicalSize) { return EFI_ACCESS_DENIED; - + } =20 // Now that the secure world can see the normal world buffer, allocate // memory to copy the communication buffer to the secure world. @@ -192,8 +198,9 @@ PiMmCpuTpFwRootMmiHandler ( ASSERT (CommBufferSize =3D=3D NULL); =20 CpuNumber =3D mMmst->CurrentlyExecutingCpu; - if (!PerCpuGuidedEventContext[CpuNumber]) + if (PerCpuGuidedEventContext[CpuNumber] =3D=3D NULL) { return EFI_NOT_FOUND; + } =20 DEBUG ((DEBUG_INFO, "CommBuffer - 0x%x, CommBufferSize - 0x%x\n", PerCpuGuidedEventContext[CpuNumber], diff --git a/StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCp= u.c b/StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.c index 617babd5ab4bcb200ee10e19ec6f99c2ef163200..7c004c8013460bb7f1690351aa6= 4eeaa9eab91dc 100644 --- a/StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.c +++ b/StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.c @@ -2,7 +2,7 @@ =20 Copyright (c) 2008 - 2009, Apple Inc. All rights reserved.
Copyright (c) 2016 HP Development Company, L.P. - Copyright (c) 2016 - 2018, ARM Limited. All rights reserved. + Copyright (c) 2016 - 2020, Arm Limited. All rights reserved. =20 SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -53,16 +53,19 @@ GetGuidedHobData ( { EFI_HOB_GUID_TYPE *Hob; =20 - if (!HobList || !HobGuid || !HobData) + if ((HobList =3D=3D NULL) || (HobGuid =3D=3D NULL) || (HobData =3D=3D NU= LL)) { return EFI_INVALID_PARAMETER; + } =20 Hob =3D GetNextGuidHob (HobGuid, HobList); - if (!Hob) + if (Hob =3D=3D NULL) { return EFI_NOT_FOUND; + } =20 *HobData =3D GET_GUID_HOB_DATA (Hob); - if (!HobData) + if (*HobData =3D=3D NULL) { return EFI_NOT_FOUND; + } =20 return EFI_SUCCESS; } --=20 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#68582): https://edk2.groups.io/g/devel/message/68582 Mute This Topic: https://groups.io/mt/78836550/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-