From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mx.groups.io with SMTP id smtpd.web10.4081.1587862840469645468 for ; Sat, 25 Apr 2020 18:00:40 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.88, mailfrom: siyuan.fu@intel.com) IronPort-SDR: hNaOKh+XamlfgG5jz7txF3c4Iia4cjwAlPTD5Wt9CxAcEAwZatXZHxZPuqjF0GxquyrqPZwnS2 Qd8YxgBFciZA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Apr 2020 18:00:39 -0700 IronPort-SDR: z675PUjQ17NhkwP1jEEfoGxXbgUCl5luYE71wPqpweqdGIvNC4xVIS7WP1UDNr3iKUuqlL0ZIL V20Q+P0olT6w== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,318,1583222400"; d="scan'208";a="458366090" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by fmsmga006.fm.intel.com with ESMTP; 25 Apr 2020 18:00:39 -0700 Received: from fmsmsx154.amr.corp.intel.com (10.18.116.70) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sat, 25 Apr 2020 18:00:39 -0700 Received: from shsmsx153.ccr.corp.intel.com (10.239.6.53) by FMSMSX154.amr.corp.intel.com (10.18.116.70) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sat, 25 Apr 2020 18:00:39 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.146]) by SHSMSX153.ccr.corp.intel.com ([169.254.12.89]) with mapi id 14.03.0439.000; Sun, 26 Apr 2020 09:00:36 +0800 From: "Siyuan, Fu" To: "devel@edk2.groups.io" , "Zhang, Shenglei" CC: Maciej Rabeda , "Wu, Jiaxin" Subject: Re: [edk2-devel] [PATCH] NetworkPkg/IScsiDxe: Enhance the check for array boundary Thread-Topic: [edk2-devel] [PATCH] NetworkPkg/IScsiDxe: Enhance the check for array boundary Thread-Index: AQHWGhi58I3R3M6KpEyI+2g3IL2n4aiKl/iw Date: Sun, 26 Apr 2020 01:00:36 +0000 Message-ID: References: <20200424091356.157536-1-shenglei.zhang@intel.com> In-Reply-To: <20200424091356.157536-1-shenglei.zhang@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: siyuan.fu@intel.com Content-Language: en-US Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: base64 UmV2aWV3ZWQtYnk6IFNpeXVhbiBGdSA8c2l5dWFuLmZ1QGludGVsLmNvbT4NCg0KPiAtLS0tLU9y aWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBkZXZlbEBlZGsyLmdyb3Vwcy5pbyA8ZGV2ZWxA ZWRrMi5ncm91cHMuaW8+IE9uIEJlaGFsZiBPZiBaaGFuZywNCj4gU2hlbmdsZWkNCj4gU2VudDog MjAyMMTqNNTCMjTI1SAxNzoxNA0KPiBUbzogZGV2ZWxAZWRrMi5ncm91cHMuaW8NCj4gQ2M6IE1h Y2llaiBSYWJlZGEgPG1hY2llai5yYWJlZGFAbGludXguaW50ZWwuY29tPjsgV3UsIEppYXhpbg0K PiA8amlheGluLnd1QGludGVsLmNvbT47IEZ1LCBTaXl1YW4gPHNpeXVhbi5mdUBpbnRlbC5jb20+ DQo+IFN1YmplY3Q6IFtlZGsyLWRldmVsXSBbUEFUQ0hdIE5ldHdvcmtQa2cvSVNjc2lEeGU6IEVu aGFuY2UgdGhlIGNoZWNrIGZvcg0KPiBhcnJheSBib3VuZGFyeQ0KPiANCj4gQXJyYXkgJ1Rhcmdl dFVybCcgb2Ygc2l6ZSAyNTUgbWF5IHVzZSBpbmRleCB2YWx1ZShzKSAyNTUgYW5kIDI1Ni4NCj4g U28gZW5oYW5jZSB0aGUgYm91bmRhcnkgY2hlY2sgdG8gZW5zdXJlIHRoZSBpbmRleCBpcyB2YWxp ZC4NCj4gDQo+IENjOiBNYWNpZWogUmFiZWRhIDxtYWNpZWoucmFiZWRhQGxpbnV4LmludGVsLmNv bT4NCj4gQ2M6IEppYXhpbiBXdSA8amlheGluLnd1QGludGVsLmNvbT4NCj4gQ2M6IFNpeXVhbiBG dSA8c2l5dWFuLmZ1QGludGVsLmNvbT4NCj4gU2lnbmVkLW9mZi1ieTogU2hlbmdsZWkgWmhhbmcg PHNoZW5nbGVpLnpoYW5nQGludGVsLmNvbT4NCj4gLS0tDQo+ICBOZXR3b3JrUGtnL0lTY3NpRHhl L0lTY3NpRGhjcC5jICB8IDIgKy0NCj4gIE5ldHdvcmtQa2cvSVNjc2lEeGUvSVNjc2lEaGNwNi5j IHwgMiArLQ0KPiAgMiBmaWxlcyBjaGFuZ2VkLCAyIGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25z KC0pDQo+IA0KPiBkaWZmIC0tZ2l0IGEvTmV0d29ya1BrZy9JU2NzaUR4ZS9JU2NzaURoY3AuYw0K PiBiL05ldHdvcmtQa2cvSVNjc2lEeGUvSVNjc2lEaGNwLmMNCj4gaW5kZXggN2NlOWJmNTc1MDEy Li5hZmE4YTg2Y2Q0MTkgMTAwNjQ0DQo+IC0tLSBhL05ldHdvcmtQa2cvSVNjc2lEeGUvSVNjc2lE aGNwLmMNCj4gKysrIGIvTmV0d29ya1BrZy9JU2NzaUR4ZS9JU2NzaURoY3AuYw0KPiBAQCAtMTIy LDcgKzEyMiw3IEBAIElTY3NpRGhjcEV4dHJhY3RSb290UGF0aCAoDQo+ICAgIC8vDQo+ICAgIGlm ICgoIU5FVF9JU19ESUdJVCAoKihGaWVsZC0+U3RyKSkpICYmICgqKEZpZWxkLT5TdHIpICE9ICdb JykpIHsNCj4gICAgICBDb25maWdOdkRhdGEtPkRuc01vZGUgPSBUUlVFOw0KPiAtICAgIGlmIChG aWVsZC0+TGVuID4gc2l6ZW9mIChDb25maWdOdkRhdGEtPlRhcmdldFVybCkpIHsNCj4gKyAgICBp ZiAoKEZpZWxkLT5MZW4gKyAyKSA+IHNpemVvZiAoQ29uZmlnTnZEYXRhLT5UYXJnZXRVcmwpKSB7 DQo+ICAgICAgICByZXR1cm4gRUZJX0lOVkFMSURfUEFSQU1FVEVSOw0KPiAgICAgIH0NCj4gICAg ICBDb3B5TWVtICgmQ29uZmlnTnZEYXRhLT5UYXJnZXRVcmwsIEZpZWxkLT5TdHIsIEZpZWxkLT5M ZW4pOw0KPiBkaWZmIC0tZ2l0IGEvTmV0d29ya1BrZy9JU2NzaUR4ZS9JU2NzaURoY3A2LmMNCj4g Yi9OZXR3b3JrUGtnL0lTY3NpRHhlL0lTY3NpRGhjcDYuYw0KPiBpbmRleCA4NmE4NzJhZGVjY2Mu LjY5MTY1MGI3MzM0ZiAxMDA2NDQNCj4gLS0tIGEvTmV0d29ya1BrZy9JU2NzaUR4ZS9JU2NzaURo Y3A2LmMNCj4gKysrIGIvTmV0d29ya1BrZy9JU2NzaUR4ZS9JU2NzaURoY3A2LmMNCj4gQEAgLTE2 MSw3ICsxNjEsNyBAQCBJU2NzaURoY3A2RXh0cmFjdFJvb3RQYXRoICgNCj4gICAgLy8gU2VydmVy IG5hbWUgaXMgZXhwcmVzc2VkIGFzIGRvbWFpbiBuYW1lLCBqdXN0IHNhdmUgaXQuDQo+ICAgIC8v DQo+ICAgIGlmIChDb25maWdOdkRhdGEtPkRuc01vZGUpIHsNCj4gLSAgICBpZiAoRmllbGQtPkxl biA+IHNpemVvZiAoQ29uZmlnTnZEYXRhLT5UYXJnZXRVcmwpKSB7DQo+ICsgICAgaWYgKChGaWVs ZC0+TGVuICsgMikgPiBzaXplb2YgKENvbmZpZ052RGF0YS0+VGFyZ2V0VXJsKSkgew0KPiAgICAg ICAgcmV0dXJuIEVGSV9JTlZBTElEX1BBUkFNRVRFUjsNCj4gICAgICB9DQo+ICAgICAgQ29weU1l bSAoJkNvbmZpZ052RGF0YS0+VGFyZ2V0VXJsLCBGaWVsZC0+U3RyLCBGaWVsZC0+TGVuKTsNCj4g LS0NCj4gMi4xOC4wLndpbmRvd3MuMQ0KPiANCj4gDQo+IA0KDQo=