Re: [PATCH v2] netfilter: conntrack: remote a return value of the 'seq_print_acct' function.

From: Leon Romanovsky
Date: Mon Jan 23 2023 - 12:49:31 EST


On Mon, Jan 23, 2023 at 02:31:54PM +0000, Gavrilov Ilia wrote:
> The static 'seq_print_acct' function always returns 0.
>
> Change the return value to 'void' and remove unnecessary checks.
>
> Found by InfoTeCS on behalf of Linux Verification Center
> (linuxtesting.org) with SVACE.
>
> Fixes: 1ca9e41770cb ("netfilter: Remove uses of seq_<foo> return values")
> Signed-off-by: Ilia.Gavrilov <Ilia.Gavrilov@xxxxxxxxxxx>
> ---
> V2: Fix coding style
> net/netfilter/nf_conntrack_standalone.c | 12 ++++--------
> 1 file changed, 4 insertions(+), 8 deletions(-)
>

Thanks,
Reviewed-by: Leon Romanovsky <leonro@xxxxxxxxxx>