diff options
| author |   wenxu <wenxu@ucloud.cn> | 2019-12-20 17:08:46 +0800 |
|---|---|---|
| committer |   Pablo Neira Ayuso <pablo@netfilter.org> | 2020-01-05 10:06:22 +0100 |
| commit | 8ca79606cdfde2e37ee4f0707b9d1874a6f0eb38 (patch) | |
| tree | 40559084260b0e9ebdc2130ea41e1ceeef52cc01 /net | |
| parent | netfilter: arp_tables: init netns pointer in xt_tgchk_param struct (diff) | |
| download | linux-dev-8ca79606cdfde2e37ee4f0707b9d1874a6f0eb38.tar.xz linux-dev-8ca79606cdfde2e37ee4f0707b9d1874a6f0eb38.zip | |
netfilter: nft_flow_offload: fix underflow in flowtable reference counter
The .deactivate and .activate interfaces already deal with the reference
counter. Otherwise, this results in spurious "Device is busy" errors.
Fixes: a3c90f7a2323 ("netfilter: nf_tables: flow offload expression")
Signed-off-by: wenxu <wenxu@ucloud.cn>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net')
| -rw-r--r-- | net/netfilter/nft_flow_offload.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/netfilter/nft_flow_offload.c b/net/netfilter/nft_flow_offload.c index dd82ff2ee19f..b70b48996801 100644 --- a/net/netfilter/nft_flow_offload.c +++ b/net/netfilter/nft_flow_offload.c @@ -200,9 +200,6 @@ static void nft_flow_offload_activate(const struct nft_ctx *ctx, static void nft_flow_offload_destroy(const struct nft_ctx *ctx, const struct nft_expr *expr) { - struct nft_flow_offload *priv = nft_expr_priv(expr); - - priv->flowtable->use--; nf_ct_netns_put(ctx->net, ctx->family); } |