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 Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2056718

Title:
  net: openvswitch: add missing .resv_start_op

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


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to