From nobody Tue Feb 10 05:10:46 2026 Delivered-To: importer@patchew.org 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+79981+1787277+3901457@groups.io; helo=mail02.groups.io; 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+79981+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1630388335; cv=none; d=zohomail.com; s=zohoarc; b=CuyT9gsdjd/GmTGsFlXqbwNmzV23VxK9WP5nwJqVFsTF0Ew4a+uASSCY3Wr1m5PphPYokUvuzQJeQvOL5fQzNl7jbUKllrTNE5JtaZJF/ecOFaX3yjVQKNSrfyRoWIUhMqmXikLAfdjP33tJr1NScfmf2TDSkfNwHx45tF+60uE= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1630388335; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=JbUXtmGH6NAezHFrhG6sCNGFlc1VY5xSUOh5B2PgUM8=; b=NX9odCnwx/J7km78xSH6pARoajy2H7JYkCvbl/IAghZkrh4BP6KeacMNwqaewllg587AYttPAlmM5ZaaTOKdhfosdoKVf8FjvUgbxs+6TP0ZeDpWI/B9RBuv3rMApWzIO6TPvJEs5890VUIkHOJA8H1oGO7tKpg8gWQ+aNpB6JU= ARC-Authentication-Results: i=1; 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+79981+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1630388335656769.1795673146158; Mon, 30 Aug 2021 22:38:55 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id IKMZYY1788612xBOI4COG57N; Mon, 30 Aug 2021 22:38:55 -0700 X-Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mx.groups.io with SMTP id smtpd.web11.1672.1630388334079803018 for ; Mon, 30 Aug 2021 22:38:54 -0700 X-IronPort-AV: E=McAfee;i="6200,9189,10092"; a="240654317" X-IronPort-AV: E=Sophos;i="5.84,365,1620716400"; d="scan'208";a="240654317" X-Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Aug 2021 22:38:46 -0700 X-IronPort-AV: E=Sophos;i="5.84,365,1620716400"; d="scan'208";a="540874281" X-Received: from duntan-mobl.ccr.corp.intel.com ([10.238.1.156]) by fmsmga002-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Aug 2021 22:38:44 -0700 From: "duntan" To: devel@edk2.groups.io Cc: Guo Dong , Ray Ni , Maurice Ma , Benjamin You , Zhiguang Liu , DunTan Subject: [edk2-devel] [Patch V2 1/2] UefiPayloadPkg: Fix the bug in dump guid HOB info functions Date: Tue, 31 Aug 2021 13:38:15 +0800 Message-Id: <20210831053816.2366-2-dun.tan@intel.com> In-Reply-To: <20210831053816.2366-1-dun.tan@intel.com> References: <20210831053816.2366-1-dun.tan@intel.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: 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,dun.tan@intel.com X-Gm-Message-State: u0DatWyKNjesrkagbIxD9IH6x1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1630388335; bh=Gxkc7QtVYSwGOaM0zKADDHdLOgqrPE6wWaPYef0Giyk=; h=Cc:Date:From:Reply-To:Subject:To; b=MA2XGK2T4oxQwxR+Mi40HkP48UgQSVNbAbkDwxVM/KSU4+ubA/ObcpGVPCV3eskQjzn uZ8hxgSNEgJ996aWE246WRT7hSxUZN2h0xi9atgC8ZT1Njr5KQ5dghsSm9Ua3v0y1WVVr mMLFDIg1j13l4mOwi4lKhm2xzjf9XcCXSWo= X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1630388336802100008 Content-Type: text/plain; charset="utf-8" The input HobLength of PrintHandler should be data size instead of whole le= ngth of HOB Cc: Guo Dong Cc: Ray Ni Cc: Maurice Ma Cc: Benjamin You Cc: Zhiguang Liu Signed-off-by: DunTan Reviewed-by: Ray Ni Reviewed-by: Zhiguang Liu --- UefiPayloadPkg/UefiPayloadEntry/PrintHob.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/UefiPayloadPkg/UefiPayloadEntry/PrintHob.c b/UefiPayloadPkg/Ue= fiPayloadEntry/PrintHob.c index 5fb638d4a4..f93aeec472 100644 --- a/UefiPayloadPkg/UefiPayloadEntry/PrintHob.c +++ b/UefiPayloadPkg/UefiPayloadEntry/PrintHob.c @@ -317,9 +317,11 @@ PrintPciRootBridgeInfoGuidHob ( { UNIVERSAL_PAYLOAD_PCI_ROOT_BRIDGES *PciRootBridges; UINTN Index; + UINTN Length; Index =3D 0; PciRootBridges =3D (UNIVERSAL_PAYLOAD_PCI_ROOT_BRIDGES *) GET_GUID_HOB_D= ATA (HobRaw); - ASSERT (HobLength >=3D sizeof (UNIVERSAL_PAYLOAD_PCI_ROOT_BRIDGES)); + Length =3D sizeof (UNIVERSAL_PAYLOAD_PCI_ROOT_BRIDGES) + PciRootBridges-= >Count * sizeof (UNIVERSAL_PAYLOAD_PCI_ROOT_BRIDGE); + ASSERT (HobLength >=3D Length); DEBUG ((DEBUG_INFO, " Revision =3D 0x%x\n", PciRootBridges->He= ader.Revision)); DEBUG ((DEBUG_INFO, " Length =3D 0x%x\n", PciRootBridges->He= ader.Length)); DEBUG ((DEBUG_INFO, " Count =3D 0x%x\n", PciRootBridges->Co= unt)); @@ -369,10 +371,12 @@ PrintExtraDataGuidHob ( { UNIVERSAL_PAYLOAD_EXTRA_DATA *ExtraData; UINTN Index; + UINTN Length; =20 Index =3D 0; ExtraData =3D (UNIVERSAL_PAYLOAD_EXTRA_DATA *) GET_GUID_HOB_DATA (HobRaw= ); - ASSERT (HobLength >=3D ExtraData->Header.Length); + Length =3D sizeof (UNIVERSAL_PAYLOAD_EXTRA_DATA) + ExtraData->Count * si= zeof (UNIVERSAL_PAYLOAD_EXTRA_DATA_ENTRY); + ASSERT (HobLength >=3D Length); DEBUG ((DEBUG_INFO, " Revision =3D 0x%x\n", ExtraData->Header.Revisio= n)); DEBUG ((DEBUG_INFO, " Length =3D 0x%x\n", ExtraData->Header.Length)= ); DEBUG ((DEBUG_INFO, " Count =3D 0x%x\n", ExtraData->Count)); @@ -443,7 +447,7 @@ PrintGuidHob ( for (Index =3D 0; Index < ARRAY_SIZE (GuidHobPrintHandleTable); Index++)= { if (CompareGuid (&Hob.Guid->Name, GuidHobPrintHandleTable[Index].Guid)= ) { DEBUG ((DEBUG_INFO, " Guid =3D %a\n", GuidHobPrintHandleTable[In= dex].GuidName)); - Status =3D GuidHobPrintHandleTable[Index].PrintHandler (Hob.Raw, Hob= .Header->HobLength); + Status =3D GuidHobPrintHandleTable[Index].PrintHandler (Hob.Raw, GET= _GUID_HOB_DATA_SIZE (Hob.Raw)); return Status; } } --=20 2.31.1.windows.1 -=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 (#79981): https://edk2.groups.io/g/devel/message/79981 Mute This Topic: https://groups.io/mt/85270356/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-