On 18. 04. 24, 12:47, Donald Hunter wrote:
The NLM_F_ACK flag is ignored for nfnetlink batch begin and end
messages. This is a problem for ynl which wants to receive an ack for
every message it sends, not just the commands in between the begin/end
messages.
Add processing for ACKs for begin/end messages and provide responses
when requested.
I have checked that iproute2, pyroute2 and systemd are unaffected by
this change since none of them use NLM_F_ACK for batch begin/end.
Signed-off-by: Donald Hunter <donald.hunter@xxxxxxxxx>
---
net/netfilter/nfnetlink.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/net/netfilter/nfnetlink.c b/net/netfilter/nfnetlink.c
index c9fbe0f707b5..4abf660c7baf 100644
--- a/net/netfilter/nfnetlink.c
+++ b/net/netfilter/nfnetlink.c
@@ -427,6 +427,9 @@ static void nfnetlink_rcv_batch(struct sk_buff *skb, struct nlmsghdr *nlh,
nfnl_unlock(subsys_id);
+ if (nlh->nlmsg_flags & NLM_F_ACK)
I believe a memset() is missing here:
+ memset(&extack, 0, sizeof(extack));
+ nfnl_err_add(&err_list, nlh, 0, &extack);
+
Otherwise:
[ 36.330875][ T1048] Oops: general protection fault, probably for non-canonical address 0x339e5eab81f1f600: 0000 [#1] PREEMPT SMP NOPTI
[ 36.334610][ T1048] CPU: 1 PID: 1048 Comm: systemd-network Not tainted 6.10.3-1-default #1 openSUSE Tumbleweed 5d3a202ce24e9b465acfbb908cc2eb4f0547bea7
[ 36.335330][ T1048] Hardware name: QEMU Standard PC (Q35 + ICH9, 2009), BIOS 1.16.3-debian-1.16.3-2 04/01/2014
[ 36.335906][ T1048] RIP: 0010:strlen+0x4/0x30
[ 36.336204][ T1048] Code: f7 75 ec 31 c0 e9 17 e0 25 00 48 89 f8 e9 0f e0 25 00 0f 1f 40 00 90 90 90 90 90 90 90 90 90 90 90 90 90 90 90 90 f3 0f 1e fa <80> 3f 00 74 14 48 89 f8 48 83 c0 01 80 38 00 75 f7 48 29 f8 e9 de
[ 36.338921][ T1048] RSP: 0018:ffffb023808f3878 EFLAGS: 00010206
[ 36.339802][ T1048] RAX: 00000000000000c2 RBX: 0000000000000000 RCX: ffff9291ca559620
[ 36.340735][ T1048] RDX: ffff9291ca559620 RSI: 0000000000000000 RDI: 339e5eab81f1f600
[ 36.341177][ T1048] RBP: ffff9291ca559620 R08: 0000000000000000 R09: ffff9291ce8a6500
[ 36.341639][ T1048] R10: 0000000000000001 R11: 0000000000000001 R12: 0000000000000000
[ 36.342063][ T1048] R13: ffff9291c1015680 R14: dead000000000100 R15: ffff9291ce8a6500
[ 36.342517][ T1048] FS: 00007f2ee943d900(0000) GS:ffff92923bd00000(0000) knlGS:0000000000000000
[ 36.342732][ T1048] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 36.342868][ T1048] CR2: 00007f9d4769c000 CR3: 0000000100b82006 CR4: 0000000000370ef0
[ 36.343044][ T1048] Call Trace:
[ 36.343329][ T1048] <TASK>
[ 36.344518][ T1048] ? __die_body.cold+0x14/0x24
[ 36.344831][ T1048] ? die_addr+0x3c/0x60
[ 36.345029][ T1048] ? exc_general_protection+0x1cc/0x3e0
[ 36.345674][ T1048] ? asm_exc_general_protection+0x26/0x30
[ 36.349001][ T1048] ? strlen+0x4/0x30
[ 36.349423][ T1048] ? nf_tables_abort+0x67c/0xee0 [nf_tables c16b4fb993ee603261e060fba374eb60b413741a]
[ 36.350380][ T1048] netlink_ack_tlv_len+0x32/0xb0
[ 36.352876][ T1048] netlink_ack+0x59/0x280
[ 36.353269][ T1048] nfnetlink_rcv_batch+0x60c/0x7e0 [nfnetlink a5ded37673006e964178e189bb08592f3ffd89ce]
extack->_msg is 0x339e5eab81f1f600 (garbage from stack).
See:
https://github.com/systemd/systemd/actions/runs/10282472628/job/28454253577?pr=33958#step:12:30
thanks,
--
js
suse labs
