From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.100; helo=mga07.intel.com; envelope-from=hao.a.wu@intel.com; receiver=edk2-devel@lists.01.org Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 07A7421183EDC for ; Sat, 27 Oct 2018 05:28:57 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Oct 2018 05:28:57 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.54,431,1534834800"; d="scan'208";a="91678534" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by FMSMGA003.fm.intel.com with ESMTP; 27 Oct 2018 05:28:57 -0700 Received: from fmsmsx155.amr.corp.intel.com (10.18.116.71) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.319.2; Sat, 27 Oct 2018 05:28:57 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by FMSMSX155.amr.corp.intel.com (10.18.116.71) with Microsoft SMTP Server (TLS) id 14.3.319.2; Sat, 27 Oct 2018 05:28:56 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.117]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.161]) with mapi id 14.03.0415.000; Sat, 27 Oct 2018 20:28:55 +0800 From: "Wu, Hao A" To: "Zeng, Star" , "edk2-devel@lists.01.org" CC: "Ni, Ruiyu" , Paulo Alcantara Thread-Topic: [edk2] [PATCH] MdeModulePkg/UdfDxe: Additional checks for ResolveSymlink() Thread-Index: AQHUbQE3knpy/m7DlUWCN87K+uccVqUxDCqAgAH64SA= Date: Sat, 27 Oct 2018 12:28:54 +0000 Message-ID: References: <20181026075457.6280-1-hao.a.wu@intel.com> <3d5d68de-0157-65c7-0fa4-b8c30bad5353@intel.com> In-Reply-To: <3d5d68de-0157-65c7-0fa4-b8c30bad5353@intel.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH] MdeModulePkg/UdfDxe: Additional checks for ResolveSymlink() X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 27 Oct 2018 12:28:58 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBaZW5nLCBTdGFyDQo+IFNlbnQ6 IEZyaWRheSwgT2N0b2JlciAyNiwgMjAxOCAxMDoxNCBQTQ0KPiBUbzogV3UsIEhhbyBBOyBlZGsy LWRldmVsQGxpc3RzLjAxLm9yZw0KPiBDYzogTmksIFJ1aXl1OyBQYXVsbyBBbGNhbnRhcmE7IFpl bmcsIFN0YXINCj4gU3ViamVjdDogUmU6IFtlZGsyXSBbUEFUQ0hdIE1kZU1vZHVsZVBrZy9VZGZE eGU6IEFkZGl0aW9uYWwgY2hlY2tzIGZvcg0KPiBSZXNvbHZlU3ltbGluaygpDQo+IA0KPiBIYW8s DQo+IA0KPiBPbiAyMDE4LzEwLzI2IDE1OjU0LCBIYW8gV3Ugd3JvdGU6DQo+ID4gUkVGOmh0dHBz Oi8vYnVnemlsbGEudGlhbm9jb3JlLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI3OQ0KPiA+DQo+ID4g VGhlIGNvbW1pdCB3aWxsIGFkZCAzIHR5cGVzIG9mIGNoZWNrcyBmb3IgZnVuY3Rpb24gUmVzb2x2 ZVN5bWxpbmsoKToNCj4gPg0KPiA+IEEuIENoZWNrIGZvciB0aGUgdmFsdWUgb2YgJ0NvbXBvbmVu dCBUeXBlJyBmaWVsZCB3aXRoaW4gYSBQYXRoIENvbXBvbmVudA0KPiA+DQo+ID4gQWNjb3JkaW5n IHRvIHRoZSBFQ01BLTE2NyBzdGFuZGFyZCAoM3JkIEVkaXRpb24gLSBKdW5lIDE5OTcpLCBTZWN0 aW9uDQo+ID4gMTQuMTYuMS4xLCB2YWxpZCB2YWx1ZXMgYXJlIDEgdG8gNS4gQWxsIG90aGVyIHZh bHVlcyB3aWxsIGJlIHRyZWF0ZWQgYXMgYQ0KPiA+IGNvcnJ1cHRlZCB2b2x1bWUuDQo+ID4NCj4g PiBCLiBDaGVjayBmb3IgdGhlIGNvbnRlbnQgcG9pbnRlZCBieSAnRmlsZScNCj4gPg0KPiA+IFNp bmNlIGNvbnRlbnQgd2l0aGluICdGaWxlJyBpcyB0aGUgb3V0cHV0IGRhdGEgZm9yIFJlc29sdmVT eW1saW5rKCkuDQo+ID4gQ2hlY2tzIGlzIGFkZGVkIHRvIGVuc3VyZSB0aGUgY29udGVudCBpbiAn RmlsZScgaXMgdmFsaWQuIE90aGVyd2lzZSwNCj4gPiBwb3NzaWJsZSBudWxsIHBvaW50ZXIgZGVy ZWZlcmVuY2UgaXNzdWUgd2lsbCBvY2N1ciBkdXJpbmcgdGhlIHN1YnNlcXVlbnQNCj4gPiB1c2Fn ZSBvZiB0aGUgZGF0YSByZXR1cm5lZCBieSBSZXNvbHZlU3ltbGluaygpLg0KPiA+DQo+ID4gQy4g Q2hlY2sgZm9yIHBvc3NpYmxlIG1lbW9yeSBkb3VibGUgZnJlZS91c2UgYWZ0ZXIgZnJlZSBjYXNl DQo+ID4NCj4gPiBGb3IgY29kZXM6DQo+ID4NCj4gPiAgICAgIGlmIChDb21wYXJlTWVtICgoVk9J RCAqKSZQcmV2aW91c0ZpbGUsIChWT0lEICopUGFyZW50LA0KPiA+ICAgICAgICAgICAgICAgICAg ICAgIHNpemVvZiAoVURGX0ZJTEVfSU5GTykpICE9IDApIHsNCj4gPiAgICAgICAgQ2xlYW51cEZp bGVJbmZvcm1hdGlvbiAoJlByZXZpb3VzRmlsZSk7DQo+ID4gICAgICB9DQo+ID4NCj4gPiAgICAg IENvcHlNZW0gKChWT0lEICopJlByZXZpb3VzRmlsZSwgKFZPSUQgKilGaWxlLCBzaXplb2YgKFVE Rl9GSUxFX0lORk8pKTsNCj4gPg0KPiA+IElmIHRoZSBjb250ZW50cyBpbiAnUHJldmlvdXNGaWxl JyBhbmQgJ0ZpbGUnIGFyZSB0aGUgc2FtZSwgY2FsbCB0bw0KPiA+ICJDbGVhbnVwRmlsZUluZm9y bWF0aW9uICgmUHJldmlvdXNGaWxlKTsiIHdpbGwgZnJlZSB0aGUgYnVmZmVycyBpbiAnRmlsZScN Cj4gPiBhcyB3ZWxsLiBUaGlzIHdpbGwgbGVhZCB0byBwb3RlbnRpYWwgbWVtb3J5IGRvdWJsZSBm cmVlL3VzZSBhZnRlciBmcmVlDQo+ID4gaXNzdWVzLg0KPiANCj4gSWYgJ1ByZXZpb3VzRmlsZScg YW5kICdGaWxlJyBhcmUgdGhlIHNhbWUsIHRoZSBjb3Bpbmcgb3BlcmF0aW9uIGJlbG93DQo+IGFs c28gaGFzIG5vIG5lZWQgdG8gYmUgZG9uZSwgcmlnaHQ/DQoNClllcy4NCkkgd2lsbCBzZW5kIGEg VjIgcGF0Y2ggdG8gYWRkcmVzcyB0aGlzLg0KDQpCZXN0IFJlZ2FyZHMsDQpIYW8gV3UNCg0KPiAN Cj4gQ29weU1lbSAoKFZPSUQgKikmUHJldmlvdXNGaWxlLCAoVk9JRCAqKUZpbGUsIHNpemVvZiAo VURGX0ZJTEVfSU5GTykpOw0KPiANCj4gVGhhbmtzLA0KPiBTdGFyDQo+IA0KPiA+DQo+ID4gQ2M6 IFBhdWxvIEFsY2FudGFyYSA8cGF1bG9AcGF1bG8uYWM+DQo+ID4gQ2M6IFBhdWxvIEFsY2FudGFy YSA8cGFsY2FudGFyYUBzdXNlLmRlPg0KPiA+IENjOiBSdWl5dSBOaSA8cnVpeXUubmlAaW50ZWwu Y29tPg0KPiA+IENjOiBTdGFyIFplbmcgPHN0YXIuemVuZ0BpbnRlbC5jb20+DQo+ID4gQ29udHJp YnV0ZWQtdW5kZXI6IFRpYW5vQ29yZSBDb250cmlidXRpb24gQWdyZWVtZW50IDEuMQ0KPiA+IFNp Z25lZC1vZmYtYnk6IEhhbyBXdSA8aGFvLmEud3VAaW50ZWwuY29tPg0KPiA+IC0tLQ0KPiA+ICAg Li4uL1VuaXZlcnNhbC9EaXNrL1VkZkR4ZS9GaWxlU3lzdGVtT3BlcmF0aW9ucy5jICAgfCAzMA0K PiArKysrKysrKysrKysrKysrKysrKy0tDQo+ID4gICAxIGZpbGUgY2hhbmdlZCwgMjggaW5zZXJ0 aW9ucygrKSwgMiBkZWxldGlvbnMoLSkNCj4gPg0KPiA+IGRpZmYgLS1naXQgYS9NZGVNb2R1bGVQ a2cvVW5pdmVyc2FsL0Rpc2svVWRmRHhlL0ZpbGVTeXN0ZW1PcGVyYXRpb25zLmMNCj4gYi9NZGVN b2R1bGVQa2cvVW5pdmVyc2FsL0Rpc2svVWRmRHhlL0ZpbGVTeXN0ZW1PcGVyYXRpb25zLmMNCj4g PiBpbmRleCBiOWViZGRmZTYyLi5hODllNWJhOWZmIDEwMDY0NA0KPiA+IC0tLSBhL01kZU1vZHVs ZVBrZy9Vbml2ZXJzYWwvRGlzay9VZGZEeGUvRmlsZVN5c3RlbU9wZXJhdGlvbnMuYw0KPiA+ICsr KyBiL01kZU1vZHVsZVBrZy9Vbml2ZXJzYWwvRGlzay9VZGZEeGUvRmlsZVN5c3RlbU9wZXJhdGlv bnMuYw0KPiA+IEBAIC0yMTQ1LDYgKzIxNDUsOCBAQCBSZXNvbHZlU3ltbGluayAoDQo+ID4gICAg IFVJTlQ4ICAgICAgICAgICAgICAgQ29tcHJlc3Npb25JZDsNCj4gPiAgICAgVURGX0ZJTEVfSU5G TyAgICAgICBQcmV2aW91c0ZpbGU7DQo+ID4NCj4gPiArICBaZXJvTWVtICgoVk9JRCAqKUZpbGUs IHNpemVvZiAoVURGX0ZJTEVfSU5GTykpOw0KPiA+ICsNCj4gPiAgICAgLy8NCj4gPiAgICAgLy8g U3ltbGluayBmaWxlcyBvbiBVREYgdm9sdW1lcyBkbyBub3QgY29udGFpbiBzbyBtdWNoIGRhdGEg b3RoZXIgdGhhbg0KPiA+ICAgICAvLyBQYXRoIENvbXBvbmVudHMgd2hpY2ggcmVzb2x2ZXMgdG8g cmVhbCBmaWxlbmFtZXMsIHNvIGl0J3MgT0sgdG8gcmVhZA0KPiBpbg0KPiA+IEBAIC0yMjU3LDYg KzIyNTksMTMgQEAgUmVzb2x2ZVN5bWxpbmsgKA0KPiA+ICAgICAgICAgfQ0KPiA+ICAgICAgICAg RmlsZU5hbWVbSW5kZXhdID0gTCdcMCc7DQo+ID4gICAgICAgICBicmVhazsNCj4gPiArICAgIGRl ZmF1bHQ6DQo+ID4gKyAgICAgIC8vDQo+ID4gKyAgICAgIC8vIEFjY29yaW5nIHRvIHRoZSBFQ01B LTE2NyBzdGFuZGFyZCAoM3JkIEVkaXRpb24gLSBKdW5lIDE5OTcpLA0KPiBTZWN0aW9uDQo+ID4g KyAgICAgIC8vIDE0LjE2LjEuMSwgYWxsIG90aGVyIHZhbHVlcyBhcmUgcmVzZXJ2ZWQuDQo+ID4g KyAgICAgIC8vDQo+ID4gKyAgICAgIFN0YXR1cyA9IEVGSV9WT0xVTUVfQ09SUlVQVEVEOw0KPiA+ ICsgICAgICBnb3RvIEVycm9yX0ZpbmRfRmlsZTsNCj4gPiAgICAgICB9DQo+ID4NCj4gPiAgICAg ICAvLw0KPiA+IEBAIC0yMjgxLDggKzIyOTAsMTggQEAgUmVzb2x2ZVN5bWxpbmsgKA0KPiA+ICAg ICAgICAgYnJlYWs7DQo+ID4gICAgICAgfQ0KPiA+DQo+ID4gLSAgICBpZiAoQ29tcGFyZU1lbSAo KFZPSUQgKikmUHJldmlvdXNGaWxlLCAoVk9JRCAqKVBhcmVudCwNCj4gPiAtICAgICAgICAgICAg ICAgICAgICBzaXplb2YgKFVERl9GSUxFX0lORk8pKSAhPSAwKSB7DQo+ID4gKyAgICAvLw0KPiA+ ICsgICAgLy8gQ2hlY2sgdGhlIGNvbnRlbnQgaW4gdGhlIGZpbGUgaW5mbyBwb2ludGVkIGJ5IEZp bGUuDQo+ID4gKyAgICAvLw0KPiA+ICsgICAgaWYgKChGaWxlLT5GaWxlRW50cnkgPT0gTlVMTCkg fHwgKEZpbGUtPkZpbGVJZGVudGlmaWVyRGVzYyA9PSBOVUxMKSkgew0KPiA+ICsgICAgICBTdGF0 dXMgPSBFRklfVk9MVU1FX0NPUlJVUFRFRDsNCj4gPiArICAgICAgZ290byBFcnJvcl9GaW5kX0Zp bGU7DQo+ID4gKyAgICB9DQo+ID4gKw0KPiA+ICsgICAgaWYgKChDb21wYXJlTWVtICgoVk9JRCAq KSZQcmV2aW91c0ZpbGUsIChWT0lEICopUGFyZW50LA0KPiA+ICsgICAgICAgICAgICAgICAgICAg IHNpemVvZiAoVURGX0ZJTEVfSU5GTykpICE9IDApICYmDQo+ID4gKyAgICAgICAgKENvbXBhcmVN ZW0gKChWT0lEICopJlByZXZpb3VzRmlsZSwgKFZPSUQgKilGaWxlLA0KPiA+ICsgICAgICAgICAg ICAgICAgICAgIHNpemVvZiAoVURGX0ZJTEVfSU5GTykpICE9IDApKSB7DQo+ID4gICAgICAgICBD bGVhbnVwRmlsZUluZm9ybWF0aW9uICgmUHJldmlvdXNGaWxlKTsNCj4gPiAgICAgICB9DQo+ID4N Cj4gPiBAQCAtMjI5NCw2ICsyMzEzLDEzIEBAIFJlc29sdmVTeW1saW5rICgNCj4gPiAgICAgLy8N Cj4gPiAgICAgRnJlZVBvb2wgKFJlYWRGaWxlSW5mby5GaWxlRGF0YSk7DQo+ID4NCj4gPiArICAv Lw0KPiA+ICsgIC8vIENoZWNrIHRoZSBjb250ZW50IGluIHRoZSByZXNvbHZlZCBmaWxlIGluZm8u DQo+ID4gKyAgLy8NCj4gPiArICBpZiAoKEZpbGUtPkZpbGVFbnRyeSA9PSBOVUxMKSB8fCAoRmls ZS0+RmlsZUlkZW50aWZpZXJEZXNjID09IE5VTEwpKSB7DQo+ID4gKyAgICByZXR1cm4gRUZJX1ZP TFVNRV9DT1JSVVBURUQ7DQo+ID4gKyAgfQ0KPiA+ICsNCj4gPiAgICAgcmV0dXJuIEVGSV9TVUND RVNTOw0KPiA+DQo+ID4gICBFcnJvcl9GaW5kX0ZpbGU6DQo+ID4NCg0K