netfilter: ctnetlink: drop the incorrect cthelper module request
authorLiping Zhang <zlpnobody@gmail.com>
Mon, 17 Apr 2017 13:18:55 +0000 (21:18 +0800)
committerPablo Neira Ayuso <pablo@netfilter.org>
Mon, 24 Apr 2017 18:06:28 +0000 (20:06 +0200)
commit14e567615679a9999ce6bf4f23d6c9e00f03e00e
treed466c2f6163a18a4f16bcd9327730563a5b95f80
parent54a5f9d9abab639039eb7288bdc26c9c67f4e79b
netfilter: ctnetlink: drop the incorrect cthelper module request

First, when creating a new ct, we will invoke request_module to try to
load the related inkernel cthelper. So there's no need to call the
request_module again when updating the ct helpinfo.

Second, ctnetlink_change_helper may be called with rcu_read_lock held,
i.e. rcu_read_lock -> nfqnl_recv_verdict -> nfqnl_ct_parse ->
ctnetlink_glue_parse -> ctnetlink_glue_parse_ct ->
ctnetlink_change_helper. But the request_module invocation may sleep,
so we can't call it with the rcu_read_lock held.

Remove it now.

Signed-off-by: Liping Zhang <zlpnobody@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nf_conntrack_netlink.c