aboutsummaryrefslogtreecommitdiffstats
path: root/net/bridge
diff options
context:
space:
mode:
authorJan Engelhardt <jengelh@medozas.de>2010-03-17 19:27:03 -0400
committerJan Engelhardt <jengelh@medozas.de>2010-03-18 09:20:07 -0400
commit85bc3f38147c5d3fb1eb9ca2236536389b592cae (patch)
tree1e853e78c883c5a2dcf72e7d432e7ebf6be04973 /net/bridge
parentf5c511c67aaec323c186543856cfddab31bed1d1 (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.c7
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);