diff options
| author | Dan Carpenter <[email protected]> | 2025-08-04 10:35:15 +0000 |
|---|---|---|
| committer | Pablo Neira Ayuso <[email protected]> | 2025-08-07 11:19:26 +0000 |
| commit | f54186df806fb1e9cb262d553f4ff942f9467cf1 (patch) | |
| tree | f068af104f5132cd506ac47e6849367d30bf8c01 /net/core/page_pool.c | |
| parent | netfilter: nft_set_pipapo: don't return bogus extension pointer (diff) | |
| download | kernel-f54186df806fb1e9cb262d553f4ff942f9467cf1.tar.gz kernel-f54186df806fb1e9cb262d553f4ff942f9467cf1.zip | |
netfilter: conntrack: clean up returns in nf_conntrack_log_invalid_sysctl()
Smatch complains that these look like error paths with missing error
codes, especially the one where we return if nf_log_is_registered() is
true:
net/netfilter/nf_conntrack_standalone.c:575 nf_conntrack_log_invalid_sysctl()
warn: missing error code? 'ret'
In fact, all these return zero deliberately. Change them to return a
literal instead which helps readability as well as silencing the warning.
Fixes: e89a68046687 ("netfilter: load nf_log_syslog on enabling nf_conntrack_log_invalid")
Signed-off-by: Dan Carpenter <[email protected]>
Acked-by: Lance Yang <[email protected]>
Signed-off-by: Pablo Neira Ayuso <[email protected]>
Diffstat (limited to 'net/core/page_pool.c')
0 files changed, 0 insertions, 0 deletions