[NETLINK]: Ignore control messages directly in netlink_run_queue()

Changes netlink_rcv_skb() to skip netlink controll messages and don't
pass them on to the message handler.

Signed-off-by: Thomas Graf <tgraf@suug.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>

2 files changed, 4 insertions, 6 deletions

diff --git a/net/netlink/af_netlink.c b/net/netlink/af_netlink.c
index 7b455980e9bf..5d1079b1838c 100644
--- a/net/netlink/af_netlink.c
+++ b/net/netlink/af_netlink.c
@@ -1479,6 +1479,10 @@ static int netlink_rcv_skb(struct sk_buff *skb, int (*cb)(struct sk_buff *,
                 if (!(nlh->nlmsg_flags & NLM_F_REQUEST))
                         goto skip;
 
+                /* Skip control messages */
+                if (nlh->nlmsg_type < NLMSG_MIN_TYPE)
+                        goto skip;
+
                 if (cb(skb, nlh, &err) < 0) {
                         /* Not an error, but we have to interrupt processing
                          * here. Note: that in this case we do not pull
diff --git a/net/netlink/genetlink.c b/net/netlink/genetlink.c
index 039516f6cd80..95391e609046 100644
--- a/net/netlink/genetlink.c
+++ b/net/netlink/genetlink.c
@@ -304,9 +304,6 @@ static int genl_rcv_msg(struct sk_buff *skb, struct nlmsghdr *nlh,
         struct genlmsghdr *hdr = nlmsg_data(nlh);
         int hdrlen, err = -EINVAL;
 
-        if (nlh->nlmsg_type < NLMSG_MIN_TYPE)
-                goto ignore;
-
         family = genl_family_find_byid(nlh->nlmsg_type);
         if (family == NULL) {
                 err = -ENOENT;
@@ -364,9 +361,6 @@ static int genl_rcv_msg(struct sk_buff *skb, struct nlmsghdr *nlh,
         *errp = err = ops->doit(skb, &info);
         return err;
 
-ignore:
-        return 0;
-
 errout:
         *errp = err;
         return -1;