We don't need local nlbufsiz (skb size) as nfulnl_alloc_skb() takes
the maximum anyway.

Signed-off-by: Michał Mirosław <[EMAIL PROTECTED]>

--- linux-2.6.20/net/netfilter/nfnetlink_log.c.orig     2007-02-10 
18:25:14.000000000 +0100
+++ linux-2.6.20/net/netfilter/nfnetlink_log.c  2007-02-11 20:20:27.000000000 
+0100
@@ -596,7 +596,6 @@ nfulnl_log_packet(unsigned int pf,
        struct nfulnl_instance *inst;
        const struct nf_loginfo *li;
        unsigned int qthreshold;
-       unsigned int nlbufsiz;
        unsigned int plen;
 
        if (li_user && li_user->type == NF_LOG_TYPE_ULOG) 
@@ -672,13 +671,8 @@ nfulnl_log_packet(unsigned int pf,
                return;
        }
 
-       if (size > inst->nlbufsiz)
-               nlbufsiz = size;
-       else
-               nlbufsiz = inst->nlbufsiz;
-
        if (!inst->skb) {
-               if (!(inst->skb = nfulnl_alloc_skb(nlbufsiz, size))) {
+               if (!(inst->skb = nfulnl_alloc_skb(inst->nlbufsiz, size))) {
                        UDEBUG("error in nfulnl_alloc_skb(%u, %u)\n",
                                inst->nlbufsiz, size);
                        goto alloc_failure;
@@ -691,7 +685,7 @@ nfulnl_log_packet(unsigned int pf,
 
                __nfulnl_send(inst);
 
-               if (!(inst->skb = nfulnl_alloc_skb(nlbufsiz, size))) {
+               if (!(inst->skb = nfulnl_alloc_skb(inst->nlbufsiz, size))) {
                        UDEBUG("error in nfulnl_alloc_skb(%u, %u)\n",
                                inst->nlbufsiz, size);
                        goto alloc_failure;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to