netfilter: nfnetlink_log: remove unnecessary error messages
authorPablo Neira Ayuso <pablo@netfilter.org>
Thu, 6 Nov 2014 11:32:28 +0000 (12:32 +0100)
committerPablo Neira Ayuso <pablo@netfilter.org>
Thu, 13 Nov 2014 12:13:00 +0000 (13:13 +0100)
In case of OOM, there's nothing userspace can do.

If there's no room to put the payload in __build_packet_message(),
jump to nla_put_failure which already performs the corresponding
error reporting.

Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nfnetlink_log.c

index b1e3a05794169283ed50d1c0fb4f44d9e7753eeb..51996ece3d2a509430862764ac70b0c55e58da67 100644 (file)
@@ -334,9 +334,6 @@ nfulnl_alloc_skb(struct net *net, u32 peer_portid, unsigned int inst_size,
 
                        skb = nfnetlink_alloc_skb(net, pkt_size,
                                                  peer_portid, GFP_ATOMIC);
-                       if (!skb)
-                               pr_err("nfnetlink_log: can't even alloc %u bytes\n",
-                                      pkt_size);
                }
        }
 
@@ -568,10 +565,8 @@ __build_packet_message(struct nfnl_log_net *log,
                struct nlattr *nla;
                int size = nla_attr_size(data_len);
 
-               if (skb_tailroom(inst->skb) < nla_total_size(data_len)) {
-                       printk(KERN_WARNING "nfnetlink_log: no tailroom!\n");
-                       return -1;
-               }
+               if (skb_tailroom(inst->skb) < nla_total_size(data_len))
+                       goto nla_put_failure;
 
                nla = (struct nlattr *)skb_put(inst->skb, nla_total_size(data_len));
                nla->nla_type = NFULA_PAYLOAD;