ACK: [PATCH] uefi: uefidump: make array uefidump_info_table const
ivanhu
ivan.hu at canonical.com
Wed Sep 6 08:21:22 UTC 2017
On 09/04/2017 09:07 PM, 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: Ivan Hu <ivan.hu at canonical.com>
More information about the fwts-devel
mailing list