Hi Ian/Vincent, It is better to add the reviewers/maintainers in the CC list. And also add the one who already give the comment. You can do this with 'git send-email -cc xxx.com' or add Cc: Name <email_address> in the commit message.
Reviewed-by: Zhichao Gao <zhichao....@intel.com> Thanks, Zhichao > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of IanX > Kuo > Sent: Thursday, March 18, 2021 12:02 AM > To: devel@edk2.groups.io > Cc: Ke, VincentX <vincentx...@intel.com> > Subject: [edk2-devel] [PATCH v4] ShellPkg/Pci: Add valid check for PCI > extended config space parser > > From: VincentX Ke <vincentx...@intel.com> > > Bugzilla: 3262 (https://bugzilla.tianocore.org/show_bug.cgi?id=3262) > > No need to print PCIe details while CapabilityId is 0xFFFF. > Limit the NextCapabilityOffset to PCI/PCIe configuration space. > > Signed-off-by: VincentX Ke <vincentx...@intel.com> > --- > ShellPkg/Library/UefiShellDebug1CommandsLib/Pci.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/Pci.c > b/ShellPkg/Library/UefiShellDebug1CommandsLib/Pci.c > index a2f04d8db5..1e5dc75e27 100644 > --- a/ShellPkg/Library/UefiShellDebug1CommandsLib/Pci.c > +++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/Pci.c > @@ -2038,12 +2038,14 @@ LocatePciCapability ( > @param[in] PciExpressCap PCI Express capability buffer. @param[in] > ExtendedConfigSpace PCI Express extended configuration space.+ > @param[in] ExtendedConfigSize PCI Express extended configuration size. > @param[in] ExtendedCapability PCI Express extended capability ID to > explain. **/ VOID PciExplainPciExpress ( IN PCI_CAPABILITY_PCIEXP > *PciExpressCap, IN UINT8 > *ExtendedConfigSpace,+ IN > UINTN ExtendedConfigSize, IN CONST UINT16 > ExtendedCapability ); @@ -2921,6 +2923,7 @@ ShellCommandRunPci ( > PciExplainPciExpress ( (PCI_CAPABILITY_PCIEXP *) ((UINT8 *) > &ConfigSpace + PcieCapabilityPtr), ExtendedConfigSpace,+ > ExtendedConfigSize, ExtendedCapability ); }@@ > -5698,12 > +5701,14 @@ PrintPciExtendedCapabilityDetails( > @param[in] PciExpressCap PCI Express capability buffer. @param[in] > ExtendedConfigSpace PCI Express extended configuration space.+ > @param[in] ExtendedConfigSize PCI Express extended configuration size. > @param[in] ExtendedCapability PCI Express extended capability ID to > explain. **/ VOID PciExplainPciExpress ( IN PCI_CAPABILITY_PCIEXP > *PciExpressCap, IN UINT8 > *ExtendedConfigSpace,+ IN > UINTN ExtendedConfigSize, IN CONST UINT16 > ExtendedCapability ) {@@ -5786,7 +5791,7 @@ PciExplainPciExpress ( > } ExtHdr = (PCI_EXP_EXT_HDR*)ExtendedConfigSpace;- while (ExtHdr- > >CapabilityId != 0 && ExtHdr->CapabilityVersion != 0) {+ while (ExtHdr- > >CapabilityId != 0 && ExtHdr->CapabilityVersion != 0 && ExtHdr- > >CapabilityId != 0xFFFF) { // // Process this item //@@ -5800,7 > >+5805,8 > @@ PciExplainPciExpress ( > // // Advance to the next item if it exists //- if (ExtHdr- > >NextCapabilityOffset != 0) {+ if (ExtHdr->NextCapabilityOffset != 0 &&+ > (ExtHdr->NextCapabilityOffset <= (UINT32) (ExtendedConfigSize + > EFI_PCIE_CAPABILITY_BASE_OFFSET - sizeof (PCI_EXP_EXT_HDR)))) > { ExtHdr = (PCI_EXP_EXT_HDR*)(ExtendedConfigSpace + ExtHdr- > >NextCapabilityOffset - EFI_PCIE_CAPABILITY_BASE_OFFSET); } else > { break;-- > 2.18.0.windows.1 > > > > -=-=-=-=-=-= > Groups.io Links: You receive all messages sent to this group. > View/Reply Online (#72997): https://edk2.groups.io/g/devel/message/72997 > Mute This Topic: https://groups.io/mt/81421888/1768756 > Group Owner: devel+ow...@edk2.groups.io > Unsubscribe: https://edk2.groups.io/g/devel/unsub [zhichao....@intel.com] > -=-=-=-=-=-= > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#73038): https://edk2.groups.io/g/devel/message/73038 Mute This Topic: https://groups.io/mt/81421888/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
