changeset 556:ccdc4c6183c0

- Payload length doesn't include macsize.
author Matt Johnston <matt@ucc.asn.au>
date Sun, 26 Jul 2009 16:11:27 +0000
parents daf52f813328
children bcc5b69d15a6
files debug.h packet.c
diffstat 2 files changed, 2 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/debug.h	Sun Jul 26 15:39:47 2009 +0000
+++ b/debug.h	Sun Jul 26 16:11:27 2009 +0000
@@ -39,7 +39,7 @@
  * Caution: Don't use this in an unfriendly environment (ie unfirewalled),
  * since the printing may not sanitise strings etc. This will add a reasonable
  * amount to your executable size. */
-/*#define DEBUG_TRACE*/
+#define DEBUG_TRACE
 
 /* All functions writing to the cleartext payload buffer call
  * CHECKCLEARTOWRITE() before writing. This is only really useful if you're
--- a/packet.c	Sun Jul 26 15:39:47 2009 +0000
+++ b/packet.c	Sun Jul 26 16:11:27 2009 +0000
@@ -261,7 +261,7 @@
 		
 	/* payload length */
 	/* - 4 - 1 is for LEN and PADLEN values */
-	len = ses.readbuf->len - padlen - 4 - 1;
+	len = ses.readbuf->len - padlen - 4 - 1 - macsize;
 	if ((len > RECV_MAX_PAYLOAD_LEN) || (len < 1)) {
 		dropbear_exit("bad packet size");
 	}