[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2173575.irdbgypaU6@ripper>
Date: Mon, 12 Feb 2024 12:23:51 +0100
From: Sven Eckelmann <sven@...fation.org>
To: syzbot <syzbot+a6a4b5bb3da165594cff@...kaller.appspotmail.com>,
 Eric Dumazet <edumazet@...gle.com>
Cc: a@...table.cc, b.a.t.m.a.n@...ts.open-mesh.org, davem@...emloft.net,
 kuba@...nel.org, linux-kernel@...r.kernel.org, mareklindner@...mailbox.ch,
 netdev@...r.kernel.org, pabeni@...hat.com, sw@...onwunderlich.de,
 syzkaller-bugs@...glegroups.com
Subject: Re: [syzbot] [batman?] BUG: soft lockup in sys_sendmsg
On Monday, 12 February 2024 11:41:38 CET Eric Dumazet wrote:
> This patch [1] looks suspicious
Shouldn't be caused by this - but this might be another way to trigger the 
problem. The problem would be visible even without it when a mtu is explicitly 
set. But the reproducer is not available so I can't actually check what is 
going on.
> I think batman-adv should reject too small MTU values.
You are refering to the size calculated by 
batadv_tt_local_table_transmit_size(), right? And yes, I would agree that it 
looks suspicious and might not have been correctly integrated in 
batadv_max_header_len() when commit a19d3d85e1b8 ("batman-adv: limit local 
translation table max size") introduced the code. But I think we also need to 
remove interfaces again when receiving NETDEV_CHANGEMTU and an interface is 
not having the correctly sized anymore. So have to check how to do this the 
best way.
Kind regards,
	Sven
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists
 
