diff options
author | Andreea-Cristina Bernat <bernat.ada@gmail.com> | 2014-08-17 08:49:41 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-08-22 15:23:10 -0400 |
commit | e6b688838e25b22e10dd273b48581cb2f62ec16e (patch) | |
tree | 25e3de0e0085bc1ea45716d8e955c22fc151476b /net/ipv4/igmp.c | |
parent | b5091b552a9fd7c45f9f8f4efa23bc78af0553cd (diff) |
net/ipv4/igmp.c: Replace rcu_dereference() with rcu_access_pointer()
The "rcu_dereference()" call is used directly in a condition.
Since its return value is never dereferenced it is recommended to use
"rcu_access_pointer()" instead of "rcu_dereference()".
Therefore, this patch makes the replacement.
The following Coccinelle semantic patch was used:
@@
@@
(
if(
(<+...
- rcu_dereference
+ rcu_access_pointer
(...)
...+>)) {...}
|
while(
(<+...
- rcu_dereference
+ rcu_access_pointer
(...)
...+>)) {...}
)
Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/igmp.c')
-rw-r--r-- | net/ipv4/igmp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/igmp.c b/net/ipv4/igmp.c index f10eab462282..890c4258804c 100644 --- a/net/ipv4/igmp.c +++ b/net/ipv4/igmp.c | |||
@@ -2539,7 +2539,7 @@ static int igmp_mc_seq_show(struct seq_file *seq, void *v) | |||
2539 | querier = "NONE"; | 2539 | querier = "NONE"; |
2540 | #endif | 2540 | #endif |
2541 | 2541 | ||
2542 | if (rcu_dereference(state->in_dev->mc_list) == im) { | 2542 | if (rcu_access_pointer(state->in_dev->mc_list) == im) { |
2543 | seq_printf(seq, "%d\t%-10s: %5d %7s\n", | 2543 | seq_printf(seq, "%d\t%-10s: %5d %7s\n", |
2544 | state->dev->ifindex, state->dev->name, state->in_dev->mc_count, querier); | 2544 | state->dev->ifindex, state->dev->name, state->in_dev->mc_count, querier); |
2545 | } | 2545 | } |