]> git.baikalelectronics.ru Git - kernel.git/commit
sched: cls_flower: fix panic on filter replace
authorDaniel Borkmann <daniel@iogearbox.net>
Fri, 17 Jul 2015 20:38:44 +0000 (22:38 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 21 Jul 2015 07:25:03 +0000 (00:25 -0700)
commitd9ccf398779b7c3b9f9ec27fe56bba496291a245
tree55fd69c3439be010605e609f6a892e7e3795608e
parente911164364f113f7e72a8c62bd5b536617b7dcd5
sched: cls_flower: fix panic on filter replace

The following test case causes a NULL pointer dereference in cls_flower:

  tc filter add dev foo parent 1: flower eth_type ipv4 action ok flowid 1:1
  tc filter replace dev foo parent 1: pref 49152 handle 0x1 \
            flower eth_type ipv6 action ok flowid 1:1

The problem is that commit 89e00481006c ("tc: introduce Flower classifier")
accidentally swapped the arguments of list_replace_rcu(), the old
element needs to be the first argument and the new element the second.

Fixes: 89e00481006c ("tc: introduce Flower classifier")
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Jiri Pirko <jiri@resnulli.us>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/cls_flower.c