diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2010-04-08 14:54:35 +0200 |
---|---|---|
committer | Patrick McHardy <kaber@trash.net> | 2010-04-08 14:54:35 +0200 |
commit | e9f13cab49f7f28b65a6f63201fca56480b2e059 (patch) | |
tree | 050e7b2e8beb53dff692215fac429d0df682fd7b | |
parent | 5dd59cc99131fb4a2775c00928b0f24b9e192a76 (diff) | |
download | op-kernel-dev-e9f13cab49f7f28b65a6f63201fca56480b2e059.zip op-kernel-dev-e9f13cab49f7f28b65a6f63201fca56480b2e059.tar.gz |
netfilter: only do skb_checksum_help on CHECKSUM_PARTIAL in nfnetlink_queue
As we will set ip_summed to CHECKSUM_NONE when necessary in
nfqnl_mangle, there is no need to zap CHECKSUM_COMPLETE in
nfqnl_build_packet_message.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Patrick McHardy <kaber@trash.net>
-rw-r--r-- | net/netfilter/nfnetlink_queue.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/netfilter/nfnetlink_queue.c b/net/netfilter/nfnetlink_queue.c index 7ba4abc..08c1a33 100644 --- a/net/netfilter/nfnetlink_queue.c +++ b/net/netfilter/nfnetlink_queue.c @@ -245,8 +245,7 @@ nfqnl_build_packet_message(struct nfqnl_instance *queue, break; case NFQNL_COPY_PACKET: - if ((entskb->ip_summed == CHECKSUM_PARTIAL || - entskb->ip_summed == CHECKSUM_COMPLETE) && + if (entskb->ip_summed == CHECKSUM_PARTIAL && skb_checksum_help(entskb)) { spin_unlock_bh(&queue->lock); return NULL; |