IB/netlink: Remove superfluous RDMA_NL_GET_OP() masking
authorMathias Krause <minipli@googlemail.com>
Mon, 30 Sep 2013 20:03:54 +0000 (22:03 +0200)
committerRoland Dreier <roland@purestorage.com>
Fri, 8 Nov 2013 22:42:54 +0000 (14:42 -0800)
'op' is the already RDMA_NL_GET_OP() masked 'type'.  No need to mask it again.

Signed-off-by: Mathias Krause <minipli@googlemail.com>
Reviewed-by: Yann Droneaud <ydroneaud@opteya.com>
Acked-by: Sean Hefty <sean.hefty@intel.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
drivers/infiniband/core/netlink.c

index da06abd..a1e9cba 100644 (file)
@@ -148,7 +148,7 @@ static int ibnl_rcv_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
        list_for_each_entry(client, &client_list, list) {
                if (client->index == index) {
                        if (op < 0 || op >= client->nops ||
-                           !client->cb_table[RDMA_NL_GET_OP(op)].dump)
+                           !client->cb_table[op].dump)
                                return -EINVAL;
 
                        {