bridge: kernel panic when unloading bridge module
authorBodo Stroesser <bstroesser@fujitsu-siemens.com>
Tue, 29 Apr 2008 10:18:13 +0000 (03:18 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 29 Apr 2008 10:18:13 +0000 (03:18 -0700)
commitd69efb16891ddfa6c0b527f912a7193054d50281
treef9d3b4a858530e0e63246467cf9d1efdd6abdd05
parent43af8532ecd74a61f9e7aeb27c026c1ee27915ca
bridge: kernel panic when unloading bridge module

There is a race condition when unloading bridge and netfilter.

The problem happens if __fake_rtable is in use by a skb
coming in, while someone starts to unload bridge.ko.
br_netfilter_fini() is called at the beginning of unload
in br_deinit() while skbs still are being forwarded and
transferred to local ip stack. Thus there is a possibility
of the __fake_rtable pointer not being removed in a skb that
goes up to ip stack. This results in a kernel panic, as
ip_rcv() calls the input-function of __fake_rtable, which
is NULL.

Moving the call of br_netfilter_fini() to the end of
br_deinit() solves the problem.

Signed-off-by: Bodo Stroesser <bstroesser@fujitsu-siemens.com>
Signed-off-by: Stephen Hemminger <stephen.hemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br.c