On Fri, Oct 18, 2019 at 02:42:20PM +0200, Javier Martinez Canillas wrote:
> From: Peter Jones <pjo...@redhat.com>
>
> Add a grub_util_readlink() helper function.
OK but for patches like that one I will be always asking for an explannation
in the commit message why we need this functionality/function(s). So, please
always add it. If you do that for this ptach feel free to add my RB.
> Signed-off-by: Peter Jones <pjo...@redhat.com>
> Reviewed-by: Adam Jackson <a...@redhat.com>
> Signed-off-by: Javier Martinez Canillas <javi...@redhat.com>
> ---
>
> grub-core/osdep/windows/hostdisk.c | 6 ++++++
> include/grub/osdep/hostfile_aros.h | 6 ++++++
> include/grub/osdep/hostfile_unix.h | 6 ++++++
> include/grub/osdep/hostfile_windows.h | 2 ++
> 4 files changed, 20 insertions(+)
>
> diff --git a/grub-core/osdep/windows/hostdisk.c
> b/grub-core/osdep/windows/hostdisk.c
> index 355100789a7..87a106c9b82 100644
> --- a/grub-core/osdep/windows/hostdisk.c
> +++ b/grub-core/osdep/windows/hostdisk.c
> @@ -365,6 +365,12 @@ grub_util_mkdir (const char *dir)
> free (windows_name);
> }
>
> +ssize_t
> +grub_util_readlink (const char *name, char *buf, size_t bufsize)
> +{
> + return readlink(name, buf, bufsize);
> +}
> +
> int
> grub_util_rename (const char *from, const char *to)
> {
> diff --git a/include/grub/osdep/hostfile_aros.h
> b/include/grub/osdep/hostfile_aros.h
> index a059c0fa40a..161fbb7bdfd 100644
> --- a/include/grub/osdep/hostfile_aros.h
> +++ b/include/grub/osdep/hostfile_aros.h
> @@ -68,6 +68,12 @@ grub_util_rename (const char *from, const char *to)
> return rename (from, to);
> }
>
> +static inline ssize_t
> +grub_util_readlink (const char *name, char *buf, size_t bufsize)
> +{
> + return readlink(name, buf, bufsize);
> +}
> +
> #define grub_util_mkdir(a) mkdir ((a), 0755)
>
> struct grub_util_fd
> diff --git a/include/grub/osdep/hostfile_unix.h
> b/include/grub/osdep/hostfile_unix.h
> index 9ffe46fa3ca..17cd3aa8b30 100644
> --- a/include/grub/osdep/hostfile_unix.h
> +++ b/include/grub/osdep/hostfile_unix.h
> @@ -71,6 +71,12 @@ grub_util_rename (const char *from, const char *to)
> return rename (from, to);
> }
>
> +static inline ssize_t
> +grub_util_readlink (const char *name, char *buf, size_t bufsize)
> +{
> + return readlink(name, buf, bufsize);
> +}
> +
> #define grub_util_mkdir(a) mkdir ((a), 0755)
>
> #if defined (__NetBSD__)
> diff --git a/include/grub/osdep/hostfile_windows.h
> b/include/grub/osdep/hostfile_windows.h
> index bf6451b6db4..8c92d0591bb 100644
> --- a/include/grub/osdep/hostfile_windows.h
> +++ b/include/grub/osdep/hostfile_windows.h
> @@ -41,6 +41,8 @@ typedef struct grub_util_fd_dir *grub_util_fd_dir_t;
>
> int
> grub_util_rename (const char *from, const char *to);
> +ssize_t
> +grub_util_readlink (const char *name, char *buf, size_t bufsize);
> int
> grub_util_unlink (const char *name);
> void
> --
> 2.21.0
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel
