Skip to content

Commit

Permalink
X25: remove bkl in inq and outq ioctls
Browse files Browse the repository at this point in the history
Signed-off-by: Andrew Hendry <andrew.hendry@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
andrew hendry authored and David S. Miller committed Nov 19, 2010
1 parent 1ecd66b commit 54aafbd
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions net/x25/af_x25.c
Original file line number Diff line number Diff line change
Expand Up @@ -1361,12 +1361,10 @@ static int x25_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg)
case TIOCOUTQ: {
int amount;

lock_kernel();
amount = sk->sk_sndbuf - sk_wmem_alloc_get(sk);
if (amount < 0)
amount = 0;
rc = put_user(amount, (unsigned int __user *)argp);
unlock_kernel();
break;
}

Expand All @@ -1377,11 +1375,11 @@ static int x25_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg)
* These two are safe on a single CPU system as
* only user tasks fiddle here
*/
lock_kernel();
lock_sock(sk);
if ((skb = skb_peek(&sk->sk_receive_queue)) != NULL)
amount = skb->len;
release_sock(sk);
rc = put_user(amount, (unsigned int __user *)argp);
unlock_kernel();
break;
}

Expand Down

0 comments on commit 54aafbd

Please sign in to comment.