mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] team: fix qom_list corruption by using list_del_init_rcu()
@ 2025-12-06 18:25 Dharanitharan R
  0 siblings, 0 replies; 4+ messages in thread
From: Dharanitharan R @ 2025-12-06 18:25 UTC (permalink / raw)
  To: syzbot+422806e5f4cce722a71f; +Cc: netdev, linux-kernel, dharanitharan725

Reported-by: syzbot+422806e5f4cce722a71f@syzkaller.appspotmail.com
Signed-off-by: Dharanitharan R <dharanitharan725@gmail.com>
---
 drivers/net/team/team_core.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/team/team_core.c b/drivers/net/team/team_core.c
index 4d5c9ae8f221..d6d724b52dbf 100644
--- a/drivers/net/team/team_core.c
+++ b/drivers/net/team/team_core.c
@@ -823,7 +823,8 @@ static void __team_queue_override_port_del(struct team *team,
 {
 	if (!port->queue_id)
 		return;
-	list_del_rcu(&port->qom_list);
+	/* Ensure safe repeated deletion */
+	list_del_init_rcu(&port->qom_list);
 }
 
 static bool team_queue_override_port_has_gt_prio_than(struct team_port *port,
-- 
2.43.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] team: fix qom_list corruption by using list_del_init_rcu()
  2025-12-07  2:58 ` [PATCH] team: fix qom_list corruption by using list_del_init_rcu() Dharanitharan R
  2025-12-07  2:59   ` syzbot
@ 2025-12-08 17:33   ` Simon Horman
  1 sibling, 0 replies; 4+ messages in thread
From: Simon Horman @ 2025-12-08 17:33 UTC (permalink / raw)
  To: Dharanitharan R; +Cc: syzbot+422806e5f4cce722a71f, netdev, linux-kernel

On Sun, Dec 07, 2025 at 02:58:08AM +0000, Dharanitharan R wrote:
> #syz test: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> 
> Reported-by: syzbot+422806e5f4cce722a71f@syzkaller.appspotmail.com
> Signed-off-by: Dharanitharan R <dharanitharan725@gmail.com>

Hi Dharanitharan,

Please slow down!

It is not appropriate to send multiple versions of a patch CCed to
netdev within in quick succession. Rather, 24h should elapse between
versions. And moreover, I'd advise against CCing netdev on patches
targeted at syzbot testing.

When you do post a fix, for a problem flagged by syzbot, to netdev please
include:

* A description of the problem, and
* How the approach taken fixes it

Typically such a patch will be a bug fix for code present in the net tree.
So it should be targeted at the net tree like this:

Subject: [PATCH net] ...

And it should probably include the following tags:

Fixes:
Reported-by:
Closes:

Especially the Fixes tag.

For an example please see:

https://lore.kernel.org/netdev/20251122002027.695151-1-zlatistiv@gmail.com/

For more information on the Netdev development process please see:

https://docs.kernel.org/process/maintainer-netdev.html

Thanks.

-- 
pw-bot: changes-requested

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] team: fix qom_list corruption by using list_del_init_rcu()
  2025-12-07  2:58 ` [PATCH] team: fix qom_list corruption by using list_del_init_rcu() Dharanitharan R
@ 2025-12-07  2:59   ` syzbot
  2025-12-08 17:33   ` Simon Horman
  1 sibling, 0 replies; 4+ messages in thread
From: syzbot @ 2025-12-07  2:59 UTC (permalink / raw)
  To: dharanitharan725; +Cc: dharanitharan725, linux-kernel, netdev, syzkaller-bugs

> #syz test: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master

This crash does not have a reproducer. I cannot test it.

>
> Reported-by: syzbot+422806e5f4cce722a71f@syzkaller.appspotmail.com
> Signed-off-by: Dharanitharan R <dharanitharan725@gmail.com>
> ---
>  drivers/net/team/team_core.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/team/team_core.c b/drivers/net/team/team_core.c
> index 4d5c9ae8f221..d6d724b52dbf 100644
> --- a/drivers/net/team/team_core.c
> +++ b/drivers/net/team/team_core.c
> @@ -823,7 +823,8 @@ static void __team_queue_override_port_del(struct team *team,
>  {
>  	if (!port->queue_id)
>  		return;
> -	list_del_rcu(&port->qom_list);
> +	/* Ensure safe repeated deletion */
> +	list_del_init_rcu(&port->qom_list);
>  }
>  
>  static bool team_queue_override_port_has_gt_prio_than(struct team_port *port,
> -- 
> 2.43.0
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] team: fix qom_list corruption by using list_del_init_rcu()
  2025-10-30 22:14 [syzbot] [net?] BUG: corrupted list in team_priority_option_set (6) syzbot
@ 2025-12-07  2:58 ` Dharanitharan R
  2025-12-07  2:59   ` syzbot
  2025-12-08 17:33   ` Simon Horman
  0 siblings, 2 replies; 4+ messages in thread
From: Dharanitharan R @ 2025-12-07  2:58 UTC (permalink / raw)
  To: syzbot+422806e5f4cce722a71f; +Cc: netdev, linux-kernel, Dharanitharan R

#syz test: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master

Reported-by: syzbot+422806e5f4cce722a71f@syzkaller.appspotmail.com
Signed-off-by: Dharanitharan R <dharanitharan725@gmail.com>
---
 drivers/net/team/team_core.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/team/team_core.c b/drivers/net/team/team_core.c
index 4d5c9ae8f221..d6d724b52dbf 100644
--- a/drivers/net/team/team_core.c
+++ b/drivers/net/team/team_core.c
@@ -823,7 +823,8 @@ static void __team_queue_override_port_del(struct team *team,
 {
 	if (!port->queue_id)
 		return;
-	list_del_rcu(&port->qom_list);
+	/* Ensure safe repeated deletion */
+	list_del_init_rcu(&port->qom_list);
 }
 
 static bool team_queue_override_port_has_gt_prio_than(struct team_port *port,
-- 
2.43.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-12-08 17:33 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-12-06 18:25 [PATCH] team: fix qom_list corruption by using list_del_init_rcu() Dharanitharan R
  -- strict thread matches above, loose matches on Subject: below --
2025-10-30 22:14 [syzbot] [net?] BUG: corrupted list in team_priority_option_set (6) syzbot
2025-12-07  2:58 ` [PATCH] team: fix qom_list corruption by using list_del_init_rcu() Dharanitharan R
2025-12-07  2:59   ` syzbot
2025-12-08 17:33   ` Simon Horman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®