netfilter: ctnetlink: fix scheduling while atomic if helper is autoloaded
authorPablo Neira Ayuso <pablo@netfilter.org>
Sat, 24 Dec 2011 18:28:47 +0000 (19:28 +0100)
committerPablo Neira Ayuso <pablo@netfilter.org>
Sat, 24 Dec 2011 18:49:04 +0000 (19:49 +0100)
This patch fixes one scheduling while atomic error:

[  385.565186] ctnetlink v0.93: registering with nfnetlink.
[  385.565349] BUG: scheduling while atomic: lt-expect_creat/16163/0x00000200

It can be triggered with utils/expect_create included in
libnetfilter_conntrack if the FTP helper is not loaded.

Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nf_conntrack_netlink.c

index 3d7ea7af76fcb5ef9beb220f21d4b998d08f359c..b6977776d715f9b1fe0eecd99c66bf09261cf2d9 100644 (file)
@@ -1358,12 +1358,15 @@ ctnetlink_create_conntrack(struct net *net, u16 zone,
                                                    nf_ct_protonum(ct));
                if (helper == NULL) {
                        rcu_read_unlock();
+                       spin_unlock_bh(&nf_conntrack_lock);
 #ifdef CONFIG_MODULES
                        if (request_module("nfct-helper-%s", helpname) < 0) {
+                               spin_lock_bh(&nf_conntrack_lock);
                                err = -EOPNOTSUPP;
                                goto err1;
                        }
 
+                       spin_lock_bh(&nf_conntrack_lock);
                        rcu_read_lock();
                        helper = __nf_conntrack_helper_find(helpname,
                                                            nf_ct_l3num(ct),