ACK: [PATCH] uefi: uefidump: make array uefidump_info_table const

Alex Hung alex.hung at canonical.com
Tue Sep 5 05:31:13 UTC 2017


On 2017-09-04 06:07 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Array uefidump_info_table is static and read-only so make it
> also const
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/uefi/uefidump/uefidump.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/uefi/uefidump/uefidump.c b/src/uefi/uefidump/uefidump.c
> index 305d2d6a..c4c44b1a 100644
> --- a/src/uefi/uefidump/uefidump.c
> +++ b/src/uefi/uefidump/uefidump.c
> @@ -1379,7 +1379,7 @@ static void uefidump_info_syspreporder(fwts_framework *fw, fwts_uefi_var *var)
>   	free(str);
>   }
>   
> -static uefidump_info uefidump_info_table[] = {
> +static const uefidump_info uefidump_info_table[] = {
>   	{ "PlatformLangCodes",	uefidump_info_platform_langcodes },
>   	{ "PlatformLang",	uefidump_info_platform_lang },
>   	{ "BootOptionSupport", 	uefidump_info_bootoptionsupport },
> @@ -1420,7 +1420,7 @@ static void uefidump_var(fwts_framework *fw, fwts_uefi_var *var)
>   {
>   	char varname[512];
>   	char guid_str[37];
> -	uefidump_info *info;
> +	const uefidump_info *info;
>   
>   	fwts_uefi_get_varname(varname, sizeof(varname), var);
>   
> 

Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list