From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mx.groups.io with SMTP id smtpd.web09.3809.1578618118657519635 for ; Thu, 09 Jan 2020 17:01:58 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.65, mailfrom: siyuan.fu@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 09 Jan 2020 17:01:58 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.69,414,1571727600"; d="scan'208";a="272259694" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by FMSMGA003.fm.intel.com with ESMTP; 09 Jan 2020 17:01:57 -0800 Received: from FMSMSX109.amr.corp.intel.com (10.18.116.9) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 9 Jan 2020 17:01:57 -0800 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by fmsmsx109.amr.corp.intel.com (10.18.116.9) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 9 Jan 2020 17:01:57 -0800 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.245]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.30]) with mapi id 14.03.0439.000; Fri, 10 Jan 2020 09:01:55 +0800 From: "Siyuan, Fu" To: "devel@edk2.groups.io" , "lersek@redhat.com" CC: "Wu, Jiaxin" , Maciej Rabeda Subject: Re: [edk2-devel] [PATCH 2/2] NetworkPkg/HttpDxe: fix 32-bit truncation in HTTPS download Thread-Topic: [edk2-devel] [PATCH 2/2] NetworkPkg/HttpDxe: fix 32-bit truncation in HTTPS download Thread-Index: AQHVxn1vb+MQ/C0erEyrTGIg7sxHT6fjFhVg Date: Fri, 10 Jan 2020 01:01:54 +0000 Message-ID: References: <20200108234313.28510-1-lersek@redhat.com> <20200108234313.28510-3-lersek@redhat.com> In-Reply-To: <20200108234313.28510-3-lersek@redhat.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNTRlYTY1OWYtMWEzOS00ZWE4LWEwZmUtZDU4ZDBmYjc5ODA3IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoid3NvMjZMdFd0ZGt3YnNrRzdFeUJqQ0VBMkVmaFh5OEU5ZmdPVktcL0hYWGRDaHlJem5yV1NRUDRxQWZaNkVRWVwvIn0= 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+IE9uIEJlaGFsZiBPZiBMYXN6bG8NCj4gRXJzZWsNCj4gU2VudDogMjAy MMTqMdTCOcjVIDc6NDMNCj4gVG86IGVkazItZGV2ZWwtZ3JvdXBzLWlvIDxkZXZlbEBlZGsyLmdy b3Vwcy5pbz4NCj4gQ2M6IFd1LCBKaWF4aW4gPGppYXhpbi53dUBpbnRlbC5jb20+OyBNYWNpZWog UmFiZWRhDQo+IDxtYWNpZWoucmFiZWRhQGxpbnV4LmludGVsLmNvbT47IEZ1LCBTaXl1YW4gPHNp eXVhbi5mdUBpbnRlbC5jb20+DQo+IFN1YmplY3Q6IFtlZGsyLWRldmVsXSBbUEFUQ0ggMi8yXSBO ZXR3b3JrUGtnL0h0dHBEeGU6IGZpeCAzMi1iaXQgdHJ1bmNhdGlvbg0KPiBpbiBIVFRQUyBkb3du bG9hZA0KPiANCj4gV2hlbiBkb3dubG9hZGluZyBvdmVyIFRMUywgZWFjaCBUTFMgbWVzc2FnZSAo IkFQUCBwYWNrZXQiKSBpcyByZXR1cm5lZCBhcw0KPiBhIChkZWNyeXB0ZWQpIGZyYWdtZW50IHRh YmxlIGJ5IEVGSV9UTFNfUFJPVE9DT0wuUHJvY2Vzc1BhY2tldCgpLg0KPiANCj4gVGhlIFRsc1By b2Nlc3NNZXNzYWdlKCkgZnVuY3Rpb24gaW4gIk5ldHdvcmtQa2cvSHR0cER4ZS9IdHRwc1N1cHBv cnQuYyINCj4gbGluZWFyaXplcyB0aGUgZnJhZ21lbnQgdGFibGUgaW50byBhIHNpbmdsZSBjb250 aWd1b3VzIGRhdGEgYmxvY2suIFRoZQ0KPiByZXN1bHRhbnQgZmxhdCBkYXRhIGJsb2NrIGNvbnRh aW5zIGJvdGggVExTIGhlYWRlcnMgYW5kIGRhdGEuDQo+IA0KPiBUaGUgSHR0cHNSZWNlaXZlKCkg ZnVuY3Rpb24gcGFyc2VzIHRoZSBhY3R1YWwgYXBwbGljYXRpb24gZGF0YSAtLSBpbiB0aGlzDQo+ IGNhc2U6IGRlY3J5cHRlZCBIVFRQIGRhdGEgLS0gb3V0IG9mIHRoZSBmbGF0dGVuZWQgVExTIGRh dGEgYmxvY2ssIHBlZWxpbmcNCj4gb2ZmIHRoZSBUTFMgaGVhZGVycy4NCj4gDQo+IFRoZSBIdHRw UmVzcG9uc2VXb3JrZXIoKSBmdW5jdGlvbiBpbiAiTmV0d29ya1BrZy9IdHRwRHhlL0h0dHBJbXBs LmMiDQo+IHByb3BhZ2F0ZXMgdGhpcyBIVFRQIGRhdGEgb3V0d2FyZHMsIGltcGxlbWVudGluZyB0 aGUNCj4gRUZJX0hUVFBfUFJPVE9DT0wuUmVzcG9uc2UoKSBmdW5jdGlvbi4NCj4gDQo+IE5vdyBj b25zaWRlciB0aGUgZm9sbG93aW5nIGRvY3VtZW50YXRpb24gZm9yDQo+IEVGSV9IVFRQX1BST1RP Q09MLlJlc3BvbnNlKCksDQo+IHF1b3RlZCBmcm9tICJNZGVQa2cvSW5jbHVkZS9Qcm90b2NvbC9I dHRwLmgiOg0KPiANCj4gPiBJdCBpcyB0aGUgcmVzcG9uc2liaWxpdHkgb2YgdGhlIGNhbGxlciB0 byBhbGxvY2F0ZSBhIGJ1ZmZlciBmb3IgQm9keSBhbmQNCj4gPiBzcGVjaWZ5IHRoZSBzaXplIGlu IEJvZHlMZW5ndGguIElmIHRoZSByZW1vdGUgaG9zdCBwcm92aWRlcyBhIHJlc3BvbnNlDQo+ID4g dGhhdCBjb250YWlucyBhIGNvbnRlbnQgYm9keSwgdXAgdG8gQm9keUxlbmd0aCBieXRlcyB3aWxs IGJlIGNvcGllZCBmcm9tDQo+ID4gdGhlIHJlY2VpdmUgYnVmZmVyIGludG8gQm9keSBhbmQgQm9k eUxlbmd0aCB3aWxsIGJlIHVwZGF0ZWQgd2l0aCB0aGUNCj4gPiBhbW91bnQgb2YgYnl0ZXMgcmVj ZWl2ZWQgYW5kIGNvcGllZCB0byBCb2R5LiBUaGlzIGFsbG93cyB0aGUgY2xpZW50IHRvDQo+ID4g ZG93bmxvYWQgYSBsYXJnZSBmaWxlIGluIGNodW5rcyBpbnN0ZWFkIG9mIGludG8gb25lIGNvbnRp Z3VvdXMgYmxvY2sgb2YNCj4gPiBtZW1vcnkuDQo+IA0KPiBOb3RlIHRoYXQsIGlmIHRoZSBjYWxs ZXItYWxsb2NhdGVkIGJ1ZmZlciBpcyBsYXJnZXIgdGhhbiB0aGUNCj4gc2VydmVyLXByb3ZpZGVk IGNodW5rLCB0aGVuIHRoZSB0cmFuc2ZlciBsZW5ndGggaXMgbGltaXRlZCBieSB0aGUgbGF0dGVy Lg0KPiBUaGlzIGlzIGluIGZhY3QgdGhlIGRvbWluYW50IGNhc2Ugd2hlbiBkb3dubG9hZGluZyBh IGh1Z2UgZmlsZSAoZm9yIHdoaWNoDQo+IFVlZmlCb290TWFuYWdlckxpYiBhbGxvY2F0ZWQgYSBo dWdlIGNvbnRpZ3VvdXMgUkFNIERpc2sgYnVmZmVyKSBpbiBzbWFsbA0KPiBUTFMgbWVzc2FnZXMu DQo+IA0KPiBGb3IgYWRqdXN0aW5nIEJvZHlMZW5ndGggYXMgZGVzY3JpYmVkIGFib3ZlIC0tIGku ZS4sIHRvIHRoZSBhcHBsaWNhdGlvbg0KPiBkYXRhIGNodW5rIHRoYXQgaGFzIGJlZW4gZXh0cmFj dGVkIGZyb20gdGhlIFRMUyBtZXNzYWdlIC0tLCB0aGUNCj4gSHR0cFJlc3BvbnNlV29ya2VyKCkg ZnVuY3Rpb24gZW1wbG95cyB0aGUgZm9sbG93aW5nIGFzc2lnbm1lbnQ6DQo+IA0KPiAgICAgSHR0 cE1zZy0+Qm9keUxlbmd0aCA9IE1JTiAoRnJhZ21lbnQuTGVuLCAoVUlOVDMyKSBIdHRwTXNnLQ0K PiA+Qm9keUxlbmd0aCk7DQo+IA0KPiBUaGUgKFVJTlQzMikgY2FzdCBpcyBtb3RpdmF0ZWQgYnkg dGhlIE1JTigpIHJlcXVpcmVtZW50IC0tIGluDQo+ICJNZGVQa2cvSW5jbHVkZS9CYXNlLmgiIC0t IHRoYXQgYm90aCBhcmd1bWVudHMgYmUgb2YgdGhlIHNhbWUgdHlwZS4NCj4gDQo+ICJGcmFnbWVu dC5MZW4iIChORVRfRlJBR01FTlQuTGVuKSBoYXMgdHlwZSBVSU5UMzIsIGFuZA0KPiAiSHR0cE1z Zy0+Qm9keUxlbmd0aCIgKEVGSV9IVFRQX01FU1NBR0UuQm9keUxlbmd0aCkgaGFzIHR5cGUgVUlO VE4uDQo+IFRoZXJlZm9yZSBhIGNhc3QgaXMgaW5kZWVkIG5lY2Vzc2FyeS4NCj4gDQo+IFVuZm9y dHVuYXRlbHksIHRoZSBjYXN0IGlzIGRvbmUgaW4gdGhlIHdyb25nIGRpcmVjdGlvbi4gQ29uc2lk ZXIgdGhlDQo+IGZvbGxvd2luZyBjaXJjdW1zdGFuY2VzOg0KPiANCj4gLSAiRnJhZ21lbnQuTGVu IiBoYXBwZW5zIHRvIGJlIGNvbnNpc3RlbnRseSAxNktpQiwgZGljdGF0ZWQgYnkgdGhlIEhUVFBT DQo+ICAgU2VydmVyJ3MgVExTIHN0YWNrLA0KPiANCj4gLSB0aGUgc2l6ZSBvZiB0aGUgZmlsZSB0 byBkb3dubG9hZCBpcyA0R2lCICsgTioxNktpQiwgd2hlcmUgTiBpcyBhDQo+ICAgcG9zaXRpdmUg aW50ZWdlci4NCj4gDQo+IEFzIHRoZSBkb3dubG9hZCBwcm9ncmVzc2VzLCBlYWNoIHJlY2VpdmVk IDE2S2lCIGFwcGxpY2F0aW9uIGRhdGEgY2h1bmsNCj4gYnJpbmdzIHRoZSAqbmV4dCogaW5wdXQg dmFsdWUgb2YgQm9keUxlbmd0aCBjbG9zZXIgZG93biB0byA0R2lCLiBUaGUgY2FzdA0KPiBpbiBN SU4oKSBhbHdheXMgbWFza3Mgb2ZmIHRoZSBoaWdoLW9yZGVyIGJpdHMgZnJvbSB0aGUgaW5wdXQg dmFsdWUgb2YNCj4gQm9keUxlbmd0aCwgYnV0IHRoaXMgaXMgbm8gcHJvYmxlbSBiZWNhdXNlIHRo ZSBsb3ctb3JkZXIgYml0cyBhcmUgbm9uemVybywNCj4gdGhlcmVmb3JlIHRoZSBNSU4oKSBhbHdh eXMgcGVybWl0cyBwcm9ncmVzcy4NCj4gDQo+IEhvd2V2ZXIsIG9uY2UgQm9keUxlbmd0aCByZWFj aGVzIDRHaUIgZXhhY3RseSBvbiBpbnB1dCwgdGhlIE1JTigpDQo+IGludm9jYXRpb24gcHJvZHVj ZXMgYSB6ZXJvIHZhbHVlLiBIdHRwUmVzcG9uc2VXb3JrZXIoKSBhZGp1c3RzIHRoZSBvdXRwdXQN Cj4gdmFsdWUgb2YgQm9keUxlbmd0aCB0byB6ZXJvLCBhbmQgdGhlbiBwYXNzZXMgaXQgdG8gSHR0 cFBhcnNlTWVzc2FnZUJvZHkoKS4NCj4gDQo+IEh0dHBQYXJzZU1lc3NhZ2VCb2R5KCkgKGluICJO ZXR3b3JrUGtnL0xpYnJhcnkvRHhlSHR0cExpYi9EeGVIdHRwTGliLmMiKQ0KPiByZWplY3RzIHRo ZSB6ZXJvIEJvZHlMZW5ndGggd2l0aCBFRklfSU5WQUxJRF9QQVJBTUVURVIsIHdoaWNoIGlzIGZ1 bGx5DQo+IHByb3BhZ2F0ZWQgb3V0d2FyZHMsIGFuZCBhYm9ydHMgdGhlIEhUVFBTIGRvd25sb2Fk LiBIdHRwQm9vdER4ZSB3cml0ZXMNCj4gdGhlDQo+IG1lc3NhZ2UgIkVycm9yOiBVbmV4cGVjdGVk IG5ldHdvcmsgZXJyb3IiIHRvIHRoZSBVRUZJIGNvbnNvbGUuDQo+IA0KPiBGb3IgZXhhbXBsZSwg YSBmaWxlIHdpdGggc2l6ZSAoNEdpQiArIDE5N01pQikgdGVybWluYXRlcyBhZnRlciBkb3dubG9h ZGluZw0KPiBqdXN0IDE5N01pQi4NCj4gDQo+IEludmVydCB0aGUgZGlyZWN0aW9uIG9mIHRoZSBj YXN0OiB3aWRlbiAiRnJhZ21lbnQuTGVuIiB0byBVSU5UTi4NCj4gDQo+IENjOiBKaWF4aW4gV3Ug PGppYXhpbi53dUBpbnRlbC5jb20+DQo+IENjOiBNYWNpZWogUmFiZWRhIDxtYWNpZWoucmFiZWRh QGxpbnV4LmludGVsLmNvbT4NCj4gQ2M6IFNpeXVhbiBGdSA8c2l5dWFuLmZ1QGludGVsLmNvbT4N Cj4gU2lnbmVkLW9mZi1ieTogTGFzemxvIEVyc2VrIDxsZXJzZWtAcmVkaGF0LmNvbT4NCj4gLS0t DQo+ICBOZXR3b3JrUGtnL0h0dHBEeGUvSHR0cEltcGwuYyB8IDIgKy0NCj4gIDEgZmlsZSBjaGFu Z2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQ0KPiANCj4gZGlmZiAtLWdpdCBhL05l dHdvcmtQa2cvSHR0cER4ZS9IdHRwSW1wbC5jDQo+IGIvTmV0d29ya1BrZy9IdHRwRHhlL0h0dHBJ bXBsLmMNCj4gaW5kZXggNmI4NzczMTRiZDU3Li4xYWNiYjYwZDEwMTQgMTAwNjQ0DQo+IC0tLSBh L05ldHdvcmtQa2cvSHR0cER4ZS9IdHRwSW1wbC5jDQo+ICsrKyBiL05ldHdvcmtQa2cvSHR0cER4 ZS9IdHRwSW1wbC5jDQo+IEBAIC0xMzQ4LDcgKzEzNDgsNyBAQCBIdHRwUmVzcG9uc2VXb3JrZXIg KA0KPiAgICAgIC8vDQo+ICAgICAgLy8gUHJvY2VzcyB0aGUgcmVjZWl2ZWQgdGhlIGJvZHkgcGFj a2V0Lg0KPiAgICAgIC8vDQo+IC0gICAgSHR0cE1zZy0+Qm9keUxlbmd0aCA9IE1JTiAoRnJhZ21l bnQuTGVuLCAoVUlOVDMyKSBIdHRwTXNnLQ0KPiA+Qm9keUxlbmd0aCk7DQo+ICsgICAgSHR0cE1z Zy0+Qm9keUxlbmd0aCA9IE1JTiAoKFVJTlROKSBGcmFnbWVudC5MZW4sIEh0dHBNc2ctDQo+ID5C b2R5TGVuZ3RoKTsNCj4gDQo+ICAgICAgQ29weU1lbSAoSHR0cE1zZy0+Qm9keSwgRnJhZ21lbnQu QnVsaywgSHR0cE1zZy0+Qm9keUxlbmd0aCk7DQo+IA0KPiAtLQ0KPiAyLjE5LjEuMy5nMzAyNDdh YTVkMjAxDQo+IA0KPiANCj4gDQoNCg==