Re: [edk2-platforms][PATCH v5 0/2] Ext4Pkg: Add Symbolic Links support

["Pedro Falcato" <pedro.falcato@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1602 bytes --]

Series-Reviewed-by: Pedro Falcato <pedro.falcato@gmail.com>

On Wed, Sep 7, 2022 at 3:02 PM Savva Mitrofanov <savvamtr@gmail.com> wrote:

> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3677
>
> Hi all,
>
> In the fifth version I corrected remarks from Pedro Falcato and Marvin
> Häuser.
> I moved symlink procedures into independent C file, corrected code style
> like
> debug messages macros and etc.
>
> This patchset adds symbolic links support with simple recursion protection
> based
> on symbolic link nest level limitation, also I included patch which adds
> BASE_CR
> to extract EXT4_FILE private structure to prevent possible code corruption
> caused
> by structure changes and rearrangements in future.
>
> REF:
> https://github.com/savvamitrofanov/edk2-platforms/tree/ext4pkg_symlink_support
>
> Cc: Marvin Häuser <mhaeuser@posteo.de>
> Cc: Pedro Falcato <pedro.falcato@gmail.com>
> Cc: Vitaly Cheptsov <vit9696@protonmail.com>
>
> Savva Mitrofanov (2):
>   Ext4Pkg: Add symbolic links support
>   Ext4Pkg: Add base containing record macro for EXT4_FILE
>
>  Features/Ext4Pkg/Ext4Dxe/Ext4Dxe.inf |   1 +
>  Features/Ext4Pkg/Ext4Dxe/Ext4Disk.h  |  13 +-
>  Features/Ext4Pkg/Ext4Dxe/Ext4Dxe.h   |  74 +++++-
>  Features/Ext4Pkg/Ext4Dxe/File.c      | 179 +++++++++++---
>  Features/Ext4Pkg/Ext4Dxe/Inode.c     |  15 ++
>  Features/Ext4Pkg/Ext4Dxe/Symlink.c   | 261 ++++++++++++++++++++
>  6 files changed, 499 insertions(+), 44 deletions(-)
>  create mode 100644 Features/Ext4Pkg/Ext4Dxe/Symlink.c
>
> --
> 2.37.3
>
>

-- 
Pedro Falcato

[-- Attachment #2: Type: text/html, Size: 2520 bytes --]