From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <pcacjr@zytor.com>
Received-SPF: Pass (sender SPF authorized) identity=mailfrom;
 client-ip=65.50.211.136; helo=mail.zytor.com; envelope-from=pcacjr@zytor.com;
 receiver=edk2-devel@lists.01.org 
Received: from mail.zytor.com (terminus.zytor.com [65.50.211.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 55453220D4C17
 for <edk2-devel@lists.01.org>; Tue, 14 Nov 2017 07:58:52 -0800 (PST)
Received: from [IPv6:2804:7f4:c480:4fed::1] ([IPv6:2804:7f4:c480:4fed:0:0:0:1])
 (authenticated bits=0)
 by mail.zytor.com (8.15.2/8.15.2) with ESMTPSA id vAEG0fQT008620
 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO);
 Tue, 14 Nov 2017 08:00:45 -0800
To: Hao Wu <hao.a.wu@intel.com>, edk2-devel@lists.01.org
Cc: Ruiyu Ni <ruiyu.ni@intel.com>, Star Zeng <star.zeng@intel.com>,
 Eric Dong <eric.dong@intel.com>
References: <20171114075130.14936-1-hao.a.wu@intel.com>
From: Paulo Alcantara <pcacjr@zytor.com>
Message-ID: <cab191f8-4f67-6da7-c5ae-f5ce91b01322@zytor.com>
Date: Tue, 14 Nov 2017 14:00:40 -0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.4.0
MIME-Version: 1.0
In-Reply-To: <20171114075130.14936-1-hao.a.wu@intel.com>
Subject: Re: [PATCH 0/2] Refine UDF related codes
X-BeenThere: edk2-devel@lists.01.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: EDK II Development  <edk2-devel.lists.01.org>
List-Unsubscribe: <https://lists.01.org/mailman/options/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=unsubscribe>
List-Archive: <http://lists.01.org/pipermail/edk2-devel/>
List-Post: <mailto:edk2-devel@lists.01.org>
List-Help: <mailto:edk2-devel-request@lists.01.org?subject=help>
List-Subscribe: <https://lists.01.org/mailman/listinfo/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=subscribe>
X-List-Received-Date: Tue, 14 Nov 2017 15:58:52 -0000
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit

Hi Hao,

On 14/11/2017 05:51, Hao Wu wrote:
> The series will do the following refinements:
> a. Merge the discovery of the El Torito feature on CD/DVD media into the
>     detect of UDF;
> b. Avoid possible loss track of the allocated buffer pointer in UdfDxe.
> 
> Cc: Paulo Alcantara <pcacjr@zytor.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> 
> Hao Wu (2):
>    MdeModulePkg/PartitionDxe: Merge the discovery of ElTorito into UDF
>    MdeModulePkg/UdfDxe: Avoid possible loss track of allocated buffer
> 
>   MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c      |  7 +++----
>   MdeModulePkg/Universal/Disk/PartitionDxe/Udf.c            | 17 +++++++++++++++++
>   MdeModulePkg/Universal/Disk/UdfDxe/FileSystemOperations.c | 12 +++++++-----
>   3 files changed, 27 insertions(+), 9 deletions(-)
> 

Looks good to me. Thanks for fixing and testing it!

Reviewed-by: Paulo Alcantara <pcacjr@zytor.com>

Paulo