From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by mx.groups.io with SMTP id smtpd.web10.13258.1596178600966306945 for ; Thu, 30 Jul 2020 23:56:41 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=nk+AErCl; spf=pass (domain: intel.com, ip: 192.55.52.120, mailfrom: wei6.xu@intel.com) IronPort-SDR: on8fWpu6ZxCYU/HOmMLKJyZqO9XiQEBW8xfeebHJD6OevaZlHF08LpqenzZuYf6yEfSuTKkxjS h6lzNFKdmkNQ== X-IronPort-AV: E=McAfee;i="6000,8403,9698"; a="149205062" X-IronPort-AV: E=Sophos;i="5.75,417,1589266800"; d="scan'208";a="149205062" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Jul 2020 23:56:40 -0700 IronPort-SDR: 4c7Lyypv33BfhhPxCh0WbL2W5MQphc4SV0V7/7aPtJ0PYj3yDD+4mJk/9lXmjnns+uU/SgRJwY OrGSatGsJy+w== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,417,1589266800"; d="scan'208";a="291133049" Received: from orsmsx601.amr.corp.intel.com ([10.22.229.14]) by orsmga006.jf.intel.com with ESMTP; 30 Jul 2020 23:56:40 -0700 Received: from orsmsx605.amr.corp.intel.com (10.22.229.18) by ORSMSX601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Thu, 30 Jul 2020 23:56:39 -0700 Received: from ORSEDG001.ED.cps.intel.com (10.7.248.4) by orsmsx605.amr.corp.intel.com (10.22.229.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.1713.5 via Frontend Transport; Thu, 30 Jul 2020 23:56:39 -0700 Received: from NAM04-CO1-obe.outbound.protection.outlook.com (104.47.45.54) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 30 Jul 2020 23:56:39 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=AasSYysL4bwLLXBuqcli2YADCAnfIsUNY9PJdGWFS3L7GRwE5VD/Q5cpQ0r4z1RoVVlBvugLou6gK6fPLeszCn2g1/IOzSLDWNx/nDPTFF6S95zLZRMh24Vtgw1FTYj3AHkmqguDOQmD8uzJ0WgRGaho7YwR/UEdD6gaH+e1A26U45ONOsvrbHXMERcy1sv9wMAj7P70F3ah8rU5gcqZSM7qeBOyR0h5TiwTcUXW+hSvTPx5Fcqb+eK6/6K789L/dYLmoFoBVLxtDepkYAO7Iik0eLZRDqqmLW6Aw2N0VxdUW1m1SsFlpcJNzNXImJMIUaKQWUNZXG9G/3VMxIlmKA== 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=rSdEsYaLmjtuKZfSyiQYkWneGDoyV2Ga9IZ89QJhDnc=; b=al+U/z5yDTMvSD2VmBakpkhpKsEDhVSEw15pllEwA/MWs1Ce3YVHoB+HVu7RThc/vuem8FYXgXr4ZHTuwpRJDne5cSOH3LNoBKwHQTD7uGXMLAB3xlJutIXZ7n5ZHeJrM2e0oEhj74pkovZ0vrB0d1KOhD+tmED6NXJ5uNNtD/g08JxK0WjGqjfWTy5sS5XX+PktziBPDHVtZlmf4aDFglhF4nM/cc7d9Xs/S2Rem97Wg+vlTsv/HmLC81knGr+ej5bz6hQm0bgHAlB+MgnPGsDs65qnwVS4/i0GMXcMZmwUeSqfdx+d+CQetgivmuFQ3NJo24DgqREF2K2o1rCSlg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=rSdEsYaLmjtuKZfSyiQYkWneGDoyV2Ga9IZ89QJhDnc=; b=nk+AErClbTSKWrl1ce38aJfnX8MyBmScxcZ+V5Kb83wUKGGoGhVoJr2/rBMgd/psZtSJN5kKVI8/2B/4hDMACtyzsvtTYZena1sTKCHo0UIq1wsHP+m8gR372z5gePeIvmmecrzOj+NLYp8OjiGlC1HtUFvGOSb+tAwb/1JxhvQ= Received: from BN7PR11MB2770.namprd11.prod.outlook.com (2603:10b6:406:b4::20) by BN8PR11MB3779.namprd11.prod.outlook.com (2603:10b6:408:8a::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3239.17; Fri, 31 Jul 2020 06:56:37 +0000 Received: from BN7PR11MB2770.namprd11.prod.outlook.com ([fe80::60f0:e1d0:e203:1a2]) by BN7PR11MB2770.namprd11.prod.outlook.com ([fe80::60f0:e1d0:e203:1a2%7]) with mapi id 15.20.3239.020; Fri, 31 Jul 2020 06:56:37 +0000 From: "Xu, Wei6" To: "devel@edk2.groups.io" , "michael.kubacki@outlook.com" CC: "Gao, Liming" , "Kinney, Michael D" , "Xu, Wei6" Subject: Re: [edk2-devel] [PATCH v1 0/7] FmpDevicePkg Minor Fixes Thread-Topic: [edk2-devel] [PATCH v1 0/7] FmpDevicePkg Minor Fixes Thread-Index: AQHWZujciolqtrcaYUukA4tdCjWNGKkhQW6A Date: Fri, 31 Jul 2020 06:56:37 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-GB, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; dmarc=none action=none header.from=intel.com; x-originating-ip: [192.55.46.36] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 9e2c9cf2-d9f7-478c-2ecd-08d8351edea1 x-ms-traffictypediagnostic: BN8PR11MB3779: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:2887; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 5RwzfuUcIgehfFovcu4ZXX+cDfcwk94ql1ejoeYl842W96AqlgxBgPpQwIIc42ojwwDIizqXa0mfMWj6AlG8gvV90Rl5rTltCBw6HMdFSdbgnnPAYP+FE/1yPwkdb9E8teh4MXbU/EeqCoVH+U6NESj9Mbsdt01VG2dmJQCJae7WCglVIYwbdJnuHxruq+Q+St98zSCQXLh2siJHnQqbv54n+cJOx2B2rhWf6jh2ebAjtJSVvwBWAdiEXpfAUyLkGhvIwrjmEVquUAhT0xjlG2pe6olXUALdCWZp7vSDyXikZfmrs1mNIh23Owircmcg053Ysi0NGxS5veIB+Evf41E6gbr1hAMi+Wlx43DyIYyxpuVjPmaNID6bGR26oWHPLeltDllChTI9haztqPYTjQ== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BN7PR11MB2770.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(396003)(376002)(366004)(136003)(346002)(39860400002)(2906002)(33656002)(52536014)(7696005)(64756008)(66556008)(5660300002)(83380400001)(186003)(26005)(66446008)(4326008)(66476007)(316002)(66946007)(76116006)(8936002)(107886003)(53546011)(6506007)(110136005)(966005)(86362001)(8676002)(9686003)(55016002)(71200400001)(54906003)(478600001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: AQMT1uc43LSmVu44Z/SsElYg7j7MKGWN39/QrhOWUwDd3y7NCraAy1SHmw3z/1w6LdE/ZUqHAR3uTTZxg0BD5pby9I5ZUp+3PQnz+OEsxEXuv4Yq57zapAkWaVw/eMY7QNoByYRxh+XVDpqy72DAK0Q6vMrcj/hK4/ITMqVusaGVpYinsXPiibKaI25MkigH4W797ihb/dgGeRLZ0w1OGWpG1v7ES1GbByNLYiCcs+shTcVb5p0LpQmU7dINb9sBReYgr++iD8+IDgVJpDwCEygtYGr/haxHPPaWf5aUupXB5myzXw4u5JhBpLFttKDsMk2MFLcISaEcRqDW0Wjc1Npdm9DE+UsXkao5g6AX3eN3KwA2NhBM1RBvBLO4wyMmYRmcvi1BU24wTnFPbSutDTFmebTaME+VrbJjVf+AlgFWIyye6n1y/D4PSNuHFIdX7uUIU6KGj8UxbKpfrvcKRafowBQu+b938XXXto+P5pmSVt8zI3QcD5NhAjuvaR14LKZjwliFdAVtQk2J6jVXQvlby+Vy+YvrqoCsir6/wfwH/ITL+yCROuqSefYAJjfKV/8RCCqgnmzetXVOvBBm3DzA7s2ZYTQhDLMq750vUmu5QUXQF90TSSSdl5kgveMmO/E9Mjwcx9FaaY4jCM0TDg== MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BN7PR11MB2770.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9e2c9cf2-d9f7-478c-2ecd-08d8351edea1 X-MS-Exchange-CrossTenant-originalarrivaltime: 31 Jul 2020 06:56:37.1742 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: bMXepEAXIwzigQ89pWcsC2XNE2tcP++5lmcCU902LJAE+7TKRO8B397K18MxBiMd9Qfaznd0y+dRH/bMQ56vYw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR11MB3779 Return-Path: wei6.xu@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Wei6 Xu -----Original Message----- From: devel@edk2.groups.io On Behalf Of Michael Kub= acki Sent: Friday, July 31, 2020 11:15 AM To: devel@edk2.groups.io Cc: Gao, Liming ; Kinney, Michael D Subject: [edk2-devel] [PATCH v1 0/7] FmpDevicePkg Minor Fixes From: Michael Kubacki REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D2869 This patch series makes minor fixes and enhancements in FmpDevicePkg. None= of the issues impact functionality in a major way. These are smaller sets of changes that are being submitted prior to a larg= er future patch series that will cover functional changes for BZ2802. Please check the git notes in patch 3 and patch 5. Cc: Liming Gao Cc: Michael D Kinney Signed-off-by: Michael Kubacki Michael Kubacki (7): FmpDevicePkg/FmpDependencyLib: Correct ValidateDependency() documentation FmpDevicePkg/FmpDependencyLib: Fix "exression" typo FmpDevicePkg/FmpDependencyLib: Handle version string overflow FmpDevicePkg/FmpDependencyCheckLib: Return unsatisfied on handle failure FmpDevicePkg/FmpDxe: Better warn of potential ImageTypeId misconfig FmpDevicePkg/FmpDxe: Indicate ESRT GUID on invalid ImageIdName FmpDevicePkg/FmpDxe: Improve function parameter validation FmpDevicePkg/FmpDxe/FmpDxe.c | 62 += +++++++++++++++---- FmpDevicePkg/Library/FmpDependencyCheckLib/FmpDependencyCheckLib.c | 2 + FmpDevicePkg/Library/FmpDependencyLib/FmpDependencyLib.c | 8 += +- FmpDevicePkg/FmpDxe/FmpDxe.h | 10 += +-- FmpDevicePkg/Include/Library/FmpDependencyLib.h | 5 += - 5 files changed, 67 insertions(+), 20 deletions(-) -- 2.27.0.windows.1