From 20f72d2b68969f75b0a116e49b133363d4b6fb9a Mon Sep 17 00:00:00 2001
From: Vadim Kochan <vadim4j@gmail.com>
Date: Sat, 16 Jul 2016 12:39:44 +0300
Subject: [PATCH] trafgen: udp: Do not use user-provided 'len' field to
 calculate csum

Do not use user-provided 'len' field for csum calculation which
is potentially dangerous due to possible read buffer overflows.

Signed-off-by: Vadim Kochan <vadim4j@gmail.com>
[tk: reword commit message]
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
---
 trafgen_l4.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/trafgen_l4.c b/trafgen_l4.c
index c1096759..79c59146 100644
--- a/trafgen_l4.c
+++ b/trafgen_l4.c
@@ -44,8 +44,6 @@ static void udp_packet_finish(struct proto_hdr *hdr)
 	if (!lower)
 		return;
 
-	total_len = proto_field_get_u16(hdr, UDP_LEN);
-
 	switch (lower->id) {
 	case PROTO_IP4:
 		csum = p4_csum((void *) proto_header_ptr(lower), proto_header_ptr(hdr),
-- 
2.11.4.GIT