diff options
| author | Jan Engelhardt <jengelh@medozas.de> | 2010-03-18 00:27:03 +0100 |
|---|---|---|
| committer | Jan Engelhardt <jengelh@medozas.de> | 2010-03-18 14:20:07 +0100 |
| commit | 85bc3f38147c5d3fb1eb9ca2236536389b592cae (patch) | |
| tree | 1e853e78c883c5a2dcf72e7d432e7ebf6be04973 /net/netfilter/xt_statistic.c | |
| parent | f5c511c67aaec323c186543856cfddab31bed1d1 (diff) | |
netfilter: xtables: do not print any messages on ENOMEM
ENOMEM is a very obvious error code (cf. EINVAL), so I think we do not
really need a warning message. Not to mention that if the allocation
fails, the user is most likely going to get a stack trace from slab
already.
Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
Diffstat (limited to 'net/netfilter/xt_statistic.c')
| -rw-r--r-- | net/netfilter/xt_statistic.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/net/netfilter/xt_statistic.c b/net/netfilter/xt_statistic.c index d8c0f8f1a78..51ac1bbb4f5 100644 --- a/net/netfilter/xt_statistic.c +++ b/net/netfilter/xt_statistic.c @@ -61,10 +61,8 @@ static bool statistic_mt_check(const struct xt_mtchk_param *par) return false; info->master = kzalloc(sizeof(*info->master), GFP_KERNEL); - if (info->master == NULL) { - printk(KERN_ERR KBUILD_MODNAME ": Out of memory\n"); + if (info->master == NULL) return false; - } info->master->count = info->u.nth.count; return true; |