]> Pileus Git - ~andy/linux/commitdiff
genetlink: rename shadowed variable
authorJohannes Berg <johannes.berg@intel.com>
Mon, 18 Nov 2013 19:54:58 +0000 (20:54 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 18 Nov 2013 20:34:00 +0000 (15:34 -0500)
Sparse pointed out that the new flags variable I had added
shadowed an existing one, rename the new one to avoid that,
making the code clearer.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/netlink/genetlink.c

index f07eb568a1db80e1811dfbf067bf0393426c6343..f54215d7b8f302c5e6b82ef77dafcd440c09be02 100644 (file)
@@ -647,21 +647,21 @@ static int ctrl_fill_info(struct genl_family *family, u32 portid, u32 seq,
                for (i = 0; i < family->n_ops; i++) {
                        struct nlattr *nest;
                        const struct genl_ops *ops = &family->ops[i];
-                       u32 flags = ops->flags;
+                       u32 op_flags = ops->flags;
 
                        if (ops->dumpit)
-                               flags |= GENL_CMD_CAP_DUMP;
+                               op_flags |= GENL_CMD_CAP_DUMP;
                        if (ops->doit)
-                               flags |= GENL_CMD_CAP_DO;
+                               op_flags |= GENL_CMD_CAP_DO;
                        if (ops->policy)
-                               flags |= GENL_CMD_CAP_HASPOL;
+                               op_flags |= GENL_CMD_CAP_HASPOL;
 
                        nest = nla_nest_start(skb, i + 1);
                        if (nest == NULL)
                                goto nla_put_failure;
 
                        if (nla_put_u32(skb, CTRL_ATTR_OP_ID, ops->cmd) ||
-                           nla_put_u32(skb, CTRL_ATTR_OP_FLAGS, flags))
+                           nla_put_u32(skb, CTRL_ATTR_OP_FLAGS, op_flags))
                                goto nla_put_failure;
 
                        nla_nest_end(skb, nest);