diff options
author | Masahide NAKAMURA <nakam@linux-ipv6.org> | 2006-12-05 13:48:27 -0800 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2006-12-06 18:39:09 -0800 |
commit | 4e33fa14faecc150e97c0e4f2320745bdc7b7112 (patch) | |
tree | 1359902f5a43b3621098d2bac6a7b209ca0da141 | |
parent | 9a217a1c7e7f36ec4996314d64267dd711dbd9bf (diff) | |
download | op-kernel-dev-4e33fa14faecc150e97c0e4f2320745bdc7b7112.zip op-kernel-dev-4e33fa14faecc150e97c0e4f2320745bdc7b7112.tar.gz |
[IPV6] RAW: Don't release unlocked sock.
When user builds IPv6 header and send it through raw socket, kernel
tries to release unlocked sock. (Kernel log shows
"BUG: bad unlock balance detected" with enabled debug option.)
The lock is held only for non-hdrincl sock in this function
then this patch fix to do nothing about lock for hdrincl one.
Signed-off-by: Masahide NAKAMURA <nakam@linux-ipv6.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/ipv6/raw.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/ipv6/raw.c b/net/ipv6/raw.c index c2e629d..4ae1b19a 100644 --- a/net/ipv6/raw.c +++ b/net/ipv6/raw.c @@ -854,7 +854,8 @@ back_from_confirm: } done: dst_release(dst); - release_sock(sk); + if (!inet->hdrincl) + release_sock(sk); out: fl6_sock_release(flowlabel); return err<0?err:len; |