diff options
author | jlemon <jlemon@FreeBSD.org> | 2000-06-02 20:18:38 +0000 |
---|---|---|
committer | jlemon <jlemon@FreeBSD.org> | 2000-06-02 20:18:38 +0000 |
commit | d94fab6e66bc1f43b12c01759d3baaca6e0df5d2 (patch) | |
tree | 9ecbf7d7d025e2c48fe1d6759185766c04e0625c /sys/netinet/ip_input.c | |
parent | 7c8d1a02ff40df8fde524b8406ff5a51ae21e223 (diff) | |
download | FreeBSD-src-d94fab6e66bc1f43b12c01759d3baaca6e0df5d2.zip FreeBSD-src-d94fab6e66bc1f43b12c01759d3baaca6e0df5d2.tar.gz |
Add boundary checks against IP options.
Obtained from: OpenBSD
Diffstat (limited to 'sys/netinet/ip_input.c')
-rw-r--r-- | sys/netinet/ip_input.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sys/netinet/ip_input.c b/sys/netinet/ip_input.c index 763674c..fc355af 100644 --- a/sys/netinet/ip_input.c +++ b/sys/netinet/ip_input.c @@ -1083,7 +1083,7 @@ ip_dooptions(m) goto bad; } optlen = cp[IPOPT_OLEN]; - if (optlen <= 0 || optlen > cnt) { + if (optlen < IPOPT_OLEN + sizeof(*cp) || optlen > cnt) { code = &cp[IPOPT_OLEN] - (u_char *)ip; goto bad; } @@ -1189,6 +1189,10 @@ nosourcerouting: break; case IPOPT_RR: + if (optlen < IPOPT_OFFSET + sizeof(*cp)) { + code = &cp[IPOPT_OFFSET] - (u_char *)ip; + goto bad; + } if ((off = cp[IPOPT_OFFSET]) < IPOPT_MINOFF) { code = &cp[IPOPT_OFFSET] - (u_char *)ip; goto bad; |