From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from szxga02-in.huawei.com (szxga02-in.huawei.com [45.249.212.188]) by mx.groups.io with SMTP id smtpd.web09.3278.1646985789156962477 for ; Fri, 11 Mar 2022 00:03:09 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: huawei.com, ip: 45.249.212.188, mailfrom: xiewenyi2@huawei.com) Received: from kwepemi100020.china.huawei.com (unknown [172.30.72.55]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4KFJKK3MjtzBrLB; Fri, 11 Mar 2022 16:01:09 +0800 (CST) Received: from kwepemm600004.china.huawei.com (7.193.23.242) by kwepemi100020.china.huawei.com (7.221.188.48) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.18; Fri, 11 Mar 2022 16:03:06 +0800 Received: from [10.174.253.58] (10.174.253.58) by kwepemm600004.china.huawei.com (7.193.23.242) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.21; Fri, 11 Mar 2022 16:03:05 +0800 Subject: Re: [PATCH EDK2 v1 1/1] RedfishPkg: fix memory leak issue To: "Chang, Abner (HPS SW/FW Technologist)" , "devel@edk2.groups.io" , "Wang, Nickle (Server BIOS)" CC: "songdongkuang@huawei.com" References: <20220310112534.2952545-1-xiewenyi2@huawei.com> <20220310112534.2952545-2-xiewenyi2@huawei.com> From: "wenyi,xie" Message-ID: Date: Fri, 11 Mar 2022 16:02:19 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.0.1 MIME-Version: 1.0 In-Reply-To: X-Originating-IP: [10.174.253.58] X-ClientProxiedBy: dggems704-chm.china.huawei.com (10.3.19.181) To kwepemm600004.china.huawei.com (7.193.23.242) X-CFilter-Loop: Reflected Content-Type: text/plain; charset="windows-1252" Content-Language: en-US Content-Transfer-Encoding: base64 SGkgQWJuZXIsDQoNClRoYW5rIHlvdSBmb3IgeW91ciByZXZpZXdpbmcsIGhlcmUgaXMgdGhl IGxpbmsgdG8gdGhlIFBSLg0KaHR0cHM6Ly9naXRodWIuY29tL3RpYW5vY29yZS9lZGsyL3B1 bGwvMjYxMg0KDQpSZWdhcmRzDQpXZW55aQ0KDQpPbiAyMDIyLzMvMTEgMTU6MDMsIENoYW5n LCBBYm5lciAoSFBTIFNXL0ZXIFRlY2hub2xvZ2lzdCkgd3JvdGU6DQo+IEhpIFdlbnlpLA0K PiBQbGVhc2UgYWRkIG15IFJCIGluIHRoZSBjb21taXQgbWVzc2FnZSBhbmQgY3JlYXRlIHRo ZSBQUiBhZ2FpbnN0IGVkazIsIGxldCBtZaBrbm93IHRoZSBsaW5rIHRvIFBSIGFzIHdlbGwu DQo+IA0KPiBUaGFua3MsDQo+IEFibmVyDQo+IC0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQ0KPiAqRnJv bToqIENoYW5nLCBBYm5lciAoSFBTIFNXL0ZXIFRlY2hub2xvZ2lzdCkNCj4gKlNlbnQ6KiBG cmlkYXksIE1hcmNoIDExLCAyMDIyIDE6NDggUE0NCj4gKlRvOiogV2VueWkgWGllIDx4aWV3 ZW55aTJAaHVhd2VpLmNvbT47IGRldmVsQGVkazIuZ3JvdXBzLmlvIDxkZXZlbEBlZGsyLmdy b3Vwcy5pbz47IFdhbmcsIE5pY2tsZSAoU2VydmVyIEJJT1MpIDxuaWNrbGUud2FuZ0BocGUu Y29tPg0KPiAqQ2M6KiBzb25nZG9uZ2t1YW5nQGh1YXdlaS5jb20gPHNvbmdkb25na3VhbmdA aHVhd2VpLmNvbT4NCj4gKlN1YmplY3Q6KiBSRTogW1BBVENIIEVESzIgdjEgMS8xXSBSZWRm aXNoUGtnOiBmaXggbWVtb3J5IGxlYWsgaXNzdWUNCj4goA0KPiBUaGFua3MgZm9yIGNhdGNo aW5nIHRoaXMuDQo+IA0KPiBSZXZpZXdlZC1ieTogQWJuZXIgQ2hhbmcgPGFibmVyLmNoYW5n QGhwZS5jb20+DQo+IA0KPj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4+IEZyb206 IFdlbnlpIFhpZSA8eGlld2VueWkyQGh1YXdlaS5jb20+DQo+PiBTZW50OiBUaHVyc2RheSwg TWFyY2ggMTAsIDIwMjIgNzoyNiBQTQ0KPj4gVG86IGRldmVsQGVkazIuZ3JvdXBzLmlvOyBD aGFuZywgQWJuZXIgKEhQUyBTVy9GVyBUZWNobm9sb2dpc3QpDQo+PiA8YWJuZXIuY2hhbmdA aHBlLmNvbT47IFdhbmcsIE5pY2tsZSAoU2VydmVyIEJJT1MpDQo+PiA8bmlja2xlLndhbmdA aHBlLmNvbT4NCj4+IENjOiBzb25nZG9uZ2t1YW5nQGh1YXdlaS5jb207IHhpZXdlbnlpMkBo dWF3ZWkuY29tDQo+PiBTdWJqZWN0OiBbUEFUQ0ggRURLMiB2MSAxLzFdIFJlZGZpc2hQa2c6 IGZpeCBtZW1vcnkgbGVhayBpc3N1ZQ0KPj4gDQo+PiBUaGUgY2FsbG9jIG1lbW9yeSBpcyBu b3QgZnJlZSB3aGVuIGZ1bmN0aW9uIGNvbGxlY3Rpb25FdmFsT3ANCj4+IHJldHVybiBpbiB0 aGUgaGFsZndheS4NCj4+IA0KPj4gQ2M6IEFibmVyIENoYW5nIDxhYm5lci5jaGFuZ0BocGUu Y29tPg0KPj4gQ2M6IE5pY2tsZSBXYW5nIDxuaWNrbGUud2FuZ0BocGUuY29tPg0KPj4gU2ln bmVkLW9mZi1ieTogV2VueWkgWGllIDx4aWV3ZW55aTJAaHVhd2VpLmNvbT4NCj4+IC0tLQ0K Pj6gIFJlZGZpc2hQa2cvUHJpdmF0ZUxpYnJhcnkvUmVkZmlzaExpYi9lZGsybGlicmVkZmlz aC9zcmMvcGF5bG9hZC5jIHwgMTYNCj4+ICsrKysrKystLS0tLS0tLS0NCj4+oCAxIGZpbGUg Y2hhbmdlZCwgNyBpbnNlcnRpb25zKCspLCA5IGRlbGV0aW9ucygtKQ0KPj4gDQo+PiBkaWZm IC0tZ2l0DQo+PiBhL1JlZGZpc2hQa2cvUHJpdmF0ZUxpYnJhcnkvUmVkZmlzaExpYi9lZGsy bGlicmVkZmlzaC9zcmMvcGF5bG9hZC5jDQo+PiBiL1JlZGZpc2hQa2cvUHJpdmF0ZUxpYnJh cnkvUmVkZmlzaExpYi9lZGsybGlicmVkZmlzaC9zcmMvcGF5bG9hZC5jDQo+PiBpbmRleCAz ZjJiODNlODM0ZDAuLjZjNmUyMjQ2YWJlMyAxMDA2NDQNCj4+IC0tLSBhL1JlZGZpc2hQa2cv UHJpdmF0ZUxpYnJhcnkvUmVkZmlzaExpYi9lZGsybGlicmVkZmlzaC9zcmMvcGF5bG9hZC5j DQo+PiArKysgYi9SZWRmaXNoUGtnL1ByaXZhdGVMaWJyYXJ5L1JlZGZpc2hMaWIvZWRrMmxp YnJlZGZpc2gvc3JjL3BheWxvYWQuYw0KPj4gQEAgLTYyMCw2ICs2MjAsNyBAQCBjb2xsZWN0 aW9uRXZhbE9wICgNCj4+oKCgIGlmICgoKCpTdGF0dXNDb2RlID09IE5VTEwpICYmIChtZW1i ZXJzID09IE5VTEwpKSB8fA0KPj6goKCgoKCgICgoKlN0YXR1c0NvZGUgIT0gTlVMTCkgJiYg KCgqKlN0YXR1c0NvZGUgPCBIVFRQX1NUQVRVU18yMDBfT0spIHx8DQo+PiAoKipTdGF0dXND b2RlID4gSFRUUF9TVEFUVVNfMjA2X1BBUlRJQUxfQ09OVEVOVCkpKSkNCj4+oKCgIHsNCj4+ ICugoKAgZnJlZSAodmFsaWQpOw0KPj6goKCgoCByZXR1cm4gbWVtYmVyczsNCj4+oKCgIH0N Cj4+IA0KPj4gQEAgLTYzMyw2ICs2MzQsNyBAQCBjb2xsZWN0aW9uRXZhbE9wICgNCj4+oKCg oKAgaWYgKCgoKlN0YXR1c0NvZGUgPT0gTlVMTCkgJiYgKHRtcCA9PSBOVUxMKSkgfHwNCj4+ oKCgoKCgoKCgICgoKlN0YXR1c0NvZGUgIT0gTlVMTCkgJiYgKCgqKlN0YXR1c0NvZGUgPCBI VFRQX1NUQVRVU18yMDBfT0spIHx8DQo+PiAoKipTdGF0dXNDb2RlID4gSFRUUF9TVEFUVVNf MjA2X1BBUlRJQUxfQ09OVEVOVCkpKSkNCj4+oKCgoKAgew0KPj4gK6CgoKCgIGZyZWUgKHZh bGlkKTsNCj4+oKCgoKCgoCByZXR1cm4gdG1wOw0KPj6goKCgoCB9DQo+PiANCj4+IEBAIC02 NTgsMTkgKzY2MCwxNSBAQCBjb2xsZWN0aW9uRXZhbE9wICgNCj4+IA0KPj6goKAgY2xlYW51 cFBheWxvYWQgKG1lbWJlcnMpOw0KPj6goKAgaWYgKHZhbGlkQ291bnQgPT0gMCkgew0KPj4g LaCgoCBmcmVlICh2YWxpZCk7DQo+PiAtoKCgIHJldHVybiBOVUxMOw0KPj4gLaAgfQ0KPj4g LQ0KPj4gLaAgaWYgKHZhbGlkQ291bnQgPT0gMSkgew0KPj4gK6CgoCByZXQgPSBOVUxMOw0K Pj4gK6AgfSBlbHNlIGlmICh2YWxpZENvdW50ID09IDEpIHsNCj4+oKCgoKAgcmV0ID0gdmFs aWRbMF07DQo+PiAtoKCgIGZyZWUgKHZhbGlkKTsNCj4+IC2goKAgcmV0dXJuIHJldDsNCj4+ oKCgIH0gZWxzZSB7DQo+PqCgoKCgIHJldCA9IGNyZWF0ZUNvbGxlY3Rpb24gKHBheWxvYWQt PnNlcnZpY2UsIHZhbGlkQ291bnQsIHZhbGlkKTsNCj4+IC2goKAgZnJlZSAodmFsaWQpOw0K Pj4gLaCgoCByZXR1cm4gcmV0Ow0KPj6goKAgfQ0KPj4gKw0KPj4gK6AgZnJlZSAodmFsaWQp Ow0KPj4gK6AgcmV0dXJuIHJldDsNCj4+oCB9DQo+PiANCj4+oCBzdGF0aWMgcmVkZmlzaFBh eWxvYWQgKg0KPj4gLS0NCj4+IDIuMjAuMS53aW5kb3dzLjENCj4gDQo=