aboutsummaryrefslogtreecommitdiffstats
path: root/net/ipv4/inet_diag.c
diff options
context:
space:
mode:
authorJulian Anastasov <ja@ssi.bg>2015-07-09 09:59:10 +0300
committerDavid S. Miller <davem@davemloft.net>2015-07-10 18:16:36 -0700
commit2c17d27c36dcce2b6bf689f41a46b9e909877c21 (patch)
tree8868fe137839c66fcef2a435d8d1b38fd0cd7533 /net/ipv4/inet_diag.c
parentnet: do not process device backlog during unregistration (diff)
downloadlinux-dev-2c17d27c36dcce2b6bf689f41a46b9e909877c21.tar.xz
linux-dev-2c17d27c36dcce2b6bf689f41a46b9e909877c21.zip
net: call rcu_read_lock early in process_backlog
Incoming packet should be either in backlog queue or in RCU read-side section. Otherwise, the final sequence of flush_backlog() and synchronize_net() may miss packets that can run without device reference: CPU 1 CPU 2 skb->dev: no reference process_backlog:__skb_dequeue process_backlog:local_irq_enable on_each_cpu for flush_backlog => IPI(hardirq): flush_backlog - packet not found in backlog CPU delayed ... synchronize_net - no ongoing RCU read-side sections netdev_run_todo, rcu_barrier: no ongoing callbacks __netif_receive_skb_core:rcu_read_lock - too late free dev process packet for freed dev Fixes: 6e583ce5242f ("net: eliminate refcounting in backlog queue") Cc: Eric W. Biederman <ebiederm@xmission.com> Cc: Stephen Hemminger <stephen@networkplumber.org> Signed-off-by: Julian Anastasov <ja@ssi.bg> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/inet_diag.c')
0 files changed, 0 insertions, 0 deletions