Hi, Hao Wu <[email protected]> writes:
> This series will refine the codes in MdeModulePkg/Universal/Disk/UdfDxe > for: > > A. Refine asserts used for memory allocation failure and error cases that > are possible to happen. Will use error handling logic for them; > > B. Address some dead codes within this module. > > Cc: Paulo Alcantara <[email protected]> > Cc: Ruiyu Ni <[email protected]> > Cc: Star Zeng <[email protected]> > > Hao Wu (7): > MdeModulePkg/UdfDxe: Use error handling for memory allocation failure > MdeModulePkg/UdfDxe: ASSERT for false positives of NULL ptr deref > MdeModulePkg/UdfDxe: Use error handling when fail to return LSN > MdeModulePkg/UdfDxe: Use debug msg instead of ASSERT in UdfOpen() > MdeModulePkg/UdfDxe: Handle dead codes in File.c > MdeModulePkg/UdfDxe: Remove dead codes in FileName.c > MdeModulePkg/UdfDxe: Handle dead codes in FileSystemOperations.c > > MdeModulePkg/Universal/Disk/UdfDxe/File.c | 19 ++- > MdeModulePkg/Universal/Disk/UdfDxe/FileName.c | 15 -- > MdeModulePkg/Universal/Disk/UdfDxe/FileSystemOperations.c | 162 > +++++++++++++++----- > 3 files changed, 142 insertions(+), 54 deletions(-) Looks good to me. Thanks! For the series: Reviewed-by: Paulo Alcantara <[email protected]> Paulo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel

