Skip to content

Commit c407beb

Browse files
committed
Florian Westphal says: ==================== netfilter updates for net 1) Remove bogus WARN_ON in br_netfilter that came in 6.8. This is now more prominent due to commit 2d72afb ("netfilter: nf_conntrack: fix crash due to removal of uninitialised entry"). From Wang Liang. 2) Better error reporting when a helper module clashes with an existing helper name: -EEXIST makes modprobe believe that the module is already loaded, so error message is elided. From Phil Sutter. * tag 'nf-25-08-27' of https://git.kernel.org/pub/scm/linux/kernel/git/netfilter/nf: netfilter: conntrack: helper: Replace -EEXIST by -EBUSY netfilter: br_netfilter: do not check confirmed bit in br_nf_local_in() after confirm ==================== Link: https://patch.msgid.link/20250827133900.16552-1-fw@strlen.de Signed-off-by: Jakub Kicinski <kuba@kernel.org>
2 parents d6a367e + 54416fd commit c407beb

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

net/bridge/br_netfilter_hooks.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -626,9 +626,6 @@ static unsigned int br_nf_local_in(void *priv,
626626
break;
627627
}
628628

629-
ct = container_of(nfct, struct nf_conn, ct_general);
630-
WARN_ON_ONCE(!nf_ct_is_confirmed(ct));
631-
632629
return ret;
633630
}
634631
#endif

net/netfilter/nf_conntrack_helper.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ int nf_conntrack_helper_register(struct nf_conntrack_helper *me)
368368
(cur->tuple.src.l3num == NFPROTO_UNSPEC ||
369369
cur->tuple.src.l3num == me->tuple.src.l3num) &&
370370
cur->tuple.dst.protonum == me->tuple.dst.protonum) {
371-
ret = -EEXIST;
371+
ret = -EBUSY;
372372
goto out;
373373
}
374374
}
@@ -379,7 +379,7 @@ int nf_conntrack_helper_register(struct nf_conntrack_helper *me)
379379
hlist_for_each_entry(cur, &nf_ct_helper_hash[h], hnode) {
380380
if (nf_ct_tuple_src_mask_cmp(&cur->tuple, &me->tuple,
381381
&mask)) {
382-
ret = -EEXIST;
382+
ret = -EBUSY;
383383
goto out;
384384
}
385385
}

0 commit comments

Comments
 (0)