From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: 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 B445A21E7493A for ; Fri, 15 Sep 2017 07:28:15 -0700 (PDT) Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga105.jf.intel.com with ESMTP; 15 Sep 2017 07:31:15 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.42,397,1500966000"; d="scan'208";a="151746587" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by fmsmga005.fm.intel.com with ESMTP; 15 Sep 2017 07:31:15 -0700 Received: from fmsmsx124.amr.corp.intel.com (10.18.125.39) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.319.2; Fri, 15 Sep 2017 07:31:15 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by fmsmsx124.amr.corp.intel.com (10.18.125.39) with Microsoft SMTP Server (TLS) id 14.3.319.2; Fri, 15 Sep 2017 07:31:14 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.175]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.93]) with mapi id 14.03.0319.002; Fri, 15 Sep 2017 22:31:12 +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//+FdgCAA6lMkP//jH2AgAEF9aA= Date: Fri, 15 Sep 2017 14:31:12 +0000 Message-ID: <0C09AFA07DD0434D9E2A0C6AEB0483103B965B08@shsmsx102.ccr.corp.intel.com> References: <8c5a7ec1d6f908eabda755bb6d3bc9a28b14210e.1505277490.git.pcacjr@zytor.com> <0C09AFA07DD0434D9E2A0C6AEB0483103B940874@shsmsx102.ccr.corp.intel.com> <37D67821-E356-4E48-B175-3107DEF070AC@zytor.com> <0C09AFA07DD0434D9E2A0C6AEB0483103B95A36E@SHSMSX151.ccr.corp.intel.com> <3DF79BE6-4CC0-4213-8A33-504C741288F4@zytor.com> In-Reply-To: <3DF79BE6-4CC0-4213-8A33-504C741288F4@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 14:28:15 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 UmVhbGx5IHRoYW5rcyBmb3IgdGhlIGNvbnRyaWJ1dGlvbiAoYnkgdXNpbmcgeW91ciBmcmVlIHRp bWUpLiA6KQ0KDQpQbGVhc2UgaGVscCB0YWtlIGNhcmUgdGhlIGlzc3VlIHJhaXNlZCBhdCBodHRw czovL2xpc3RzLjAxLm9yZy9waXBlcm1haWwvZWRrMi1kZXZlbC8yMDE3LVNlcHRlbWJlci8wMTQ2 OTQuaHRtbCBhbmQgaHR0cHM6Ly9idWd6aWxsYS50aWFub2NvcmUub3JnL3Nob3dfYnVnLmNnaT9p ZD03MDcgd2l0aCBoaWdoIHByaW9yaXR5LA0KYW5kIGFsc28gdGhlIHJldmlldyByZXF1ZXN0IGF0 IGh0dHBzOi8vbGlzdHMuMDEub3JnL3BpcGVybWFpbC9lZGsyLWRldmVsLzIwMTctU2VwdGVtYmVy LzAxNDY5NS5odG1sLg0KDQoNClRoYW5rcywNClN0YXINCi0tLS0tT3JpZ2luYWwgTWVzc2FnZS0t LS0tDQpGcm9tOiBQYXVsbyBBbGNhbnRhcmEgW21haWx0bzpwY2FjanJAenl0b3IuY29tXSANClNl bnQ6IEZyaWRheSwgU2VwdGVtYmVyIDE1LCAyMDE3IDI6NDkgUE0NClRvOiBaZW5nLCBTdGFyIDxz dGFyLnplbmdAaW50ZWwuY29tPjsgTGFzemxvIEVyc2VrIChsZXJzZWtAcmVkaGF0LmNvbSkgPGxl cnNla0ByZWRoYXQuY29tPjsgTmksIFJ1aXl1IDxydWl5dS5uaUBpbnRlbC5jb20+OyBZYW8sIEpp ZXdlbiA8amlld2VuLnlhb0BpbnRlbC5jb20+DQpDYzogRG9uZywgRXJpYyA8ZXJpYy5kb25nQGlu dGVsLmNvbT47IEJpLCBEYW5kYW4gPGRhbmRhbi5iaUBpbnRlbC5jb20+OyBlZGsyLWRldmVsQGxp c3RzLjAxLm9yZzsgS2lubmV5LCBNaWNoYWVsIEQgPG1pY2hhZWwuZC5raW5uZXlAaW50ZWwuY29t Pg0KU3ViamVjdDogUkU6IFtQQVRDSF0gTWRlTW9kdWxlUGtnL1VkZkR4ZTogUmVtb3ZlIG5lZ2F0 aXZlIGNvbXBhcmlzb24gb2YgdW5zaWduZWQgbnVtYmVyDQoNCg0KDQpTdGFyLA0KDQpPbiBTZXB0 ZW1iZXIgMTUsIDIwMTcgMzoxNDozOCBBTSBHTVQtMDM6MDAsICJaZW5nLCBTdGFyIiA8c3Rhci56 ZW5nQGludGVsLmNvbT4gd3JvdGU6DQo+SGksDQo+DQo+QmFzZWQgb24gcmVjZW50IGlzc3VlcyBh Ym91dCBVREYgc2luY2UgdGhlIGNvZGUgd2FzIHB1c2hlZCBmb3IgDQo+aHR0cHM6Ly9saXN0cy4w MS5vcmcvcGlwZXJtYWlsL2VkazItZGV2ZWwvMjAxNy1TZXB0ZW1iZXIvMDE0MzYwLmh0bWwsIEkg DQo+d2FudCB0byByYWlzZSBzb21lIHF1ZXN0aW9ucyBraW5kbHkuDQo+DQo+aHR0cHM6Ly9saXN0 cy4wMS5vcmcvcGlwZXJtYWlsL2VkazItZGV2ZWwvMjAxNy1TZXB0ZW1iZXIvMDE0NDA5Lmh0bWwN Cj5odHRwczovL2xpc3RzLjAxLm9yZy9waXBlcm1haWwvZWRrMi1kZXZlbC8yMDE3LVNlcHRlbWJl ci8wMTQ1MTguaHRtbA0KPmh0dHBzOi8vbGlzdHMuMDEub3JnL3BpcGVybWFpbC9lZGsyLWRldmVs LzIwMTctU2VwdGVtYmVyLzAxNDU0Mi5odG1sDQo+aHR0cHM6Ly9saXN0cy4wMS5vcmcvcGlwZXJt YWlsL2VkazItZGV2ZWwvMjAxNy1TZXB0ZW1iZXIvMDE0NDg5Lmh0bWwNCj5odHRwczovL2xpc3Rz LjAxLm9yZy9waXBlcm1haWwvZWRrMi1kZXZlbC8yMDE3LVNlcHRlbWJlci8wMTQ1NTEuaHRtbA0K Pmh0dHBzOi8vbGlzdHMuMDEub3JnL3BpcGVybWFpbC9lZGsyLWRldmVsLzIwMTctU2VwdGVtYmVy LzAxNDU2MC5odG1sDQo+aHR0cHM6Ly9saXN0cy4wMS5vcmcvcGlwZXJtYWlsL2VkazItZGV2ZWwv MjAxNy1TZXB0ZW1iZXIvMDE0NjQ5Lmh0bWwNCj5odHRwczovL2xpc3RzLjAxLm9yZy9waXBlcm1h aWwvZWRrMi1kZXZlbC8yMDE3LVNlcHRlbWJlci8wMTQ2OTQuaHRtbA0KPmh0dHBzOi8vbGlzdHMu MDEub3JnL3BpcGVybWFpbC9lZGsyLWRldmVsLzIwMTctU2VwdGVtYmVyLzAxNDY5NS5odG1sDQo+ DQo+SXMgdGhlIGNvZGUgZXhwZWN0ZWQgdG8gYmUgZ290IHVwc3RyZWFtIG9yaWdpbmFsbHk/IChJ IG1heSBiZSBhIHN0dXBpZCANCj5xdWVzdGlvbiBzaW5jZSB0aGUgY29kZSBoYXMgYmVlbiBnb3R0 ZW4gdXBzdHJlYW0sIEkganVzdCB3YW50IHRvIGRvdWJsZSANCj5jb25maXJtIHRoYXQgYXMgUGF1 bG8ncyByZXBseSBiZWxvdyAiSSBiZWxpZXZlZCB0aGF0IGl0IHdvdWxkIG5ldmVyIGdldA0KPnVw c3RyZWFtIi4pDQo+SXMgdGhlIGNvZGUgcmVhZHkgdG8gYmUgaW4gbWFzdGVyPyBTaG91bGQgaXQg YmUgaW4gU3RhZ2luZyBicmFuY2ggDQo+Zmlyc3Q/DQoNClNvcnJ5IGlmIEkgd2Fzbid0IGNsZWFy IGVub3VnaCwgYnV0IHdoZW4gSSBzZW50IFVERiBzdXBwb3J0IHRvIE1MIGluIH4zIHllYXJzIGFn bywgbm9ib2R5IHNlZW1lZCB0byBjYXJlIGFuZCBoYWQgbm8gZmVlZGJhY2sgc2luY2UgdGhlbiwg c28gSSB0aG91Z2h0IGl0IHdvdWxkIG5ldmVyIGdldCBhY2NlcHRlZCBhbmQgdGhlbiBJIGdhdmUg dXAgb24gdHJ5aW5nIHRvIGltcHJvdmUsIGNsZWFudXAsIGV0Yy4gLS0gSSBldmVuIHN0YXJ0ZWQg d29ya2luZyB3cml0ZSBzdXBwb3J0IGZvciBpdCBidXQgYWxzbyBnYXZlIHVwLiBJdCB3b3VsZCBu b3QgbWFrZSBzZW5zZSB0byB3b3JrIG9uIGl0IGFueW1vcmUuIFJpZ2h0Pw0KDQpJIHdvdWxkIG5v dCBrbm93IHRvIHRlbGwgeW91IGluIHdoaWNoIGJyYW5jaCBpdCBzaG91bGQgZ28sIGJ1dCBhcyB0 aGUgZHJpdmVyIGdpdmVzIHlvdSB0aGUgYWJpbGl0eSB0byBhY2Nlc3MgYW4gVURGIGZzIGFzIGl0 J3Mgc3VwcG9zZWQgdG8sIHRoZW4sIElNSE8sIGl0J2QgYmUgcmVhZHkgdG8gZ28gZWl0aGVyIHdh eS4gDQoNCj4NCj4NCj5QYXVsbywNCj5Db3VsZCB5b3UgaGVscCBkbyBtb3JlIGV2YWx1YXRpb24g dG8gdGhlIGNvZGUgYXMgeW91IHNhaWQgIkkgKmRvKiBrbm93IA0KPnRoYXQgdGhlIGNvZGUgcmVh bGx5IG5lZWRzIHJlZmFjdG9yaW5nLCBkb2N1bWVudGF0aW9uLCBldGMiPyBJIGJlbGlldmUgDQo+ eW91IGFyZSBtb3N0IGZhbWlsaWFyIHdpdGggdGhlIGNvZGUgYW5kIGtub3cgaXRzIHF1YWxpdHku IDopDQo+DQo+DQo+QlRXOiBNb3JlIHRlc3Qgc2VlbXMgbmVlZCB0byBiZSBkb25lIGJlZm9yZSB0 aGUgY29kZSBjaGVjayBpbiwgZm9yIA0KPmV4YW1wbGUsIGJ1aWxkIHdpdGggdmFyaW91cyB0b29s IGNoYWlucywgRUNDIHNjYW4gZm9yIGNvZGluZyBzdHlsZSwgDQo+c3RhdGljIHRvb2wgc2Nhbiwg ZXRjLiBUaGF0IGlzIHdoYXQgd2UoZXNwZWNpYWxseSBtZSkgbmVlZCB0byBpbXByb3ZlIA0KPmlu IGZ1dHVyZSB3aGVuIGRldmVsb3BpbmcgYW5kIHJldmlld2luZy4NCj5Bbnl3YXksIGxldCdzIGhl bHAga2VlcCBpbXByb3ZpbmcgVURGIGNvZGVzLg0KDQpZZXAuIFdlIHNob3VsZCByZWZhY3RvciB0 aGUgY29kZSwgZml4IGNvZGluZyBzdHlsZSBpc3N1ZXMsIGltcHJvdmUgZG9jdW1ldGF0aW9uLiBT aW5jZSBJIHdhcyB0aGUgb25seSBvbmUgd2hvIHRlc3RlZCBpdCwgaXQgZGVmaW5pdGVseSBuZWVk cyBhIGxpdHRsZSBtb3JlIHRlc3RpbmcuDQoNCldpdGggeW91IGhlbHAsIHdlIGRlZmluaXRlbHkg Y2FuIHJlc29sdmUgYWxsIHRob3NlIFRPRE8gaXRlbXMuDQoNClRoYW5rcyENClBhdWxvDQoNCj4N Cj4NCj4NCj5UaGFua3MsDQo+U3Rhcg0KPi0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0tDQo+RnJv bTogUGF1bG8gQWxjYW50YXJhIFttYWlsdG86cGNhY2pyQHp5dG9yLmNvbV0NCj5TZW50OiBXZWRu ZXNkYXksIFNlcHRlbWJlciAxMywgMjAxNyAxOjQ3IFBNDQo+VG86IFplbmcsIFN0YXIgPHN0YXIu emVuZ0BpbnRlbC5jb20+OyBlZGsyLWRldmVsQGxpc3RzLjAxLm9yZw0KPkNjOiBEb25nLCBFcmlj IDxlcmljLmRvbmdAaW50ZWwuY29tPjsgTmksIFJ1aXl1IDxydWl5dS5uaUBpbnRlbC5jb20+OyAN Cj5CaSwgRGFuZGFuIDxkYW5kYW4uYmlAaW50ZWwuY29tPg0KPlN1YmplY3Q6IFJFOiBbUEFUQ0hd IE1kZU1vZHVsZVBrZy9VZGZEeGU6IFJlbW92ZSBuZWdhdGl2ZSBjb21wYXJpc29uIG9mIA0KPnVu c2lnbmVkIG51bWJlcg0KPg0KPg0KPg0KPk9uIFNlcHRlbWJlciAxMywgMjAxNyAyOjA4OjU0IEFN IEdNVC0wMzowMCwgIlplbmcsIFN0YXIiDQo+PHN0YXIuemVuZ0BpbnRlbC5jb20+IHdyb3RlOg0K Pj5JIGRvIG5vdCB1bmRlcnN0YW5kIHRoZSBjb250ZXh0IG9mIHRoZSBjb2RlLg0KPj5UaGUgY2hh bmdlIGlzIGdvb2QgdG8gZml4IHRoZSBidWlsZCBmYWlsdXJlLCBidXQgSSB3YW50IHRvIGFzayBh IA0KPj5xdWVzdGlvbiBiZWZvcmUgSSBnYXZlIFJiLiA6KQ0KPj4NCj4+SXMgaXQgcG9zc2libGUg UmVhZEZpbGVJbmZvLT5GaWxlUG9zaXRpb24gbGVzcyB0aGFuIEZpbGVQb3NpdGlvbj8NCj4NCj5O b3BlLiBXaGVuIGRvaW5nIG15IHRlc3RzLCBJIGJyaWVmbHkgbG9va2VkIGF0IGNvZGUgaG93IGl0 J3MgdXNlZCBhbmQgDQo+YWxzbyBhZGRlZCBhbiBBU1NFUlQoKSB0byBtYWtlIHN1cmUgaXQgaXMg bmV2ZXIgbGVzc2VyIHRoYW4gDQo+RmlsZVBvc2l0aW9uLg0KPg0KPkJUVywgSSAqZG8qIGtub3cg dGhhdCB0aGUgY29kZSByZWFsbHkgbmVlZHMgcmVmYWN0b3JpbmcsIGRvY3VtZW50YXRpb24sIA0K PmV0Yy4gLS0gIEkgZGlkbnQgZG8gdGhhdCBiZWZvcmUgYmVjYXVzZSBJIGJlbGlldmVkIHRoYXQg aXQgd291bGQgbmV2ZXIgDQo+Z2V0IHVwc3RyZWFtIC0tIHNpbmNlIGl0cyBub3cgLS0gSSB3aWxs IGxvb2sgZm9yd2FyZCB0byB0aGF0IGluIG15IGZyZWUgDQo+dGltZS4NCj4NCj5JdHMgMjo0NmFt IGhlcmUgc28gSSBzaG91bGQgZ2V0IHNvbWUgc2xlZXAgOi0pIFRoYW5rcyENCj4NCj5QYXVsbw0K Pg0KPj4NCj4+DQo+PlRoYW5rcywNCj4+U3Rhcg0KPj4tLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0t LQ0KPj5Gcm9tOiBQYXVsbyBBbGNhbnRhcmEgW21haWx0bzpwY2FjanJAenl0b3IuY29tXQ0KPj5T ZW50OiBXZWRuZXNkYXksIFNlcHRlbWJlciAxMywgMjAxNyAxMjo0NSBQTQ0KPj5UbzogZWRrMi1k ZXZlbEBsaXN0cy4wMS5vcmcNCj4+Q2M6IFBhdWxvIEFsY2FudGFyYSA8cGNhY2pyQHp5dG9yLmNv bT47IFplbmcsIFN0YXIgDQo+PjxzdGFyLnplbmdAaW50ZWwuY29tPjsgRG9uZywgRXJpYyA8ZXJp Yy5kb25nQGludGVsLmNvbT47IE5pLCBSdWl5dSANCj4+PHJ1aXl1Lm5pQGludGVsLmNvbT47IEJp LCBEYW5kYW4gPGRhbmRhbi5iaUBpbnRlbC5jb20+DQo+PlN1YmplY3Q6IFtQQVRDSF0gTWRlTW9k dWxlUGtnL1VkZkR4ZTogUmVtb3ZlIG5lZ2F0aXZlIGNvbXBhcmlzb24gb2YgDQo+PnVuc2lnbmVk IG51bWJlcg0KPj4NCj4+VGhpcyBwYXRjaCBnZXRzIHJpZCBvZiBhIG5lZ2F0aXZlIGNvbXBhcmlz b24gb2YgYW4gVUlOVDY0IHR5cGUNCj4oT2Zmc2V0KQ0KPj5hcyBpdCdsbCBuZXZlciBldmFsdWF0 ZSB0byB0cnVlLg0KPj4NCj4+Q2M6IFN0YXIgWmVuZyA8c3Rhci56ZW5nQGludGVsLmNvbT4NCj4+ Q2M6IEVyaWMgRG9uZyA8ZXJpYy5kb25nQGludGVsLmNvbT4NCj4+Q2M6IFJ1aXl1IE5pIDxydWl5 dS5uaUBpbnRlbC5jb20+DQo+PkNjOiBEYW5kYW4gQmkgPGRhbmRhbi5iaUBpbnRlbC5jb20+DQo+ PkNvbnRyaWJ1dGVkLXVuZGVyOiBUaWFub0NvcmUgQ29udHJpYnV0aW9uIEFncmVlbWVudCAxLjEN Cj4+UmVwb3J0ZWQtYnk6IFN0YXIgWmVuZyA8c3Rhci56ZW5nQGludGVsLmNvbT4NCj4+U2lnbmVk LW9mZi1ieTogUGF1bG8gQWxjYW50YXJhIDxwY2FjanJAenl0b3IuY29tPg0KPj4tLS0NCj4+IE1k ZU1vZHVsZVBrZy9Vbml2ZXJzYWwvRGlzay9VZGZEeGUvRmlsZVN5c3RlbU9wZXJhdGlvbnMuYyB8 IDMgLS0tDQo+PiAxIGZpbGUgY2hhbmdlZCwgMyBkZWxldGlvbnMoLSkNCj4+DQo+PmRpZmYgLS1n aXQgYS9NZGVNb2R1bGVQa2cvVW5pdmVyc2FsL0Rpc2svVWRmRHhlL0ZpbGVTeXN0ZW1PcGVyYXRp b25zLmMNCj4+Yi9NZGVNb2R1bGVQa2cvVW5pdmVyc2FsL0Rpc2svVWRmRHhlL0ZpbGVTeXN0ZW1P cGVyYXRpb25zLmMNCj4+aW5kZXggNzI4NjI2NTM3My4uMjAzOWY4MDI4OSAxMDA2NDQNCj4+LS0t IGEvTWRlTW9kdWxlUGtnL1VuaXZlcnNhbC9EaXNrL1VkZkR4ZS9GaWxlU3lzdGVtT3BlcmF0aW9u cy5jDQo+PisrKyBiL01kZU1vZHVsZVBrZy9Vbml2ZXJzYWwvRGlzay9VZGZEeGUvRmlsZVN5c3Rl bU9wZXJhdGlvbnMuYw0KPj5AQCAtMTA4Miw5ICsxMDgyLDYgQEAgUmVhZEZpbGUgKA0KPj4gDQo+ PiAgICAgICAgaWYgKEZpbGVQb3NpdGlvbiArIEV4dGVudExlbmd0aCA+IFJlYWRGaWxlSW5mby0+ RmlsZVBvc2l0aW9uKQ0KPnsNCj4+ICAgICAgICAgICBPZmZzZXQgPSBSZWFkRmlsZUluZm8tPkZp bGVQb3NpdGlvbiAtIEZpbGVQb3NpdGlvbjsNCj4+LSAgICAgICAgICBpZiAoT2Zmc2V0IDwgMCkg ew0KPj4tICAgICAgICAgICAgT2Zmc2V0ID0gLShPZmZzZXQpOw0KPj4tICAgICAgICAgIH0NCj4+ ICAgICAgICAgfSBlbHNlIHsNCj4+ICAgICAgICAgICBPZmZzZXQgPSAwOw0KPj4gICAgICAgICB9 DQo+Pi0tDQo+PjIuMTEuMA0KPg0KPi0tDQo+U2VudCBmcm9tIG15IEFuZHJvaWQgZGV2aWNlIHdp dGggSy05IE1haWwuIFBsZWFzZSBleGN1c2UgbXkgYnJldml0eS4NCg0KLS0NClNlbnQgZnJvbSBt eSBBbmRyb2lkIGRldmljZSB3aXRoIEstOSBNYWlsLiBQbGVhc2UgZXhjdXNlIG15IGJyZXZpdHku DQo=