From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.136; helo=mga12.intel.com; envelope-from=ruiyu.ni@intel.com; receiver=edk2-devel@lists.01.org Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) (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 2DA71223CCEEC for ; Thu, 1 Feb 2018 16:48:47 -0800 (PST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 01 Feb 2018 16:54:25 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,445,1511856000"; d="scan'208";a="28172902" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by orsmga001.jf.intel.com with ESMTP; 01 Feb 2018 16:54:25 -0800 Received: from fmsmsx121.amr.corp.intel.com (10.18.125.36) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.319.2; Thu, 1 Feb 2018 16:54:24 -0800 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by fmsmsx121.amr.corp.intel.com (10.18.125.36) with Microsoft SMTP Server (TLS) id 14.3.319.2; Thu, 1 Feb 2018 16:54:24 -0800 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.127]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.116]) with mapi id 14.03.0319.002; Fri, 2 Feb 2018 08:54:22 +0800 From: "Ni, Ruiyu" To: Laszlo Ersek , "Shi, Steven" CC: "Yao, Jiewen" , "Zeng, Star" , "edk2-devel@lists.01.org" Thread-Topic: [edk2] [PATCH] MdeModulePkg/SmmCore: Fix hang due to already-freed memory deference Thread-Index: AQHTm0WuCoQtGeznk0iDyq2dONO9J6OPMjcAgAESuwA= Date: Fri, 2 Feb 2018 00:54:22 +0000 Message-ID: <734D49CCEBEEF84792F5B80ED585239D5BB73DE5@SHSMSX104.ccr.corp.intel.com> References: <20180201101539.320452-1-ruiyu.ni@intel.com> <1bde96c6-7ca3-0ee8-9990-6e0ca17026fe@redhat.com> In-Reply-To: <1bde96c6-7ca3-0ee8-9990-6e0ca17026fe@redhat.com> Accept-Language: en-US, zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMDkyYzZmYWUtNTg5OS00Mjk0LTkyM2UtY2U1Y2YwNjI0MTRiIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjIuNS4xOCIsIlRydXN0ZWRMYWJlbEhhc2giOiJrSUVtaGI5ZkN5T1k3VkVKVnR6SVJRY0NocHhra1NOU29XODRcL2gzYXROMXF6R2FIN2l1VmRqT1VmWjZ0TU5LVCJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH] MdeModulePkg/SmmCore: Fix hang due to already-freed memory deference X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 02 Feb 2018 00:48:48 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogTGFzemxvIEVyc2VrIFtt YWlsdG86bGVyc2VrQHJlZGhhdC5jb21dDQo+IFNlbnQ6IEZyaWRheSwgRmVicnVhcnkgMiwgMjAx OCAxMjoxMiBBTQ0KPiBUbzogTmksIFJ1aXl1IDxydWl5dS5uaUBpbnRlbC5jb20+OyBlZGsyLWRl dmVsQGxpc3RzLjAxLm9yZw0KPiBDYzogWWFvLCBKaWV3ZW4gPGppZXdlbi55YW9AaW50ZWwuY29t PjsgWmVuZywgU3RhciA8c3Rhci56ZW5nQGludGVsLmNvbT4NCj4gU3ViamVjdDogUmU6IFtlZGsy XSBbUEFUQ0hdIE1kZU1vZHVsZVBrZy9TbW1Db3JlOiBGaXggaGFuZyBkdWUgdG8gYWxyZWFkeS0N Cj4gZnJlZWQgbWVtb3J5IGRlZmVyZW5jZQ0KPiANCj4gSGVsbG8gUmF5LA0KPiANCj4gT24gMDIv MDEvMTggMTE6MTUsIFJ1aXl1IE5pIHdyb3RlOg0KPiA+IFNtaUhhbmRsZXJVblJlZ2lzdGVyKCkg dmFsaWRhdGVzIHRoZSBEaXNwYXRjaEhhbmRsZSBieSBjaGVja2luZw0KPiA+IHdoZXRoZXIgdGhl IGZpcnN0IDMyYml0IG1hdGNoZXMgdG8gYSBjZXJ0YWluIHNpZ25hdHVyZQ0KPiA+IChTTUlfSEFO RExFUl9TSUdOQVRVUkUpLg0KPiA+IEJ1dCBpZiBhIGNhbGxlciBjYWxscyAqVW5SZWdpc3Rlcigp IHR3aWNlIGFuZCB0aGUgbWVtb3J5IGZyZWVkIGJ5DQo+ID4gZmlyc3QgY2FsbCBzdGlsbCBjb250 YWlucyB0aGUgc2lnbmF0dXJlLCB0aGUgc2Vjb25kIGhhbmcgbWF5IGhhbmcuDQo+ID4NCj4gPiBU aGUgcGF0Y2ggZml4ZXMgdGhpcyBpc3N1ZSBieSBsb2NhdGluZyB0aGUgRGlzcGF0Y2hIYW5kbGUg aW4gYWxsIFNNSQ0KPiA+IGhhbmRsZXJzLCBpbnN0ZWFkIG9mIGNoZWNraW5nIHRoZSBzaWduYXR1 cmUuDQo+ID4NCj4gPiBDb250cmlidXRlZC11bmRlcjogVGlhbm9Db3JlIENvbnRyaWJ1dGlvbiBB Z3JlZW1lbnQgMS4xDQo+ID4gU2lnbmVkLW9mZi1ieTogUnVpeXUgTmkgPHJ1aXl1Lm5pQGludGVs LmNvbT4NCj4gPiBDYzogSmlld2VuIFlhbyA8amlld2VuLnlhb0BpbnRlbC5jb20+DQo+ID4gQ2M6 IFN0YXIgWmVuZyA8c3Rhci56ZW5nQGludGVsLmNvbT4NCj4gPiAtLS0NCj4gPiAgTWRlTW9kdWxl UGtnL0NvcmUvUGlTbW1Db3JlL1NtaS5jIHwgMzcNCj4gPiArKysrKysrKysrKysrKysrKysrKysr KysrKysrKysrKy0tLS0tDQo+ID4gIDEgZmlsZSBjaGFuZ2VkLCAzMiBpbnNlcnRpb25zKCspLCA1 IGRlbGV0aW9ucygtKQ0KPiANCj4gSSdtIG1pbGRseSBjdXJpb3VzOiBjYW4gd2UganVzdCB6ZXJv IG91dCB0aGUgc2lnbmF0dXJlIHdoZW4gdGhlIGRlLXJlZ2lzdHJhdGlvbiAvDQo+IGZyZWVpbmcg aGFwcGVucz8gT3RoZXJ3aXNlLCB0aGUgbmVzdGVkIGxvb3AgYWRkZWQgYmVsb3cgd2lsbCBwZW5h bGl6ZQ0KPiAocGVyZm9ybWFuY2Utd2lzZSkgY29ycmVjdGx5IHdyaXR0ZW4gY2xpZW50IGNvZGUg YXMgd2VsbC4NCj4gDQo+ID4gZGlmZiAtLWdpdCBhL01kZU1vZHVsZVBrZy9Db3JlL1BpU21tQ29y ZS9TbWkuYw0KPiA+IGIvTWRlTW9kdWxlUGtnL0NvcmUvUGlTbW1Db3JlL1NtaS5jDQo+ID4gaW5k ZXggYWQ0ODNhMTg3N2NlLi42NTk2ZWE5NTYwZDEgMTAwNjQ0DQo+ID4gLS0tIGEvTWRlTW9kdWxl UGtnL0NvcmUvUGlTbW1Db3JlL1NtaS5jDQo+ID4gKysrIGIvTWRlTW9kdWxlUGtnL0NvcmUvUGlT bW1Db3JlL1NtaS5jDQo+ID4gQEAgLTI5MCw2ICsyOTAsNyBAQCBTbWlIYW5kbGVyVW5SZWdpc3Rl ciAoDQo+ID4gICAgU21pRW50cnkgPSBTbWlIYW5kbGVyLT5TbWlFbnRyeTsNCj4gPg0KPiA+ICAg IFJlbW92ZUVudHJ5TGlzdCAoJlNtaUhhbmRsZXItPkxpbmspOw0KPiA+ICsgIFNtaUhhbmRsZXIt PlNpZ25hdHVyZSA9IDA7DQo+ID4gICAgRnJlZVBvb2wgKFNtaUhhbmRsZXIpOw0KPiA+DQo+ID4g ICAgaWYgKFNtaUVudHJ5ID09IE5VTEwpIHsNCj4gDQo+IEdlbmVyYWxseSwgaWYgY2xpZW50IGNv ZGUgdmlvbGF0ZXMgYW4gaW50ZXJmYWNlIGNvbnRyYWN0LCB0aGVuIHRoZSBjYWxsZWQgZnVuY3Rp b24gaXMNCj4gbm90IHJlc3BvbnNpYmxlIGZvciBjYXRjaGluZyB0aGUgZXJyb3IgYW5kIHByZXZl bnRpbmcgdW5kZWZpbmVkIGJlaGF2aW9yLiBGb3INCj4gInF1YWxpdHkgb2Ygc2VydmljZSIsIHdl IGNhbiBnbyB0byBjZXJ0YWluIGxlbmd0aHMgbm9uZXRoZWxlc3MsIGJ1dCBpdCBzaG91bGQNCj4g aG9wZWZ1bGx5IG5vdCBodXJ0IHZhbGlkIGNsaWVudCBjb2RlLg0KPiANCj4gRm9yIGV4YW1wbGUs IEkgc2VlbSB0byByZW1lbWJlciB0aGF0IHRoZSBsaXN0IGRhdGEgc3RydWN0dXJlIGltcGxlbWVu dGF0aW9uDQo+IGNoZWNrcyB0aGUgaW50ZXJuYWwgY29uc2lzdGVuY3kgKHdoaWNoIGNhbiBiZSBj b3N0bHkpIG9ubHkgaWYgYSBQQ0QgaXMgc2V0IHRvIGENCj4gY2VydGFpbiB2YWx1ZS4gSXMgdGhh dCByaWdodD8gSXMgaXQgYW4gb3B0aW9uIGhlcmU/IChJZiB0aGUgYWJvdmUgemVyb2luZyBpcyBu b3QgZ29vZA0KPiBmb3Igc29tZSByZWFzb24uKQ0KPiANCj4gQW55d2F5LCBJJ20gYXNraW5nIG1h aW5seSBmb3IgbXkgb3duIGVkdWNhdGlvbi4NCj4gDQo+IFRoYW5rcyENCj4gTGFzemxvDQoNCkxh c3psbywNCkkgYWdyZWUgeW91ciBmaXggaXMgcXVpdGUgc2ltcGxlIGFuZCBubyBwZXJmb3JtYW5j ZSBpbXBhY3QuDQpCdXQgaWYgY2FsbGVyIHN1cHBsaWVzIGFuIGludmFsaWQgRGlzcGF0Y2hIYW5k bGUsIGNoZWNraW5nIHRoZSBzaWduYXR1cmUNCm1lYW5zIHRvIHJlYWQgdGhlIG1lbW9yeSB3aG9z ZSBhZGRyZXNzIGlzIHByb3ZpZGVkIGJ5IGNhbGxlci4NCkkgcmVtZW1iZXIgU3RldmVuIFNoaSBz dWJtaXR0ZWQgc2V2ZXJhbCBidWdzIHJlZ2FyZGluZyB0aGlzIGJlY2F1c2UNCmhlIGNvbnNpZGVy ZWQgc3VjaCByZWFkaW5nIGFjY2VzcyBpcyBiYWQuDQoNCg0KU3RldmVuLA0KQW55IGNvbW1lbnRz Pw0KDQo+IA0KPiA+IGRpZmYgLS1naXQgYS9NZGVNb2R1bGVQa2cvQ29yZS9QaVNtbUNvcmUvU21p LmMNCj4gPiBiL01kZU1vZHVsZVBrZy9Db3JlL1BpU21tQ29yZS9TbWkuYw0KPiA+IGluZGV4IGFk NDgzYTE4NzcuLjBjMDllN2ZhMTAgMTAwNjQ0DQo+ID4gLS0tIGEvTWRlTW9kdWxlUGtnL0NvcmUv UGlTbW1Db3JlL1NtaS5jDQo+ID4gKysrIGIvTWRlTW9kdWxlUGtnL0NvcmUvUGlTbW1Db3JlL1Nt aS5jDQo+ID4gQEAgLTEsNyArMSw3IEBADQo+ID4gIC8qKiBAZmlsZQ0KPiA+ICAgIFNNSSBtYW5h Z2VtZW50Lg0KPiA+DQo+ID4gLSAgQ29weXJpZ2h0IChjKSAyMDA5IC0gMjAxNywgSW50ZWwgQ29y cG9yYXRpb24uIEFsbCByaWdodHMNCj4gPiByZXNlcnZlZC48QlI+DQo+ID4gKyAgQ29weXJpZ2h0 IChjKSAyMDA5IC0gMjAxOCwgSW50ZWwgQ29ycG9yYXRpb24uIEFsbCByaWdodHMNCj4gPiArIHJl c2VydmVkLjxCUj4NCj4gPiAgICBUaGlzIHByb2dyYW0gYW5kIHRoZSBhY2NvbXBhbnlpbmcgbWF0 ZXJpYWxzIGFyZSBsaWNlbnNlZCBhbmQgbWFkZQ0KPiBhdmFpbGFibGUNCj4gPiAgICB1bmRlciB0 aGUgdGVybXMgYW5kIGNvbmRpdGlvbnMgb2YgdGhlIEJTRCBMaWNlbnNlIHdoaWNoIGFjY29tcGFu aWVzIHRoaXMNCj4gPiAgICBkaXN0cmlidXRpb24uICBUaGUgZnVsbCB0ZXh0IG9mIHRoZSBsaWNl bnNlIG1heSBiZSBmb3VuZCBhdA0KPiA+IEBAIC0yNzYsMTQgKzI3Niw0MSBAQCBTbWlIYW5kbGVy VW5SZWdpc3RlciAoICB7DQo+ID4gICAgU01JX0hBTkRMRVIgICpTbWlIYW5kbGVyOw0KPiA+ICAg IFNNSV9FTlRSWSAgICAqU21pRW50cnk7DQo+ID4gKyAgTElTVF9FTlRSWSAgICpFbnRyeUxpbms7 DQo+ID4gKyAgTElTVF9FTlRSWSAgICpIYW5kbGVyTGluazsNCj4gPg0KPiA+IC0gIFNtaUhhbmRs ZXIgPSAoU01JX0hBTkRMRVIgKikgRGlzcGF0Y2hIYW5kbGU7DQo+ID4gLQ0KPiA+IC0gIGlmIChT bWlIYW5kbGVyID09IE5VTEwpIHsNCj4gPiArICBpZiAoRGlzcGF0Y2hIYW5kbGUgPT0gTlVMTCkg ew0KPiA+ICAgICAgcmV0dXJuIEVGSV9JTlZBTElEX1BBUkFNRVRFUjsNCj4gPiAgICB9DQo+ID4N Cj4gPiAtICBpZiAoU21pSGFuZGxlci0+U2lnbmF0dXJlICE9IFNNSV9IQU5ETEVSX1NJR05BVFVS RSkgew0KPiA+ICsgIC8vDQo+ID4gKyAgLy8gTG9vayBmb3IgaXQgaW4gcm9vdCBTTUkgaGFuZGxl cnMgIC8vICBTbWlIYW5kbGVyID0gTlVMTDsgIGZvciAoDQo+ID4gKyBIYW5kbGVyTGluayA9IEdl dEZpcnN0Tm9kZSAoJm1Sb290U21pRW50cnkuU21pSGFuZGxlcnMpDQo+ID4gKyAgICAgIDsgIUlz TnVsbCAoJm1Sb290U21pRW50cnkuU21pSGFuZGxlcnMsIEhhbmRsZXJMaW5rKSAmJiAoU21pSGFu ZGxlciAhPQ0KPiBEaXNwYXRjaEhhbmRsZSkNCj4gPiArICAgICAgOyBIYW5kbGVyTGluayA9IEdl dE5leHROb2RlICgmbVJvb3RTbWlFbnRyeS5TbWlIYW5kbGVycywgSGFuZGxlckxpbmspDQo+ID4g KyAgICAgICkgew0KPiA+ICsgICAgU21pSGFuZGxlciA9IENSIChIYW5kbGVyTGluaywgU01JX0hB TkRMRVIsIExpbmssDQo+ID4gKyBTTUlfSEFORExFUl9TSUdOQVRVUkUpOyAgfQ0KPiA+ICsNCj4g PiArICAvLw0KPiA+ICsgIC8vIExvb2sgZm9yIGl0IGluIG5vbi1yb290IFNNSSBoYW5kbGVycyAg Ly8gIGZvciAoIEVudHJ5TGluayA9DQo+ID4gKyBHZXRGaXJzdE5vZGUgKCZtU21pRW50cnlMaXN0 KQ0KPiA+ICsgICAgICA7ICFJc051bGwgKCZtU21pRW50cnlMaXN0LCBFbnRyeUxpbmspICYmIChT bWlIYW5kbGVyICE9IERpc3BhdGNoSGFuZGxlKQ0KPiA+ICsgICAgICA7IEVudHJ5TGluayA9IEdl dE5leHROb2RlICgmbVNtaUVudHJ5TGlzdCwgRW50cnlMaW5rKQ0KPiA+ICsgICAgICApIHsNCj4g PiArICAgIFNtaUVudHJ5ID0gQ1IgKEVudHJ5TGluaywgU01JX0VOVFJZLCBBbGxFbnRyaWVzLCBT TUlfRU5UUllfU0lHTkFUVVJFKTsNCj4gPiArICAgIGZvciAoIEhhbmRsZXJMaW5rID0gR2V0Rmly c3ROb2RlICgmU21pRW50cnktPlNtaUhhbmRsZXJzKQ0KPiA+ICsgICAgICAgIDsgIUlzTnVsbCAo JlNtaUVudHJ5LT5TbWlIYW5kbGVycywgSGFuZGxlckxpbmspICYmIChTbWlIYW5kbGVyICE9DQo+ IERpc3BhdGNoSGFuZGxlKQ0KPiA+ICsgICAgICAgIDsgSGFuZGxlckxpbmsgPSBHZXROZXh0Tm9k ZSAoJlNtaUVudHJ5LT5TbWlIYW5kbGVycywgSGFuZGxlckxpbmspDQo+ID4gKyAgICAgICAgKSB7 DQo+ID4gKyAgICAgIFNtaUhhbmRsZXIgPSBDUiAoSGFuZGxlckxpbmssIFNNSV9IQU5ETEVSLCBM aW5rLA0KPiBTTUlfSEFORExFUl9TSUdOQVRVUkUpOw0KPiA+ICsgICAgfQ0KPiA+ICsgIH0NCj4g PiArDQo+ID4gKyAgaWYgKFNtaUhhbmRsZXIgIT0gRGlzcGF0Y2hIYW5kbGUpIHsNCj4gPiAgICAg IHJldHVybiBFRklfSU5WQUxJRF9QQVJBTUVURVI7DQo+ID4gICAgfQ0KPiA+DQo+ID4NCg0K