netfilter: nf_tables: check for NULL in nf_tables_newchain pcpu stats allocation
authorSabrina Dubroca <sd@queasysnail.net>
Tue, 21 Oct 2014 09:08:21 +0000 (11:08 +0200)
committerPablo Neira Ayuso <pablo@netfilter.org>
Wed, 22 Oct 2014 12:12:51 +0000 (14:12 +0200)
alloc_percpu returns NULL on failure, not a negative error code.

Fixes: ff3cd7b3c922 ("netfilter: nf_tables: refactor chain statistic routines")
Signed-off-by: Sabrina Dubroca <sd@queasysnail.net>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nf_tables_api.c

index 65eb2a1160d5443cc44fc16bf47f554040358d4f..11ab4b078f3bb68323b1f050f3a1d5e83d271f36 100644 (file)
@@ -1328,10 +1328,10 @@ static int nf_tables_newchain(struct sock *nlsk, struct sk_buff *skb,
                        basechain->stats = stats;
                } else {
                        stats = netdev_alloc_pcpu_stats(struct nft_stats);
-                       if (IS_ERR(stats)) {
+                       if (stats == NULL) {
                                module_put(type->owner);
                                kfree(basechain);
-                               return PTR_ERR(stats);
+                               return -ENOMEM;
                        }
                        rcu_assign_pointer(basechain->stats, stats);
                }