]> git.baikalelectronics.ru Git - kernel.git/commit
ipv6: Plug sk_buff leak in ipv6_rcv (net/ipv6/ip6_input.c)
authorJesper Nilsson <jesper.nilsson@axis.com>
Fri, 27 Mar 2009 07:17:45 +0000 (00:17 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 27 Mar 2009 07:17:45 +0000 (00:17 -0700)
commitf1c6a4490947eecf83445c9034a6ea3686510b84
tree9c8064b98f5432a04d922a9911ce46d340d2c26b
parentb53478de799c819130c7808413ff01f6cdae4ddc
ipv6: Plug sk_buff leak in ipv6_rcv (net/ipv6/ip6_input.c)

Commit afac4212a2647b21d8978cd579e2c437373e6d2f
(ipv6: Add disable_ipv6 sysctl to disable IPv6 operaion on specific interface)
seems to have introduced a leak of sk_buff's for ipv6 traffic,
at least in some configurations where idev is NULL, or when ipv6
is disabled via sysctl.

The problem is that if the first condition of the if-statement
returns non-NULL, it returns an skb with only one reference,
and when the other conditions apply, execution jumps to the "out"
label, which does not call kfree_skb for it.

To plug this leak, change to use the "drop" label instead.
(this relies on it being ok to call kfree_skb on NULL)
This also allows us to avoid calling rcu_read_unlock here,
and removes the only user of the "out" label.

Signed-off-by: Jesper Nilsson <jesper.nilsson@axis.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/ip6_input.c