ACK: [PATCH 4/6] uefidump: add dumping global variable PlatformRecovery####

[Alex Hung]

On 12/18/2015 02:31 PM, Ivan Hu wrote:
> sync up with uefi spec 2.5
>
> Signed-off-by: Ivan Hu <ivan.hu at canonical.com>
> ---
>   src/uefi/uefidump/uefidump.c | 11 +++++++++++
>   1 file changed, 11 insertions(+)
>
> diff --git a/src/uefi/uefidump/uefidump.c b/src/uefi/uefidump/uefidump.c
> index a618010..43b2b34 100644
> --- a/src/uefi/uefidump/uefidump.c
> +++ b/src/uefi/uefidump/uefidump.c
> @@ -1411,6 +1411,17 @@ static void uefidump_var(fwts_framework *fw, fwts_uefi_var *var)
>   		return;
>   	}
>
> +	/*
> +	 * Check the platformRecovery of boot option PlatformRecovery####. #### is a printed hex value.
> +	 * PlatformRecovery#### variables share the same structure as Boot#### variables.
> +	 */
> +	if ((strlen(varname) == 20) && (strncmp(varname, "PlatformRecovery", 16) == 0)
> +			&& isxdigit(varname[16]) && isxdigit(varname[17])
> +			&& isxdigit(varname[18]) && isxdigit(varname[19])) {
> +		uefidump_info_bootdev(fw, var);
> +		return;
> +	}
> +
>   	/* otherwise just do a plain old hex dump */
>   	uefidump_var_hexdump(fw, var);
>   }
>
Acked-by: Alex Hung <alex.hung at canonical.com>