netfilter: xtables: fix reentrancy
authorEric Dumazet <eric.dumazet@gmail.com>
Sun, 20 Mar 2011 14:40:06 +0000 (15:40 +0100)
committerPatrick McHardy <kaber@trash.net>
Sun, 20 Mar 2011 14:40:06 +0000 (15:40 +0100)
commitdb856674ac69e31946e56085239757cca3f7655f
tree73fc82e92a119dcf1a0ce4e2ca45ef410165776f
parent5c1aba467828bf0574ec5754c84884d573f590af
netfilter: xtables: fix reentrancy

commit f3c5c1bfd4308 (make ip_tables reentrant) introduced a race in
handling the stackptr restore, at the end of ipt_do_table()

We should do it before the call to xt_info_rdunlock_bh(), or we allow
cpu preemption and another cpu overwrites stackptr of original one.

A second fix is to change the underflow test to check the origptr value
instead of 0 to detect underflow, or else we allow a jump from different
hooks.

Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Cc: Jan Engelhardt <jengelh@medozas.de>
Signed-off-by: Patrick McHardy <kaber@trash.net>
net/ipv4/netfilter/ip_tables.c
net/ipv6/netfilter/ip6_tables.c