diff options
| author |   Pablo Neira Ayuso <pablo@netfilter.org> | 2016-09-02 21:00:59 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2016-09-07 11:02:06 +0200 |
| commit | 22609b43b194917dce2188ae9a78bc40a14e67b5 (patch) | |
| tree | 5c6e1932b47032319530ae445ce76ba9e2bded6a /net/netfilter/nft_quota.c | |
| parent | netfilter: nft_quota: fix overquota logic (diff) | |
| download | linux-dev-22609b43b194917dce2188ae9a78bc40a14e67b5.tar.xz linux-dev-22609b43b194917dce2188ae9a78bc40a14e67b5.zip | |
netfilter: nft_quota: introduce nft_overquota()
This is patch renames the existing function to nft_overquota() and make
it return a boolean that tells us if we have exceeded our byte quota.
Just a cleanup.
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net/netfilter/nft_quota.c')
| -rw-r--r-- | net/netfilter/nft_quota.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/net/netfilter/nft_quota.c b/net/netfilter/nft_quota.c index 92b6ff16dbb3..c00104c07095 100644 --- a/net/netfilter/nft_quota.c +++ b/net/netfilter/nft_quota.c @@ -21,10 +21,10 @@ struct nft_quota { atomic64_t remain; }; -static inline long nft_quota(struct nft_quota *priv, - const struct nft_pktinfo *pkt) +static inline bool nft_overquota(struct nft_quota *priv, + const struct nft_pktinfo *pkt) { - return atomic64_sub_return(pkt->skb->len, &priv->remain); + return atomic64_sub_return(pkt->skb->len, &priv->remain) < 0; } static void nft_quota_eval(const struct nft_expr *expr, @@ -33,7 +33,7 @@ static void nft_quota_eval(const struct nft_expr *expr, { struct nft_quota *priv = nft_expr_priv(expr); - if ((nft_quota(priv, pkt) < 0) ^ priv->invert) + if (nft_overquota(priv, pkt) ^ priv->invert) regs->verdict.code = NFT_BREAK; } |