diff options
author | Jan Engelhardt <jengelh@medozas.de> | 2010-03-17 19:27:03 -0400 |
---|---|---|
committer | Jan Engelhardt <jengelh@medozas.de> | 2010-03-18 09:20:07 -0400 |
commit | 85bc3f38147c5d3fb1eb9ca2236536389b592cae (patch) | |
tree | 1e853e78c883c5a2dcf72e7d432e7ebf6be04973 /net/bridge | |
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/bridge')
-rw-r--r-- | net/bridge/netfilter/ebt_ulog.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/net/bridge/netfilter/ebt_ulog.c b/net/bridge/netfilter/ebt_ulog.c index c6ac657074a6..84340ab30ed3 100644 --- a/net/bridge/netfilter/ebt_ulog.c +++ b/net/bridge/netfilter/ebt_ulog.c | |||
@@ -305,13 +305,10 @@ static int __init ebt_ulog_init(void) | |||
305 | ebtulognl = netlink_kernel_create(&init_net, NETLINK_NFLOG, | 305 | ebtulognl = netlink_kernel_create(&init_net, NETLINK_NFLOG, |
306 | EBT_ULOG_MAXNLGROUPS, NULL, NULL, | 306 | EBT_ULOG_MAXNLGROUPS, NULL, NULL, |
307 | THIS_MODULE); | 307 | THIS_MODULE); |
308 | if (!ebtulognl) { | 308 | if (!ebtulognl) |
309 | printk(KERN_WARNING KBUILD_MODNAME ": out of memory trying to " | ||
310 | "call netlink_kernel_create\n"); | ||
311 | ret = -ENOMEM; | 309 | ret = -ENOMEM; |
312 | } else if ((ret = xt_register_target(&ebt_ulog_tg_reg)) != 0) { | 310 | else if ((ret = xt_register_target(&ebt_ulog_tg_reg)) != 0) |
313 | netlink_kernel_release(ebtulognl); | 311 | netlink_kernel_release(ebtulognl); |
314 | } | ||
315 | 312 | ||
316 | if (ret == 0) | 313 | if (ret == 0) |
317 | nf_log_register(NFPROTO_BRIDGE, &ebt_ulog_logger); | 314 | nf_log_register(NFPROTO_BRIDGE, &ebt_ulog_logger); |