Messages in this thread | | | Subject | Re: panic on rmmod of nf_conntrack_irc | From | Eric Leblond <> | Date | Tue, 14 Apr 2009 22:46:40 +0200 |
| |
Hi,
Le mardi 14 avril 2009 à 22:14 +0200, Eric Dumazet a écrit : > Mariusz Kozlowski a écrit : > > On Tue, 14 Apr 2009 14:16:37 +0200 > > Patrick McHardy <kaber@trash.net> wrote: > > > >> Eric Dumazet wrote: > >>> Patrick McHardy a écrit : > >>>> Mariusz Kozlowski wrote: > >>>>> netfilter: nf_conntrack: use SLAB_DESTROY_BY_RCU and get rid of > >>>>> call_rcu() > >>>> Thanks for the report. Does this patch fix it? > >>>> > >>> Hi Patrick, sorry for the delay, I was in holidays. > >> No problem, me too :) > >> ...
> > Check commit ca735b3aaa945626ba65a3e51145bfe4ecd9e222 > > netfilter: use a linked list of loggers
...
> Signed-off-by: Eric Leblond <eric@inl.fr> > Signed-off-by: Patrick McHardy <kaber@trash.net> > > It seems "struct list_head list[NFPROTO_NUMPROTO];" is not initialized in "struct nf_logger" ? > > Please try following patch ?
I've just tested your patch. Without it, I was able to trigger the bug (modprobe ebt_ulog ; rmmod ebt_ulog). All run cleanly with it.
All my apologies for having forgot to do this test.
BR, -- Eric Leblond <eric@inl.fr> INL: http://www.inl.fr/ NuFW: http://www.nufw.org/ [unhandled content-type:application/pgp-signature] | |