diff options
author | andrew hendry <andrew.hendry@gmail.com> | 2010-11-25 02:18:45 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-11-28 11:12:22 -0800 |
commit | 3f0a069a1d5c0ccace735e3a62c1bcef53e4c354 (patch) | |
tree | 13d1f2bcaf11cb933d35eee19de3d90dc82b5833 | |
parent | 74a7e440807d34e586e9feb8e14851b5c80fbfe5 (diff) | |
download | linux-3f0a069a1d5c0ccace735e3a62c1bcef53e4c354.tar.gz linux-3f0a069a1d5c0ccace735e3a62c1bcef53e4c354.tar.bz2 linux-3f0a069a1d5c0ccace735e3a62c1bcef53e4c354.zip |
X25 remove bkl in call user data length ioctl
Signed-off-by: Andrew Hendry <andrew.hendry@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/x25/af_x25.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/net/x25/af_x25.c b/net/x25/af_x25.c index 8cfc419cef4b..ad96ee90fe27 100644 --- a/net/x25/af_x25.c +++ b/net/x25/af_x25.c @@ -1562,19 +1562,20 @@ out_dtefac_release: case SIOCX25SCUDMATCHLEN: { struct x25_subaddr sub_addr; rc = -EINVAL; - lock_kernel(); + lock_sock(sk); if(sk->sk_state != TCP_CLOSE) - break; + goto out_cud_release; rc = -EFAULT; if (copy_from_user(&sub_addr, argp, sizeof(sub_addr))) - break; + goto out_cud_release; rc = -EINVAL; if(sub_addr.cudmatchlength > X25_MAX_CUD_LEN) - break; + goto out_cud_release; x25->cudmatchlength = sub_addr.cudmatchlength; - unlock_kernel(); rc = 0; +out_cud_release: + release_sock(sk); break; } |