net sched actions: fix sparse warning
authorRoman Mashak <mrv@mojatatu.com>
Wed, 27 Jun 2018 17:33:32 +0000 (13:33 -0400)
committerDavid S. Miller <davem@davemloft.net>
Thu, 28 Jun 2018 13:12:03 +0000 (22:12 +0900)
The variable _data in include/asm-generic/sections.h defines sections,
this causes sparse warning in pedit:

net/sched/act_pedit.c:293:35: warning: symbol '_data' shadows an earlier one
./include/asm-generic/sections.h:36:13: originally declared here

Therefore rename the variable.

Reviewed-by: Simon Horman <simon.horman@netronome.com>
Signed-off-by: Roman Mashak <mrv@mojatatu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/act_pedit.c

index e4b29ee79ba8a41c1b317b5f7d7a4393f30b0cb0..9c2d8a31a5c5e5fe0d6e7f8fc34075283606d97e 100644 (file)
@@ -290,7 +290,7 @@ static int tcf_pedit(struct sk_buff *skb, const struct tc_action *a,
                enum pedit_cmd cmd = TCA_PEDIT_KEY_EX_CMD_SET;
 
                for (i = p->tcfp_nkeys; i > 0; i--, tkey++) {
-                       u32 *ptr, _data;
+                       u32 *ptr, hdata;
                        int offset = tkey->off;
                        int hoffset;
                        u32 val;
@@ -337,7 +337,7 @@ static int tcf_pedit(struct sk_buff *skb, const struct tc_action *a,
                        }
 
                        ptr = skb_header_pointer(skb, hoffset + offset,
-                                                4, &_data);
+                                                4, &hdata);
                        if (!ptr)
                                goto bad;
                        /* just do it, baby */
@@ -355,7 +355,7 @@ static int tcf_pedit(struct sk_buff *skb, const struct tc_action *a,
                        }
 
                        *ptr = ((*ptr & tkey->mask) ^ val);
-                       if (ptr == &_data)
+                       if (ptr == &hdata)
                                skb_store_bits(skb, hoffset + offset, ptr, 4);
                }