From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.88, mailfrom: shenglei.zhang@intel.com) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by groups.io with SMTP; Mon, 16 Sep 2019 20:36:05 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 Sep 2019 20:36:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,515,1559545200"; d="scan'208";a="191269500" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga006.jf.intel.com with ESMTP; 16 Sep 2019 20:36:04 -0700 Received: from fmsmsx154.amr.corp.intel.com (10.18.116.70) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Mon, 16 Sep 2019 20:36:04 -0700 Received: from shsmsx104.ccr.corp.intel.com (10.239.4.70) by FMSMSX154.amr.corp.intel.com (10.18.116.70) with Microsoft SMTP Server (TLS) id 14.3.439.0; Mon, 16 Sep 2019 20:36:04 -0700 Received: from shsmsx106.ccr.corp.intel.com ([169.254.10.86]) by SHSMSX104.ccr.corp.intel.com ([169.254.5.32]) with mapi id 14.03.0439.000; Tue, 17 Sep 2019 11:36:02 +0800 From: "Zhang, Shenglei" To: "devel@edk2.groups.io" , "Desimone, Nathaniel L" CC: "Kubacki, Michael A" , "Chiu, Chasel" , "Gao, Liming" Subject: Re: [edk2-devel] [PATCH] MinPlatformPkg/TestPointCheckLib: Add check for pointers Thread-Topic: [edk2-devel] [PATCH] MinPlatformPkg/TestPointCheckLib: Add check for pointers Thread-Index: AQHVaRV0RM9r65z6ek6/Ig9JQMlfdqcoZJpggAJEgQCABJPvoA== Date: Tue, 17 Sep 2019 03:36:01 +0000 Message-ID: References: <20190912025442.36436-1-shenglei.zhang@intel.com> <15C3C5B9A4267C61.28831@groups.io> <4e8f9867-73b8-9ca8-aafc-b4a1567ca61c@intel.com> In-Reply-To: <4e8f9867-73b8-9ca8-aafc-b4a1567ca61c@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: shenglei.zhang@intel.com Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 SGkgTmF0aGFuaWVsLA0KDQo+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0tDQo+IEZyb206IERl c2ltb25lLCBOYXRoYW5pZWwgTA0KPiBTZW50OiBTYXR1cmRheSwgU2VwdGVtYmVyIDE0LCAyMDE5 IDY6MzEgQU0NCj4gVG86IFpoYW5nLCBTaGVuZ2xlaSA8c2hlbmdsZWkuemhhbmdAaW50ZWwuY29t PjsgZGV2ZWxAZWRrMi5ncm91cHMuaW8NCj4gQ2M6IEt1YmFja2ksIE1pY2hhZWwgQSA8bWljaGFl bC5hLmt1YmFja2lAaW50ZWwuY29tPjsgQ2hpdSwgQ2hhc2VsDQo+IDxjaGFzZWwuY2hpdUBpbnRl bC5jb20+OyBHYW8sIExpbWluZyA8bGltaW5nLmdhb0BpbnRlbC5jb20+DQo+IFN1YmplY3Q6IFJl OiBbZWRrMi1kZXZlbF0gW1BBVENIXSBNaW5QbGF0Zm9ybVBrZy9UZXN0UG9pbnRDaGVja0xpYjog QWRkDQo+IGNoZWNrIGZvciBwb2ludGVycw0KPiANCj4gSGkgU2hlbmdsZWksDQo+IA0KPiBMb29r aW5nIGF0IHRoaXMgcGF0Y2ggbW9yZSBjbG9zZWx5LiBUaGVyZSBhcHBlYXIgdG8gYmUgYnVncy4u LiBwbGVhc2UNCj4gc2VlIGJlbG93LiBQbGVhc2UgZml4IHRoaXMgYWxvbmcgd2l0aCB5b3VyIHBv b3IgdXNlIG9mIHNlbWktY29sb25zIGFuZA0KPiB3aGl0ZS1zcGFjZS4NCj4gDQo+IFRoYW5rcywN Cj4gDQo+IE5hdGUNCj4gDQo+IE9uIDkvMTIvMjAxOSAxMTo1NCBBTSwgTmF0ZSBEZVNpbW9uZSB3 cm90ZToNCj4gPiBZb3VyIHdoaXRlc3BhY2UgZG9lc24ndCBxdWl0ZSBtYXRjaCB0aGUgZWRrMiBj b2Rpbmcgc3R5bGUgZ3VpZGVsaW5lcywgYnV0DQo+IHdlIGNhbiBmaXggdGhhdCBkdXJpbmcgY29t bWl0Lg0KPiA+DQo+ID4gUmV2aWV3ZWQtYnk6IE5hdGUgRGVTaW1vbmUgPG5hdGhhbmllbC5sLmRl c2ltb25lQGludGVsLmNvbT4NCj4gPg0KPiA+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0tDQo+ ID4gRnJvbTogWmhhbmcsIFNoZW5nbGVpDQo+ID4gU2VudDogV2VkbmVzZGF5LCBTZXB0ZW1iZXIg MTEsIDIwMTkgNzo1NSBQTQ0KPiA+IFRvOiBkZXZlbEBlZGsyLmdyb3Vwcy5pbw0KPiA+IENjOiBL dWJhY2tpLCBNaWNoYWVsIEEgPG1pY2hhZWwuYS5rdWJhY2tpQGludGVsLmNvbT47IENoaXUsIENo YXNlbA0KPiA8Y2hhc2VsLmNoaXVAaW50ZWwuY29tPjsgRGVzaW1vbmUsIE5hdGhhbmllbCBMDQo+ IDxuYXRoYW5pZWwubC5kZXNpbW9uZUBpbnRlbC5jb20+OyBHYW8sIExpbWluZyA8bGltaW5nLmdh b0BpbnRlbC5jb20+DQo+ID4gU3ViamVjdDogW1BBVENIXSBNaW5QbGF0Zm9ybVBrZy9UZXN0UG9p bnRDaGVja0xpYjogQWRkIGNoZWNrIGZvcg0KPiBwb2ludGVycw0KPiA+DQo+ID4gSW4gRHhlQ2hl Y2tCb290VmFyaWFibGUuYywgYWRkIGNoZWNrIGZvciBCb290T3JkZXIgYW5kIFZhcmlhYmxlIHRo YXQNCj4gcmV0dXJuIEVGSV9OT1RfRk9VTkQgd2hlbiB0aGV5IGFyZSBOVUxMLg0KPiA+IEluIER4 ZUNoZWNrR2NkLmMsIGFkZCBjaGVjayBmb3IgR2NkSW9NYXAgdG8gZW5zdXJlIGl0IG5vdCBOVUxM IHdoZW4NCj4gYWxsb2NhdGluZyBtZW1vcnkgdG8gd2hhdCBpdCBwb2ludHMgdG8uDQo+ID4NCj4g PiBDYzogTWljaGFlbCBLdWJhY2tpIDxtaWNoYWVsLmEua3ViYWNraUBpbnRlbC5jb20+DQo+ID4g Q2M6IENoYXNlbCBDaGl1IDxjaGFzZWwuY2hpdUBpbnRlbC5jb20+DQo+ID4gQ2M6IE5hdGUgRGVT aW1vbmUgPG5hdGhhbmllbC5sLmRlc2ltb25lQGludGVsLmNvbT4NCj4gPiBDYzogTGltaW5nIEdh byA8bGltaW5nLmdhb0BpbnRlbC5jb20+DQo+ID4gU2lnbmVkLW9mZi1ieTogU2hlbmdsZWkgWmhh bmcgPHNoZW5nbGVpLnpoYW5nQGludGVsLmNvbT4NCj4gPiAtLS0NCj4gPg0KPiA+IHYyOiBVcGRh dGUgY29weXJpZ2h0DQo+ID4NCj4gPiAgIC4uLi9UZXN0L0xpYnJhcnkvVGVzdFBvaW50Q2hlY2tM aWIvRHhlQ2hlY2tCb290VmFyaWFibGUuYyB8IDggKysrKysrKy0NCj4gPiAgIC4uLi9UZXN0L0xp YnJhcnkvVGVzdFBvaW50Q2hlY2tMaWIvRHhlQ2hlY2tHY2QuYyAgICAgICAgICB8IDYgKysrKy0t DQo+ID4gICAyIGZpbGVzIGNoYW5nZWQsIDExIGluc2VydGlvbnMoKyksIDMgZGVsZXRpb25zKC0p DQo+ID4NCj4gPiBkaWZmIC0tZ2l0DQo+IGEvUGxhdGZvcm0vSW50ZWwvTWluUGxhdGZvcm1Qa2cv VGVzdC9MaWJyYXJ5L1Rlc3RQb2ludENoZWNrTGliL0R4ZUNoZWNrDQo+IEJvb3RWYXJpYWJsZS5j DQo+IGIvUGxhdGZvcm0vSW50ZWwvTWluUGxhdGZvcm1Qa2cvVGVzdC9MaWJyYXJ5L1Rlc3RQb2lu dENoZWNrTGliL0R4ZUNoZWNrDQo+IEJvb3RWYXJpYWJsZS5jDQo+ID4gaW5kZXggODViZDViM2Qu Ljk4MTMwNjgzIDEwMDY0NA0KPiA+IC0tLQ0KPiBhL1BsYXRmb3JtL0ludGVsL01pblBsYXRmb3Jt UGtnL1Rlc3QvTGlicmFyeS9UZXN0UG9pbnRDaGVja0xpYi9EeGVDaGVjaw0KPiBCb290VmFyaWFi bGUuYw0KPiA+ICsrKw0KPiBiL1BsYXRmb3JtL0ludGVsL01pblBsYXRmb3JtUGtnL1Rlc3QvTGli cmFyeS9UZXN0UG9pbnRDaGVja0xpYi9EeGVDaA0KPiA+ICsrKyBlY2tCb290VmFyaWFibGUuYw0K PiA+IEBAIC0xLDYgKzEsNiBAQA0KPiA+ICAgLyoqIEBmaWxlDQo+ID4NCj4gPiAtQ29weXJpZ2h0 IChjKSAyMDE3LCBJbnRlbCBDb3Jwb3JhdGlvbi4gQWxsIHJpZ2h0cyByZXNlcnZlZC48QlI+DQo+ ID4gK0NvcHlyaWdodCAoYykgMjAxNy0yMDE5LCBJbnRlbCBDb3Jwb3JhdGlvbi4gQWxsIHJpZ2h0 cyByZXNlcnZlZC48QlI+DQo+ID4gICBTUERYLUxpY2Vuc2UtSWRlbnRpZmllcjogQlNELTItQ2xh dXNlLVBhdGVudA0KPiA+DQo+ID4gICAqKi8NCj4gPiBAQCAtMTMwLDYgKzEzMCw5IEBAIFRlc3RQ b2ludENoZWNrTG9hZE9wdGlvblZhcmlhYmxlICgNCj4gPiAgICAgZm9yIChMaXN0SW5kZXggPSAw OyBMaXN0SW5kZXggPA0KPiBzaXplb2YobUxvYWRPcHRpb25WYXJpYWJsZUxpc3QpL3NpemVvZiht TG9hZE9wdGlvblZhcmlhYmxlTGlzdFswXSk7DQo+IExpc3RJbmRleCsrKSB7DQo+ID4gICAgICAg VW5pY29kZVNQcmludCAoQm9vdE9yZGVyTmFtZSwgc2l6ZW9mKEJvb3RPcmRlck5hbWUpLCBMIiVz T3JkZXIiLA0KPiBtTG9hZE9wdGlvblZhcmlhYmxlTGlzdFtMaXN0SW5kZXhdKTsNCj4gPiAgICAg ICBTdGF0dXMgPSBHZXRWYXJpYWJsZTIgKEJvb3RPcmRlck5hbWUsICZnRWZpR2xvYmFsVmFyaWFi bGVHdWlkLA0KPiAoVk9JRCAqKikmQm9vdE9yZGVyLCAmT3JkZXJTaXplKTsNCj4gPiArICAgIGlm KEJvb3RPcmRlciA9PSBOVUxMKSB7DQo+ID4gKyAgICAgIHJldHVybiBFRklfTk9UX0ZPVU5EOzsN Cj4gPiArICAgIH0NCj4gPiAgICAgICBpZiAoRUZJX0VSUk9SKFN0YXR1cykpIHsNCj4gPiAgICAg ICAgIGNvbnRpbnVlOw0KPiA+ICAgICAgIH0NCj4gPiBAQCAtMjIyLDYgKzIyNSw5IEBAIFRlc3RQ b2ludENoZWNrS2V5T3B0aW9uVmFyaWFibGUgKA0KPiA+ICAgICAgIGZvciAoSW5kZXggPSAwOyA7 IEluZGV4KyspIHsNCj4gPiAgICAgICAgIFVuaWNvZGVTUHJpbnQgKEtleU9wdGlvbk5hbWUsIHNp emVvZihLZXlPcHRpb25OYW1lKSwgTCIlcyUwNHgiLA0KPiBtS2V5T3B0aW9uVmFyaWFibGVMaXN0 W0xpc3RJbmRleF0sIEluZGV4KTsNCj4gPiAgICAgICAgIFN0YXR1cyA9IEdldFZhcmlhYmxlMiAo S2V5T3B0aW9uTmFtZSwgJmdFZmlHbG9iYWxWYXJpYWJsZUd1aWQsDQo+ICZWYXJpYWJsZSwgJlNp emUpOw0KPiA+ICsgICAgICBpZihWYXJpYWJsZSA9PSBOVUxMKSB7DQo+ID4gKyAgICAgICAgcmV0 dXJuIEVGSV9OT1RfRk9VTkQ7Ow0KPiA+ICsgICAgICB9DQo+ID4gICAgICAgICBpZiAoIUVGSV9F UlJPUihTdGF0dXMpKSB7DQo+ID4gICAgICAgICAgIER1bXBLZXlPcHRpb24gKEtleU9wdGlvbk5h bWUsIFZhcmlhYmxlLCBTaXplKTsNCj4gPiAgICAgICAgIH0gZWxzZSB7DQo+ID4gZGlmZiAtLWdp dA0KPiBhL1BsYXRmb3JtL0ludGVsL01pblBsYXRmb3JtUGtnL1Rlc3QvTGlicmFyeS9UZXN0UG9p bnRDaGVja0xpYi9EeGVDaGVjaw0KPiBHY2QuYw0KPiBiL1BsYXRmb3JtL0ludGVsL01pblBsYXRm b3JtUGtnL1Rlc3QvTGlicmFyeS9UZXN0UG9pbnRDaGVja0xpYi9EeGVDaGVjaw0KPiBHY2QuYw0K PiA+IGluZGV4IDgyNzA5ZDQ0Li5jOTBiMzdmMiAxMDA2NDQNCj4gPiAtLS0NCj4gYS9QbGF0Zm9y bS9JbnRlbC9NaW5QbGF0Zm9ybVBrZy9UZXN0L0xpYnJhcnkvVGVzdFBvaW50Q2hlY2tMaWIvRHhl Q2hlY2sNCj4gR2NkLmMNCj4gPiArKysNCj4gYi9QbGF0Zm9ybS9JbnRlbC9NaW5QbGF0Zm9ybVBr Zy9UZXN0L0xpYnJhcnkvVGVzdFBvaW50Q2hlY2tMaWIvRHhlQ2gNCj4gPiArKysgZWNrR2NkLmMN Cj4gPiBAQCAtMSw2ICsxLDYgQEANCj4gPiAgIC8qKiBAZmlsZQ0KPiA+DQo+ID4gLUNvcHlyaWdo dCAoYykgMjAxNywgSW50ZWwgQ29ycG9yYXRpb24uIEFsbCByaWdodHMgcmVzZXJ2ZWQuPEJSPg0K PiA+ICtDb3B5cmlnaHQgKGMpIDIwMTctMjAxOSwgSW50ZWwgQ29ycG9yYXRpb24uIEFsbCByaWdo dHMgcmVzZXJ2ZWQuPEJSPg0KPiA+ICAgU1BEWC1MaWNlbnNlLUlkZW50aWZpZXI6IEJTRC0yLUNs YXVzZS1QYXRlbnQNCj4gPg0KPiA+ICAgKiovDQo+ID4gQEAgLTI0MSw3ICsyNDEsOSBAQCBUZXN0 UG9pbnREdW1wR2NkICgNCj4gPiAgICAgICAgIH0NCj4gPiAgICAgICB9DQo+ID4gICAgICAgaWYg KEdjZE1lbW9yeU1hcCAhPSBOVUxMKSB7DQo+ID4gLSAgICAgICpHY2RJb01hcCA9IEFsbG9jYXRl Q29weVBvb2wgKE51bWJlck9mRGVzY3JpcHRvcnMgKg0KPiBzaXplb2YoRUZJX0dDRF9JT19TUEFD RV9ERVNDUklQVE9SKSwgSW9NYXApOw0KPiA+ICsgICAgICBpZiAoR2NkSW9NYXAgIT0gTlVMTCl7 DQo+ID4gKyAgICAgICAgKkdjZElvTWFwID0gQWxsb2NhdGVDb3B5UG9vbCAoTnVtYmVyT2ZEZXNj cmlwdG9ycyAqDQo+IHNpemVvZihFRklfR0NEX0lPX1NQQUNFX0RFU0NSSVBUT1IpLCBJb01hcCk7 DQo+ID4gKyAgICAgIH0NCj4gDQo+IEdjZElvTWFwIHdpbGwgYWx3YXlzIGJlIE5VTEwuIFBsZWFz ZSBzZWUgbGluZSAxOTkgb2YgdGhpcyBmaWxlLiBJDQo+IGJlbGlldmUgeW91ciBwYXRjaCBpcyBp bnRyb2R1Y2luZyBhIG5ldyBidWcuDQoNCkkgZmFpbCB0byBnZXQgeW91ciBwb2ludC4gV291bGQg eW91IGV4cGxhaW4gd2h5IEdjZElvTWFwIHdpbGwgYWx3YXlzIGJlIE5VTEw/ICBJdCBsb29rcyBs aWtlIGFuIGlucHV0DQpwYXJhbWV0ZXIgZnJvbSBvdGhlciBwbGFjZS4NCg0KVGhhbmtzLA0KU2hl bmdsZWkNCg0KPiANCj4gPiAgICAgICAgICpHY2RJb01hcE51bWJlck9mRGVzY3JpcHRvcnMgPSBO dW1iZXJPZkRlc2NyaXB0b3JzOw0KPiA+ICAgICAgIH0NCj4gPiAgICAgfQ0KPiA+IC0tDQo+ID4g Mi4xOC4wLndpbmRvd3MuMQ0KPiA+DQo+ID4NCj4gPiANCj4gPg0K