lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1957790.GKX7oQKdZx@prime>
Date: Thu, 10 Apr 2025 14:42:34 +0200
From: Simon Wunderlich <sw@...onwunderlich.de>
To: Paolo Abeni <pabeni@...hat.com>, Eric Dumazet <edumazet@...gle.com>
Cc: Sven Eckelmann <sven@...fation.org>,
 "David S. Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>,
 netdev@...r.kernel.org
Subject:
 Re: [PATCH net v3] batman-adv: Fix double-hold of meshif when getting enabled

On Thursday, April 10, 2025 1:20:21 PM CEST Eric Dumazet wrote:
> On Thu, Apr 10, 2025 at 12:13 PM Paolo Abeni <pabeni@...hat.com> wrote:
> > On 4/9/25 9:35 AM, Sven Eckelmann wrote:
> > > It was originally meant to replace the dev_hold with netdev_hold. But
> > > this
> > > was missed in this place and thus there was an imbalance when trying to
> > > remove the interfaces.
> > > 
> > > Fixes: 00b35530811f ("batman-adv: adopt netdev_hold() / netdev_put()")
> > > Signed-off-by: Sven Eckelmann <sven@...fation.org>
> > 
> > You must wait at least 24h before reposting, see:
> > 
> > https://elixir.bootlin.com/linux/v6.14-rc6/source/Documentation/process/ma
> > intainer-netdev.rst#L15
> > 
> > Also this is somewhat strange: the same patch come from 2 different
> > persons (sometimes with garbled SoBs), and we usually receive PR for
> > batman patches.
> > 
> > @Svev, @Simon: could you please double check you are on the same page
> > and clarify the intent here?
> 
> Also I do not see credits given to syzbot  team ?
> 
> https://lkml.org/lkml/2025/4/8/1988

Sorry, there were some mistakes on our side. We'll take a deep breath and 
repost this on another day, most likely as part of a usual pull request. Sorry 
for the noise.

Cheers,
       Simon
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ