Eric Dumazet 918eb39962 net: add missing bh_unlock_sock() calls
Simon Kirby reported lockdep warnings and following messages :

[104661.897577] huh, entered softirq 3 NET_RX ffffffff81613740
preempt_count 00000101, exited with 00000102?

[104661.923653] huh, entered softirq 3 NET_RX ffffffff81613740
preempt_count 00000101, exited with 00000102?

Problem comes from commit 0e734419
(ipv4: Use inet_csk_route_child_sock() in DCCP and TCP.)

If inet_csk_route_child_sock() returns NULL, we should release socket
lock before freeing it.

Another lock imbalance exists if __inet_inherit_port() returns an error
since commit 093d282321da ( tproxy: fix hash locking issue when using
port redirection in __inet_inherit_port()) a backport is also needed for
>= 2.6.37 kernels.

Reported-by: Simon Kirby <sim@hostway.ca>
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Tested-by: Eric Dumazet <eric.dumazet@gmail.com>
CC: Balazs Scheidler <bazsi@balabit.hu>
CC: KOVACS Krisztian <hidden@balabit.hu>
Reviewed-by: Thomas Gleixner <tglx@linutronix.de>
Tested-by: Simon Kirby <sim@hostway.ca>
Signed-off-by: David S. Miller <davem@davemloft.net>
2011-11-03 18:06:18 -04:00
..
2011-04-22 11:04:14 -07:00
2011-04-22 11:04:14 -07:00
2011-03-12 15:08:49 -08:00
2011-10-13 16:05:07 -04:00
2011-10-19 03:10:46 -04:00
2011-05-12 23:03:46 -04:00
2011-10-19 03:10:46 -04:00
2011-07-01 16:11:15 -07:00
2011-08-12 02:55:28 -07:00
2011-02-01 15:35:25 -08:00
2011-06-08 17:05:30 -07:00
2011-03-31 11:26:23 -03:00
2011-10-24 17:36:08 -04:00
2011-03-31 11:26:23 -03:00
2011-10-24 17:36:08 -04:00
2010-10-27 11:37:32 -07:00
2011-11-02 00:51:27 -04:00
2010-07-12 12:57:54 -07:00