From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Wed, 28 Oct 2015 17:00:09 +0000 (-0400)
Subject: Disable trailing garbage checks
X-Git-Tag: release_3_0_11~197
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0f4c5ef96ee2b27f29a58b57fabb550557bca98a;p=thirdparty%2Ffreeradius-server.git

Disable trailing garbage checks

Gets tripped by too many capture devices (F5/Netscout)
---

diff --git a/src/main/radsniff.c b/src/main/radsniff.c
index 375fe263f24..2e5c4863881 100644
--- a/src/main/radsniff.c
+++ b/src/main/radsniff.c
@@ -1094,12 +1094,22 @@ static void rs_packet_process(uint64_t count, rs_event_t *event, struct pcap_pkt
 				diff, udp_len);
 			return;
 		}
-		/* Trailing data */
+
+#if 0
+		/*
+		 *	It seems many probes add trailing garbage to the end
+		 *	of each capture frame.  This has been observed with
+		 *	the F5 and Netscout.
+		 *
+		 *	Leaving the code here in case it's ever needed for
+		 *	debugging.
+		 */
 		else if (diff < 0) {
 			REDEBUG("Packet too big by %zi bytes, UDP header + Payload should be %hu bytes",
 				diff * -1, udp_len);
 			return;
 		}
+#endif
 	}
 	if ((version == 4) && conf->verify_udp_checksum) {
 		uint16_t expected;