aboutsummaryrefslogtreecommitdiffstats
path: root/net/ipv4/igmp.c
diff options
context:
space:
mode:
authorAndreea-Cristina Bernat <bernat.ada@gmail.com>2014-08-17 08:49:41 -0400
committerDavid S. Miller <davem@davemloft.net>2014-08-22 15:23:10 -0400
commite6b688838e25b22e10dd273b48581cb2f62ec16e (patch)
tree25e3de0e0085bc1ea45716d8e955c22fc151476b /net/ipv4/igmp.c
parentb5091b552a9fd7c45f9f8f4efa23bc78af0553cd (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.c2
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 }