ACK: [PATCH 2/2] uefi: securebootcert: specify the failure reason for the variable not found

Alex Hung alex.hung at canonical.com
Tue May 16 18:08:11 UTC 2017


On 2017-05-16 01:30 AM, Ivan Hu wrote:
> Comments added, no function changed.
>
> Signed-off-by: Ivan Hu <ivan.hu at canonical.com>
> ---
>  src/uefi/securebootcert/securebootcert.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/uefi/securebootcert/securebootcert.c b/src/uefi/securebootcert/securebootcert.c
> index b836855..b431b4c 100644
> --- a/src/uefi/securebootcert/securebootcert.c
> +++ b/src/uefi/securebootcert/securebootcert.c
> @@ -478,10 +478,10 @@ static int securebootcert_test1(fwts_framework *fw)
>  	if (securebooted || deployed) {
>  		if (!(var_found & VAR_DB_FOUND))
>  			fwts_failed(fw, LOG_LEVEL_HIGH, "SecureBootCertVariableNotFound",
> -				"The secure boot variable DB not found.");
> +				"Secureboot or deployed mode on, but the variable DB not found.");
>  		if (!(var_found & VAR_KEK_FOUND))
>  			fwts_failed(fw, LOG_LEVEL_HIGH, "SecureBootCertVariableNotFound",
> -				"The secure boot variable KEK not found.");
> +				"Secureboot or deployed mode on, but the variable KEK not found.");
>  	} else {
>  		if (!(var_found & VAR_DB_FOUND))
>  			fwts_log_info(fw, "Not in readiness for secureboot, variable DB not found.");		
>
Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list