From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web12.13430.1633620767434593266 for ; Thu, 07 Oct 2021 08:32:47 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: pierre.gondois@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 02E30113E; Thu, 7 Oct 2021 08:32:41 -0700 (PDT) Received: from e120189.arm.com (unknown [10.57.72.88]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id ECF7A3F66F; Thu, 7 Oct 2021 08:32:39 -0700 (PDT) From: "PierreGondois" To: devel@edk2.groups.io, Sami Mujawar , Alexei.Fedorov@arm.com Subject: [PATCH v2 02/21] DynamicTablesPkg: Add missing parameter check Date: Thu, 7 Oct 2021 16:31:51 +0100 Message-Id: <20211007153210.26608-3-Pierre.Gondois@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20211007153210.26608-1-Pierre.Gondois@arm.com> References: <20211007153210.26608-1-Pierre.Gondois@arm.com> From: Pierre Gondois Add missing check of 'AcpiTableInfo' in AddAcpiHeader(). Signed-off-by: Pierre Gondois --- Notes: v2: - New patch. [Pierre] DynamicTablesPkg/Library/Common/TableHelperLib/TableHelper.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/DynamicTablesPkg/Library/Common/TableHelperLib/TableHelper.c b/DynamicTablesPkg/Library/Common/TableHelperLib/TableHelper.c index 9249e6b87f70..f98da7ffdd67 100644 --- a/DynamicTablesPkg/Library/Common/TableHelperLib/TableHelper.c +++ b/DynamicTablesPkg/Library/Common/TableHelperLib/TableHelper.c @@ -122,11 +122,13 @@ AddAcpiHeader ( ASSERT (CfgMgrProtocol != NULL); ASSERT (Generator != NULL); ASSERT (AcpiHeader != NULL); + ASSERT (AcpiTableInfo != NULL); ASSERT (Length >= sizeof (EFI_ACPI_DESCRIPTION_HEADER)); if ((CfgMgrProtocol == NULL) || (Generator == NULL) || (AcpiHeader == NULL) || + (AcpiTableInfo == NULL) || (Length < sizeof (EFI_ACPI_DESCRIPTION_HEADER)) ) { return EFI_INVALID_PARAMETER; -- 2.17.1