diff options
| author |   Jan Engelhardt <jengelh@medozas.de> | 2010-03-19 17:16:42 +0100 |
|---|---|---|
| committer | Jan Engelhardt <jengelh@medozas.de> | 2010-03-25 16:03:13 +0100 |
| commit | b0f38452ff73da7e9e0ddc68cd5c6b93c897ca0d (patch) | |
| tree | fd1bbb75556c88a581fc9dfe9faa045578e468ff /net/netfilter/xt_rateest.c | |
| parent | netfilter: xtables: untangle spaghetti if clauses in checkentry (diff) | |
| download | linux-dev-b0f38452ff73da7e9e0ddc68cd5c6b93c897ca0d.tar.xz linux-dev-b0f38452ff73da7e9e0ddc68cd5c6b93c897ca0d.zip | |
netfilter: xtables: change xt_match.checkentry return type
Restore function signatures from bool to int so that we can report
memory allocation failures or similar using -ENOMEM rather than
always having to pass -EINVAL back.
This semantic patch may not be too precise (checking for functions
that use xt_mtchk_param rather than functions referenced by
xt_match.checkentry), but reviewed, it produced the intended result.
// <smpl>
@@
type bool;
identifier check, par;
@@
-bool check
+int check
(struct xt_mtchk_param *par) { ... }
// </smpl>
Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
Diffstat (limited to 'net/netfilter/xt_rateest.c')
| -rw-r--r-- | net/netfilter/xt_rateest.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netfilter/xt_rateest.c b/net/netfilter/xt_rateest.c index 4fc6a917f6de..3b5e3d613b18 100644 --- a/net/netfilter/xt_rateest.c +++ b/net/netfilter/xt_rateest.c @@ -74,7 +74,7 @@ xt_rateest_mt(const struct sk_buff *skb, const struct xt_match_param *par) return ret; } -static bool xt_rateest_mt_checkentry(const struct xt_mtchk_param *par) +static int xt_rateest_mt_checkentry(const struct xt_mtchk_param *par) { struct xt_rateest_match_info *info = par->matchinfo; struct xt_rateest *est1, *est2; |