ACK: [PATCH 05/12] acpi: checksum: print size_t values using %zd specifier

Alex Hung alex.hung at canonical.com
Thu Sep 6 21:38:36 UTC 2018


On 2018-09-06 07:19 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/acpi/checksum/checksum.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/acpi/checksum/checksum.c b/src/acpi/checksum/checksum.c
> index f475c4e5..8ac019a8 100644
> --- a/src/acpi/checksum/checksum.c
> +++ b/src/acpi/checksum/checksum.c
> @@ -67,10 +67,10 @@ static void checksum_rsdp(fwts_framework *fw, fwts_acpi_table_info *table)
>   		if (table->length < sizeof(fwts_acpi_table_rsdp)) {
>   			fwts_failed(fw, LOG_LEVEL_CRITICAL,
>   				"ACPITableCheckSumShortRSDP",
> -				"RSDP was expected to be %d bytes long, "
> -				"got a shortened size of %d bytes.",
> -				(int)sizeof(fwts_acpi_table_rsdp),
> -				(int)table->length);
> +				"RSDP was expected to be %zd bytes long, "
> +				"got a shortened size of %zd bytes.",
> +				sizeof(fwts_acpi_table_rsdp),
> +				table->length);
>   			/* Won't test on a short RSDP */
>   			return;
>   		}
> 


Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list