ACK: [PATCH] fwts_efi_module.h: constify read-only function arguments

ivanhu ivan.hu at canonical.com
Mon Apr 12 07:23:29 UTC 2021



On 4/11/21 7:19 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Make read-only function arguments const.
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/lib/include/fwts_efi_module.h | 2 +-
>  src/lib/src/fwts_efi_module.c     | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/lib/include/fwts_efi_module.h b/src/lib/include/fwts_efi_module.h
> index 172dfc7c..355616f5 100644
> --- a/src/lib/include/fwts_efi_module.h
> +++ b/src/lib/include/fwts_efi_module.h
> @@ -23,7 +23,7 @@
>  int fwts_lib_efi_runtime_load_module(fwts_framework *fw);
>  int fwts_lib_efi_runtime_unload_module(fwts_framework *fw);
>  int fwts_lib_efi_runtime_open(void);
> -int fwts_lib_efi_runtime_close(int fd);
> +int fwts_lib_efi_runtime_close(const int fd);
>  int fwts_lib_efi_runtime_kernel_lockdown(fwts_framework *fw);
>  int fwts_lib_efi_runtime_module_init(fwts_framework *fw, int *fd);
>  
> diff --git a/src/lib/src/fwts_efi_module.c b/src/lib/src/fwts_efi_module.c
> index 37aa93d9..5669f863 100644
> --- a/src/lib/src/fwts_efi_module.c
> +++ b/src/lib/src/fwts_efi_module.c
> @@ -179,7 +179,7 @@ int fwts_lib_efi_runtime_open(void)
>   *  fwts_lib_efi_runtime_close()
>   *	close the device
>   */
> -int fwts_lib_efi_runtime_close(int fd)
> +int fwts_lib_efi_runtime_close(const int fd)
>  {
>  	return close(fd);
>  }
> 

Acked-by: Ivan Hu <ivan.hu at canonical.com>



More information about the fwts-devel mailing list