Public bug reported:

Intro:
======
When hit a kernel warning when using openvswitch kernel module. Digging into 
the source code, we found it's due to the code snippet
        if (WARN_ON(i.cmd >= family->resv_start_op &&
                   (i.doit.validate || i.dumpit.validate)))
                 return -EINVAL;                                                
            

in the gene_validate_ops() in net/netlink/genetlink.c, introduced in 
108880a07bab genetlink: add iterator for walking family ops
from buglink 
https://bugs.launchpad.net/bugs/2053155

How to fix:
===========
We need to cherry-pick the missing patch
Fixes: e4ba4554209f ("net: openvswitch: add missing .resv_start_op")

Author: Jakub Kicinski <k...@kernel.org>
Date:   Thu Oct 27 20:25:01 2022 -0700

    net: openvswitch: add missing .resv_start_op

    I missed one of the families in OvS when annotating .resv_start_op.
    This triggers the warning added in commit ce48ebdd5651 ("genetlink:
    limit the use of validation workarounds to old ops").

    Reported-by: syzbot+40eb8c0447c0e47a7...@syzkaller.appspotmail.com
    Fixes: 9c5d03d36251 ("genetlink: start to validate reserved header bytes")
    Link: https://lore.kernel.org/r/20221028032501.2724270-1-k...@kernel.org
    Signed-off-by: Jakub Kicinski <k...@kernel.org> 

Thanks!

** Affects: linux-bluefield (Ubuntu)
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux-bluefield in Ubuntu.
https://bugs.launchpad.net/bugs/2056718

Title:
  net: openvswitch: add missing .resv_start_op

Status in linux-bluefield package in Ubuntu:
  New

Bug description:
  Intro:
  ======
  When hit a kernel warning when using openvswitch kernel module. Digging into 
the source code, we found it's due to the code snippet
          if (WARN_ON(i.cmd >= family->resv_start_op &&
                     (i.doit.validate || i.dumpit.validate)))
                   return -EINVAL;                                              
              

  in the gene_validate_ops() in net/netlink/genetlink.c, introduced in 
  108880a07bab genetlink: add iterator for walking family ops
  from buglink 
  https://bugs.launchpad.net/bugs/2053155

  How to fix:
  ===========
  We need to cherry-pick the missing patch
  Fixes: e4ba4554209f ("net: openvswitch: add missing .resv_start_op")

  Author: Jakub Kicinski <k...@kernel.org>
  Date:   Thu Oct 27 20:25:01 2022 -0700

      net: openvswitch: add missing .resv_start_op

      I missed one of the families in OvS when annotating .resv_start_op.
      This triggers the warning added in commit ce48ebdd5651 ("genetlink:
      limit the use of validation workarounds to old ops").

      Reported-by: syzbot+40eb8c0447c0e47a7...@syzkaller.appspotmail.com
      Fixes: 9c5d03d36251 ("genetlink: start to validate reserved header bytes")
      Link: https://lore.kernel.org/r/20221028032501.2724270-1-k...@kernel.org
      Signed-off-by: Jakub Kicinski <k...@kernel.org> 

  Thanks!

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux-bluefield/+bug/2056718/+subscriptions


-- 
Mailing list: https://launchpad.net/~kernel-packages
Post to     : kernel-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kernel-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to