ACK: [PATCH 2/2] uefi: securebootcert: specify the failure reason for the variable not found
Colin Ian King
colin.king at canonical.com
Tue May 16 08:35:34 UTC 2017
On 16/05/17 09:30, 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: Colin Ian King <colin.king at canonical.com>
More information about the fwts-devel
mailing list