diff options
| author | Ralf Baechle | 2006-07-11 01:23:21 +0200 |
|---|---|---|
| committer | David S. Miller | 2006-07-12 22:58:43 +0200 |
| commit | 5cc29e3bea77092c48e5c71d792c7461e6a6c4cf (patch) | |
| tree | c72c228a673d61f11e5782d564b912eabda09a64 /net/netrom | |
| parent | [AX.25]: Fix locking of ax25 protocol function list. (diff) | |
| download | kernel-qcow2-linux-5cc29e3bea77092c48e5c71d792c7461e6a6c4cf.tar.gz kernel-qcow2-linux-5cc29e3bea77092c48e5c71d792c7461e6a6c4cf.tar.xz kernel-qcow2-linux-5cc29e3bea77092c48e5c71d792c7461e6a6c4cf.zip | |
[NETROM]: Fix locking order when establishing a NETROM circuit.
When establishing a new circuit in nr_rx_frame the locks are taken in
a different order than in the rest of the stack. This should be
harmless but triggers lockdep. Either way, reordering the code a
little solves the issue.
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/netrom')
| -rw-r--r-- | net/netrom/af_netrom.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/net/netrom/af_netrom.c b/net/netrom/af_netrom.c index ecc796878f38..672d5f803f8f 100644 --- a/net/netrom/af_netrom.c +++ b/net/netrom/af_netrom.c @@ -986,18 +986,18 @@ int nr_rx_frame(struct sk_buff *skb, struct net_device *dev) nr_make->vl = 0; nr_make->state = NR_STATE_3; sk_acceptq_added(sk); - - nr_insert_socket(make); - skb_queue_head(&sk->sk_receive_queue, skb); - nr_start_heartbeat(make); - nr_start_idletimer(make); - if (!sock_flag(sk, SOCK_DEAD)) sk->sk_data_ready(sk, skb->len); bh_unlock_sock(sk); + + nr_insert_socket(make); + + nr_start_heartbeat(make); + nr_start_idletimer(make); + return 1; } |