diff options
author | brian <brian@FreeBSD.org> | 1999-05-14 09:35:51 +0000 |
---|---|---|
committer | brian <brian@FreeBSD.org> | 1999-05-14 09:35:51 +0000 |
commit | c9568142723fb3671c8a8e7d21d6274a2ef01c58 (patch) | |
tree | 4f51a3acd869db57c53b60a31d4f14aa92042de9 /usr.sbin/ppp | |
parent | 31a93d7badb7cdb6ebdade2dd9407d424ad888c0 (diff) | |
download | FreeBSD-src-c9568142723fb3671c8a8e7d21d6274a2ef01c58.zip FreeBSD-src-c9568142723fb3671c8a8e7d21d6274a2ef01c58.tar.gz |
Ensure that we're not going to overflow our ``struct tun''
when we mbuf_Read() into it.
Add the link name to a few diagnostics.
Diffstat (limited to 'usr.sbin/ppp')
-rw-r--r-- | usr.sbin/ppp/ip.c | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/usr.sbin/ppp/ip.c b/usr.sbin/ppp/ip.c index cc471a5..f6acf85 100644 --- a/usr.sbin/ppp/ip.c +++ b/usr.sbin/ppp/ip.c @@ -17,7 +17,7 @@ * IMPLIED WARRANTIES, INCLUDING, WITHOUT LIMITATION, THE IMPLIED * WARRANTIES OF MERCHANTIBILITY AND FITNESS FOR A PARTICULAR PURPOSE. * - * $Id: ip.c,v 1.59 1999/05/08 11:06:42 brian Exp $ + * $Id: ip.c,v 1.60 1999/05/09 20:02:19 brian Exp $ * * TODO: * o Return ICMP message for filterd packet @@ -395,6 +395,12 @@ ip_Input(struct bundle *bundle, struct link *l, struct mbuf *bp) tun_fill_header(tun, AF_INET); nb = mbuf_Length(bp); + if (nb > sizeof tun.data) { + log_Printf(LogWARN, "ip_Input: %s: Packet too large (got %d, max %d)\n", + l->name, nb, (int)(sizeof tun.data)); + mbuf_Free(bp); + return NULL; + } mbuf_Read(bp, tun.data, nb); if (PacketCheck(bundle, tun.data, nb, &bundle->filter.in) < 0) @@ -410,9 +416,10 @@ ip_Input(struct bundle *bundle, struct link *l, struct mbuf *bp) nw = write(bundle->dev.fd, &tun, nb); if (nw != nb) { if (nw == -1) - log_Printf(LogERROR, "ip_Input: wrote %d, got %s\n", nb, strerror(errno)); + log_Printf(LogERROR, "ip_Input: %s: wrote %d, got %s\n", + l->name, nb, strerror(errno)); else - log_Printf(LogERROR, "ip_Input: wrote %d, got %d\n", nb, nw); + log_Printf(LogERROR, "ip_Input: %s: wrote %d, got %d\n", l->name, nb, nw); } return NULL; |