aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGuy Harris <guy@alum.mit.edu>2001-10-08 17:37:52 +0000
committerGuy Harris <guy@alum.mit.edu>2001-10-08 17:37:52 +0000
commit971ceec9d233e9e59f9cf0c84c81e7794b7770a3 (patch)
treef34a9fb096f52d7efc0c5ca9f36d7eac8117edaa
parentbebcff62d799ad2c3a9973d5152ee61cedb6ed64 (diff)
Remove the check I added to see whether the length of the packet, based
on the location of the UCB_ETX, is greater than the length of the tvbuff - that can never happen, as the UCB_ETX is, as it was found, definitely inside the tvbuff. svn path=/trunk/; revision=4010
-rw-r--r--packet-ucp.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/packet-ucp.c b/packet-ucp.c
index 886d53fbe3..d99f6a161c 100644
--- a/packet-ucp.c
+++ b/packet-ucp.c
@@ -2,7 +2,7 @@
* Routines for Universal Computer Protocol dissection
* Copyright 2001, Tom Uijldert <tom.uijldert@cmg.nl>
*
- * $Id: packet-ucp.c,v 1.1 2001/10/08 17:30:23 guy Exp $
+ * $Id: packet-ucp.c,v 1.2 2001/10/08 17:37:52 guy Exp $
*
* Ethereal - Network traffic analyzer
* By Gerald Combs <gerald@ethereal.com>
@@ -650,10 +650,6 @@ check_ucp(tvbuff_t *tvb, int *endpkt)
*endpkt = 0;
return UCP_MALFORMED;
}
- if (length > (int) tvb_length(tvb)) {
- *endpkt = 0;
- return UCP_MALFORMED;
- }
for (; offset < (guint) (length - 2); offset++)
checksum += tvb_get_guint8(tvb, offset);
checksum &= 0xFF;