| Message ID | 20221028032501.2724270-1-kuba@kernel.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | e4ba4554209f626c52e2e57f26cba49a62663c8b |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net] net: openvswitch: add missing .resv_start_op | expand |
Hello: This patch was applied to netdev/net.git (master) by Jakub Kicinski <kuba@kernel.org>: On Thu, 27 Oct 2022 20:25:01 -0700 you wrote: > 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+40eb8c0447c0e47a7e9b@syzkaller.appspotmail.com > Fixes: 9c5d03d36251 ("genetlink: start to validate reserved header bytes") > Signed-off-by: Jakub Kicinski <kuba@kernel.org> > > [...] Here is the summary with links: - [net] net: openvswitch: add missing .resv_start_op https://git.kernel.org/netdev/net/c/e4ba4554209f You are awesome, thank you!
diff --git a/net/openvswitch/datapath.c b/net/openvswitch/datapath.c index 155263e73512..8b84869eb2ac 100644 --- a/net/openvswitch/datapath.c +++ b/net/openvswitch/datapath.c @@ -2544,6 +2544,7 @@ struct genl_family dp_vport_genl_family __ro_after_init = { .parallel_ops = true, .small_ops = dp_vport_genl_ops, .n_small_ops = ARRAY_SIZE(dp_vport_genl_ops), + .resv_start_op = OVS_VPORT_CMD_SET + 1, .mcgrps = &ovs_dp_vport_multicast_group, .n_mcgrps = 1, .module = THIS_MODULE,
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+40eb8c0447c0e47a7e9b@syzkaller.appspotmail.com Fixes: 9c5d03d36251 ("genetlink: start to validate reserved header bytes") Signed-off-by: Jakub Kicinski <kuba@kernel.org> --- CC: pshelar@ovn.org CC: paul@paul-moore.com CC: dev@openvswitch.org --- net/openvswitch/datapath.c | 1 + 1 file changed, 1 insertion(+)