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

[Colin Ian King]

On 18/12/15 06:31, 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: Colin Ian King <colin.king at canonical.com>