diff options
| author | Michal Kubecek <mkubecek@suse.cz> | 2020-06-07 21:32:50 +0200 |
|---|---|---|
| committer | Michal Kubecek <mkubecek@suse.cz> | 2020-06-07 21:32:50 +0200 |
| commit | c07ea9e10498a09a08cc9b507704471d4d7f505b (patch) | |
| tree | b8e65eab820ee95487ee295c1bf9e7c7317c8399 | |
| parent | 6e4ba0c5e220a0bbdf6be236a0af75548529d0f3 (diff) | |
| download | ethtool-c07ea9e10498a09a08cc9b507704471d4d7f505b.tar.gz | |
netlink: fix unwanted switch fall through in family_info_cb()
Recently added switch branch for CTRL_ATTR_OPS in family_info_cb() is
missing final break statement so that it will fall through into
CTRL_ATTR_MCAST_GROUPS.
Fixes: 6c19c0d559c8 ("netlink: use genetlink ops information to decide about fallback")
Signed-off-by: Michal Kubecek <mkubecek@suse.cz>
| -rw-r--r-- | netlink/netlink.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/netlink/netlink.c b/netlink/netlink.c index ccd8844..17b7788 100644 --- a/netlink/netlink.c +++ b/netlink/netlink.c @@ -211,6 +211,7 @@ static int family_info_cb(const struct nlmsghdr *nlhdr, void *data) ret = genl_read_ops(nlctx, attr); if (ret < 0) return MNL_CB_ERROR; + break; case CTRL_ATTR_MCAST_GROUPS: find_mc_group(nlctx, attr); break; |