From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) (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 A259321D492C5 for ; Thu, 14 Sep 2017 23:11:41 -0700 (PDT) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 14 Sep 2017 23:14:41 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.42,396,1500966000"; d="scan'208";a="128950127" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga004.jf.intel.com with ESMTP; 14 Sep 2017 23:14:40 -0700 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.319.2; Thu, 14 Sep 2017 23:14:40 -0700 Received: from shsmsx151.ccr.corp.intel.com ([169.254.3.98]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.22]) with mapi id 14.03.0319.002; Fri, 15 Sep 2017 14:14:38 +0800 From: "Zeng, Star" To: Paulo Alcantara , "Laszlo Ersek (lersek@redhat.com)" , "Ni, Ruiyu" , "Yao, Jiewen" CC: "Dong, Eric" , "Bi, Dandan" , "edk2-devel@lists.01.org" , "Kinney, Michael D" , "Zeng, Star" Thread-Topic: [PATCH] MdeModulePkg/UdfDxe: Remove negative comparison of unsigned number Thread-Index: AQHTLEsk9oaBZ5JdJkSAcz7I8VLM2KKyQ1ew//+FdgCAA6lMkA== Date: Fri, 15 Sep 2017 06:14:38 +0000 Message-ID: <0C09AFA07DD0434D9E2A0C6AEB0483103B95A36E@SHSMSX151.ccr.corp.intel.com> References: <8c5a7ec1d6f908eabda755bb6d3bc9a28b14210e.1505277490.git.pcacjr@zytor.com> <0C09AFA07DD0434D9E2A0C6AEB0483103B940874@shsmsx102.ccr.corp.intel.com> <37D67821-E356-4E48-B175-3107DEF070AC@zytor.com> In-Reply-To: <37D67821-E356-4E48-B175-3107DEF070AC@zytor.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: Remove negative comparison of unsigned number X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 Sep 2017 06:11:41 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 SGksDQoNCkJhc2VkIG9uIHJlY2VudCBpc3N1ZXMgYWJvdXQgVURGIHNpbmNlIHRoZSBjb2RlIHdh cyBwdXNoZWQgZm9yIGh0dHBzOi8vbGlzdHMuMDEub3JnL3BpcGVybWFpbC9lZGsyLWRldmVsLzIw MTctU2VwdGVtYmVyLzAxNDM2MC5odG1sLCBJIHdhbnQgdG8gcmFpc2Ugc29tZSBxdWVzdGlvbnMg a2luZGx5Lg0KDQpodHRwczovL2xpc3RzLjAxLm9yZy9waXBlcm1haWwvZWRrMi1kZXZlbC8yMDE3 LVNlcHRlbWJlci8wMTQ0MDkuaHRtbA0KaHR0cHM6Ly9saXN0cy4wMS5vcmcvcGlwZXJtYWlsL2Vk azItZGV2ZWwvMjAxNy1TZXB0ZW1iZXIvMDE0NTE4Lmh0bWwNCmh0dHBzOi8vbGlzdHMuMDEub3Jn L3BpcGVybWFpbC9lZGsyLWRldmVsLzIwMTctU2VwdGVtYmVyLzAxNDU0Mi5odG1sDQpodHRwczov L2xpc3RzLjAxLm9yZy9waXBlcm1haWwvZWRrMi1kZXZlbC8yMDE3LVNlcHRlbWJlci8wMTQ0ODku aHRtbA0KaHR0cHM6Ly9saXN0cy4wMS5vcmcvcGlwZXJtYWlsL2VkazItZGV2ZWwvMjAxNy1TZXB0 ZW1iZXIvMDE0NTUxLmh0bWwNCmh0dHBzOi8vbGlzdHMuMDEub3JnL3BpcGVybWFpbC9lZGsyLWRl dmVsLzIwMTctU2VwdGVtYmVyLzAxNDU2MC5odG1sDQpodHRwczovL2xpc3RzLjAxLm9yZy9waXBl cm1haWwvZWRrMi1kZXZlbC8yMDE3LVNlcHRlbWJlci8wMTQ2NDkuaHRtbA0KaHR0cHM6Ly9saXN0 cy4wMS5vcmcvcGlwZXJtYWlsL2VkazItZGV2ZWwvMjAxNy1TZXB0ZW1iZXIvMDE0Njk0Lmh0bWwN Cmh0dHBzOi8vbGlzdHMuMDEub3JnL3BpcGVybWFpbC9lZGsyLWRldmVsLzIwMTctU2VwdGVtYmVy LzAxNDY5NS5odG1sDQoNCklzIHRoZSBjb2RlIGV4cGVjdGVkIHRvIGJlIGdvdCB1cHN0cmVhbSBv cmlnaW5hbGx5PyAoSSBtYXkgYmUgYSBzdHVwaWQgcXVlc3Rpb24gc2luY2UgdGhlIGNvZGUgaGFz IGJlZW4gZ290dGVuIHVwc3RyZWFtLCBJIGp1c3Qgd2FudCB0byBkb3VibGUgY29uZmlybSB0aGF0 IGFzIFBhdWxvJ3MgcmVwbHkgYmVsb3cgIkkgYmVsaWV2ZWQgdGhhdCBpdCB3b3VsZCBuZXZlciBn ZXQgdXBzdHJlYW0iLikNCklzIHRoZSBjb2RlIHJlYWR5IHRvIGJlIGluIG1hc3Rlcj8gU2hvdWxk IGl0IGJlIGluIFN0YWdpbmcgYnJhbmNoIGZpcnN0Pw0KDQoNClBhdWxvLA0KQ291bGQgeW91IGhl bHAgZG8gbW9yZSBldmFsdWF0aW9uIHRvIHRoZSBjb2RlIGFzIHlvdSBzYWlkICJJICpkbyoga25v dyB0aGF0IHRoZSBjb2RlIHJlYWxseSBuZWVkcyByZWZhY3RvcmluZywgZG9jdW1lbnRhdGlvbiwg ZXRjIj8gSSBiZWxpZXZlIHlvdSBhcmUgbW9zdCBmYW1pbGlhciB3aXRoIHRoZSBjb2RlIGFuZCBr bm93IGl0cyBxdWFsaXR5LiA6KQ0KDQoNCkJUVzogTW9yZSB0ZXN0IHNlZW1zIG5lZWQgdG8gYmUg ZG9uZSBiZWZvcmUgdGhlIGNvZGUgY2hlY2sgaW4sIGZvciBleGFtcGxlLCBidWlsZCB3aXRoIHZh cmlvdXMgdG9vbCBjaGFpbnMsIEVDQyBzY2FuIGZvciBjb2Rpbmcgc3R5bGUsIHN0YXRpYyB0b29s IHNjYW4sIGV0Yy4gVGhhdCBpcyB3aGF0IHdlKGVzcGVjaWFsbHkgbWUpIG5lZWQgdG8gaW1wcm92 ZSBpbiBmdXR1cmUgd2hlbiBkZXZlbG9waW5nIGFuZCByZXZpZXdpbmcuDQpBbnl3YXksIGxldCdz IGhlbHAga2VlcCBpbXByb3ZpbmcgVURGIGNvZGVzLg0KDQoNCg0KVGhhbmtzLA0KU3Rhcg0KLS0t LS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCkZyb206IFBhdWxvIEFsY2FudGFyYSBbbWFpbHRvOnBj YWNqckB6eXRvci5jb21dIA0KU2VudDogV2VkbmVzZGF5LCBTZXB0ZW1iZXIgMTMsIDIwMTcgMTo0 NyBQTQ0KVG86IFplbmcsIFN0YXIgPHN0YXIuemVuZ0BpbnRlbC5jb20+OyBlZGsyLWRldmVsQGxp c3RzLjAxLm9yZw0KQ2M6IERvbmcsIEVyaWMgPGVyaWMuZG9uZ0BpbnRlbC5jb20+OyBOaSwgUnVp eXUgPHJ1aXl1Lm5pQGludGVsLmNvbT47IEJpLCBEYW5kYW4gPGRhbmRhbi5iaUBpbnRlbC5jb20+ DQpTdWJqZWN0OiBSRTogW1BBVENIXSBNZGVNb2R1bGVQa2cvVWRmRHhlOiBSZW1vdmUgbmVnYXRp dmUgY29tcGFyaXNvbiBvZiB1bnNpZ25lZCBudW1iZXINCg0KDQoNCk9uIFNlcHRlbWJlciAxMywg MjAxNyAyOjA4OjU0IEFNIEdNVC0wMzowMCwgIlplbmcsIFN0YXIiIDxzdGFyLnplbmdAaW50ZWwu Y29tPiB3cm90ZToNCj5JIGRvIG5vdCB1bmRlcnN0YW5kIHRoZSBjb250ZXh0IG9mIHRoZSBjb2Rl Lg0KPlRoZSBjaGFuZ2UgaXMgZ29vZCB0byBmaXggdGhlIGJ1aWxkIGZhaWx1cmUsIGJ1dCBJIHdh bnQgdG8gYXNrIGEgDQo+cXVlc3Rpb24gYmVmb3JlIEkgZ2F2ZSBSYi4gOikNCj4NCj5JcyBpdCBw b3NzaWJsZSBSZWFkRmlsZUluZm8tPkZpbGVQb3NpdGlvbiBsZXNzIHRoYW4gRmlsZVBvc2l0aW9u Pw0KDQpOb3BlLiBXaGVuIGRvaW5nIG15IHRlc3RzLCBJIGJyaWVmbHkgbG9va2VkIGF0IGNvZGUg aG93IGl0J3MgdXNlZCBhbmQgYWxzbyBhZGRlZCBhbiBBU1NFUlQoKSB0byBtYWtlIHN1cmUgaXQg aXMgbmV2ZXIgbGVzc2VyIHRoYW4gRmlsZVBvc2l0aW9uLg0KDQpCVFcsIEkgKmRvKiBrbm93IHRo YXQgdGhlIGNvZGUgcmVhbGx5IG5lZWRzIHJlZmFjdG9yaW5nLCBkb2N1bWVudGF0aW9uLCBldGMu IC0tICBJIGRpZG50IGRvIHRoYXQgYmVmb3JlIGJlY2F1c2UgSSBiZWxpZXZlZCB0aGF0IGl0IHdv dWxkIG5ldmVyIGdldCB1cHN0cmVhbSAtLSBzaW5jZSBpdHMgbm93IC0tIEkgd2lsbCBsb29rIGZv cndhcmQgdG8gdGhhdCBpbiBteSBmcmVlIHRpbWUuDQoNCkl0cyAyOjQ2YW0gaGVyZSBzbyBJIHNo b3VsZCBnZXQgc29tZSBzbGVlcCA6LSkgVGhhbmtzIQ0KDQpQYXVsbw0KDQo+DQo+DQo+VGhhbmtz LA0KPlN0YXINCj4tLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPkZyb206IFBhdWxvIEFsY2Fu dGFyYSBbbWFpbHRvOnBjYWNqckB6eXRvci5jb21dDQo+U2VudDogV2VkbmVzZGF5LCBTZXB0ZW1i ZXIgMTMsIDIwMTcgMTI6NDUgUE0NCj5UbzogZWRrMi1kZXZlbEBsaXN0cy4wMS5vcmcNCj5DYzog UGF1bG8gQWxjYW50YXJhIDxwY2FjanJAenl0b3IuY29tPjsgWmVuZywgU3RhciANCj48c3Rhci56 ZW5nQGludGVsLmNvbT47IERvbmcsIEVyaWMgPGVyaWMuZG9uZ0BpbnRlbC5jb20+OyBOaSwgUnVp eXUgDQo+PHJ1aXl1Lm5pQGludGVsLmNvbT47IEJpLCBEYW5kYW4gPGRhbmRhbi5iaUBpbnRlbC5j b20+DQo+U3ViamVjdDogW1BBVENIXSBNZGVNb2R1bGVQa2cvVWRmRHhlOiBSZW1vdmUgbmVnYXRp dmUgY29tcGFyaXNvbiBvZiANCj51bnNpZ25lZCBudW1iZXINCj4NCj5UaGlzIHBhdGNoIGdldHMg cmlkIG9mIGEgbmVnYXRpdmUgY29tcGFyaXNvbiBvZiBhbiBVSU5UNjQgdHlwZSAoT2Zmc2V0KSAN Cj5hcyBpdCdsbCBuZXZlciBldmFsdWF0ZSB0byB0cnVlLg0KPg0KPkNjOiBTdGFyIFplbmcgPHN0 YXIuemVuZ0BpbnRlbC5jb20+DQo+Q2M6IEVyaWMgRG9uZyA8ZXJpYy5kb25nQGludGVsLmNvbT4N Cj5DYzogUnVpeXUgTmkgPHJ1aXl1Lm5pQGludGVsLmNvbT4NCj5DYzogRGFuZGFuIEJpIDxkYW5k YW4uYmlAaW50ZWwuY29tPg0KPkNvbnRyaWJ1dGVkLXVuZGVyOiBUaWFub0NvcmUgQ29udHJpYnV0 aW9uIEFncmVlbWVudCAxLjENCj5SZXBvcnRlZC1ieTogU3RhciBaZW5nIDxzdGFyLnplbmdAaW50 ZWwuY29tPg0KPlNpZ25lZC1vZmYtYnk6IFBhdWxvIEFsY2FudGFyYSA8cGNhY2pyQHp5dG9yLmNv bT4NCj4tLS0NCj4gTWRlTW9kdWxlUGtnL1VuaXZlcnNhbC9EaXNrL1VkZkR4ZS9GaWxlU3lzdGVt T3BlcmF0aW9ucy5jIHwgMyAtLS0NCj4gMSBmaWxlIGNoYW5nZWQsIDMgZGVsZXRpb25zKC0pDQo+ DQo+ZGlmZiAtLWdpdCBhL01kZU1vZHVsZVBrZy9Vbml2ZXJzYWwvRGlzay9VZGZEeGUvRmlsZVN5 c3RlbU9wZXJhdGlvbnMuYw0KPmIvTWRlTW9kdWxlUGtnL1VuaXZlcnNhbC9EaXNrL1VkZkR4ZS9G aWxlU3lzdGVtT3BlcmF0aW9ucy5jDQo+aW5kZXggNzI4NjI2NTM3My4uMjAzOWY4MDI4OSAxMDA2 NDQNCj4tLS0gYS9NZGVNb2R1bGVQa2cvVW5pdmVyc2FsL0Rpc2svVWRmRHhlL0ZpbGVTeXN0ZW1P cGVyYXRpb25zLmMNCj4rKysgYi9NZGVNb2R1bGVQa2cvVW5pdmVyc2FsL0Rpc2svVWRmRHhlL0Zp bGVTeXN0ZW1PcGVyYXRpb25zLmMNCj5AQCAtMTA4Miw5ICsxMDgyLDYgQEAgUmVhZEZpbGUgKA0K PiANCj4gICAgICAgIGlmIChGaWxlUG9zaXRpb24gKyBFeHRlbnRMZW5ndGggPiBSZWFkRmlsZUlu Zm8tPkZpbGVQb3NpdGlvbikgew0KPiAgICAgICAgICAgT2Zmc2V0ID0gUmVhZEZpbGVJbmZvLT5G aWxlUG9zaXRpb24gLSBGaWxlUG9zaXRpb247DQo+LSAgICAgICAgICBpZiAoT2Zmc2V0IDwgMCkg ew0KPi0gICAgICAgICAgICBPZmZzZXQgPSAtKE9mZnNldCk7DQo+LSAgICAgICAgICB9DQo+ICAg ICAgICAgfSBlbHNlIHsNCj4gICAgICAgICAgIE9mZnNldCA9IDA7DQo+ICAgICAgICAgfQ0KPi0t DQo+Mi4xMS4wDQoNCi0tDQpTZW50IGZyb20gbXkgQW5kcm9pZCBkZXZpY2Ugd2l0aCBLLTkgTWFp bC4gUGxlYXNlIGV4Y3VzZSBteSBicmV2aXR5Lg0K