diff options
author | Alexander V. Chernikov <melifaro@FreeBSD.org> | 2014-01-09 10:40:36 +0000 |
---|---|---|
committer | Alexander V. Chernikov <melifaro@FreeBSD.org> | 2014-01-09 10:40:36 +0000 |
commit | 88d2c8fc8754e31ec2ef85b23a1bcc1aea4b4d85 (patch) | |
tree | dafc21e92d66f03f711c84601f7bcd02b62330a8 /sbin/route | |
parent | 796aef8d174304cada696be47670322e7702f2be (diff) | |
download | src-88d2c8fc8754e31ec2ef85b23a1bcc1aea4b4d85.tar.gz src-88d2c8fc8754e31ec2ef85b23a1bcc1aea4b4d85.zip |
Do some more sanity checks in route(8): require netmask to have
the same address family as destination.
Found by: jmg
MFC after: 2 weeks
Notes
Notes:
svn path=/head/; revision=260472
Diffstat (limited to 'sbin/route')
-rw-r--r-- | sbin/route/route.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/sbin/route/route.c b/sbin/route/route.c index 5fad87535b05..2a76932f68cc 100644 --- a/sbin/route/route.c +++ b/sbin/route/route.c @@ -958,11 +958,18 @@ newroute(int argc, char **argv) } } + /* Do some sanity checks on resulting request */ if (so[RTAX_DST].ss_len == 0) { warnx("destination parameter required"); usage(NULL); } + if (so[RTAX_NETMASK].ss_len != 0 && + so[RTAX_DST].ss_family != so[RTAX_NETMASK].ss_family) { + warnx("destination and netmask family need to be the same"); + usage(NULL); + } + if (nrflags & F_FORCEHOST) { nrflags |= F_ISHOST; #ifdef INET6 |