diff options
| author | David S. Miller <davem@sunset.davemloft.net> | 2005-05-04 01:15:59 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2005-05-04 01:15:59 +0200 |
| commit | 0f4821e7b93fe72e89b8ff393bd8e705bd178aa5 (patch) | |
| tree | e9116a4ce6593b186c3962fcdc5baf73f243d0b5 /net/xfrm | |
| parent | [NETLINK]: Fix infinite loops in synchronous netlink changes. (diff) | |
| download | linux-0f4821e7b93fe72e89b8ff393bd8e705bd178aa5.tar.xz linux-0f4821e7b93fe72e89b8ff393bd8e705bd178aa5.zip | |
[XFRM/RTNETLINK]: Decrement qlen properly in {xfrm_,rt}netlink_rcv().
If we free up a partially processed packet because it's
skb->len dropped to zero, we need to decrement qlen because
we are dropping out of the top-level loop so it will do
the decrement for us.
Spotted by Herbert Xu.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/xfrm')
| -rw-r--r-- | net/xfrm/xfrm_user.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/xfrm/xfrm_user.c b/net/xfrm/xfrm_user.c index e8740a4a1d78..5ddda2c98af9 100644 --- a/net/xfrm/xfrm_user.c +++ b/net/xfrm/xfrm_user.c @@ -1024,8 +1024,10 @@ static void xfrm_netlink_rcv(struct sock *sk, int len) if (skb->len) skb_queue_head(&sk->sk_receive_queue, skb); - else + else { kfree_skb(skb); + qlen--; + } break; } kfree_skb(skb); |