ACK: [PATCH] uefi/uefirtvariable: fix options check error messages
ivanhu
ivan.hu at canonical.com
Thu Feb 2 01:32:08 UTC 2017
On 2017年01月20日 20:04, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Fix the name of the options names in the error messages so that
> they match the action option names.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
> src/uefi/uefirtvariable/uefirtvariable.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/uefi/uefirtvariable/uefirtvariable.c b/src/uefi/uefirtvariable/uefirtvariable.c
> index 50bb267..4e3b420 100644
> --- a/src/uefi/uefirtvariable/uefirtvariable.c
> +++ b/src/uefi/uefirtvariable/uefirtvariable.c
> @@ -1900,21 +1900,21 @@ static int options_check(fwts_framework *fw)
>
> if ((uefi_get_variable_multiple < 1) ||
> (uefi_get_variable_multiple > UEFI_GET_VARIABLE_MULTIPLE_MAX)) {
> - fprintf(stderr, "--uefi-get-variable-multiple is %" PRIu32", it "
> + fprintf(stderr, "--uefi-get-var-multiple is %" PRIu32", it "
> "should be 1..%" PRIu32 "\n",
> uefi_get_variable_multiple, UEFI_GET_VARIABLE_MULTIPLE_MAX);
> return FWTS_ERROR;
> }
> if ((uefi_set_variable_multiple < 1) ||
> (uefi_set_variable_multiple > UEFI_SET_VARIABLE_MULTIPLE_MAX)) {
> - fprintf(stderr, "--uefi-set-variable-multiple is %" PRIu32", it "
> + fprintf(stderr, "--uefi-set-var-multiple is %" PRIu32", it "
> "should be 1..%" PRIu32 "\n",
> uefi_set_variable_multiple, UEFI_SET_VARIABLE_MULTIPLE_MAX);
> return FWTS_ERROR;
> }
> if ((uefi_query_variable_multiple < 1) ||
> (uefi_query_variable_multiple > UEFI_QUERY_VARIABLE_MULTIPLE_MAX)) {
> - fprintf(stderr, "--uefi-query-variable-multiple is %" PRIu32", it "
> + fprintf(stderr, "--uefi-query-var-multiple is %" PRIu32", it "
> "should be 1..%" PRIu32 "\n",
> uefi_query_variable_multiple, UEFI_QUERY_VARIABLE_MULTIPLE_MAX);
> return FWTS_ERROR;
>
Acked-by: Ivan Hu <ivan.hu at canonical.com>
More information about the fwts-devel
mailing list