Mercurial > dropbear
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"); }