[PATCH] mtd-utils: fectest: Fix time formatting with _TIME_BITS=64 on 32-bit system

Zhihao Cheng chengzhihao1 at huawei.com
Fri Apr 12 19:13:01 PDT 2024


在 2024/4/13 4:55, Ben Hutchings 写道:
> fectest.c formats a struct timeval with the assumption that time_t and
> suseconds_t are aliases for long, but some 32-bit systems now define
> them as long long in order to support timestamps beyond 2038.
> 
> As this is an elapsed time and not a timestamp, both fields should be
> within the range of long, so cast them to long before formatting
> rather than moving to long long.
> 
> Signed-off-by: Ben Hutchings <ben.hutchings at mind.be>
> ---
>   misc-utils/fectest.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Fix compiler warning.
Reviewed-by: Zhihao Cheng <chengzhihao1 at huawei.com>
> 
> diff --git a/misc-utils/fectest.c b/misc-utils/fectest.c
> index eb1d33e..f560f2b 100644
> --- a/misc-utils/fectest.c
> +++ b/misc-utils/fectest.c
> @@ -87,6 +87,6 @@ int main(void)
>   		exit(1);
>   	}
>   
> -	printf("Decoded in %ld.%06lds\n", now.tv_sec, now.tv_usec);
> +	printf("Decoded in %ld.%06lds\n", (long)now.tv_sec, (long)now.tv_usec);
>   	return 0;
>   }
> 




More information about the linux-mtd mailing list